Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Jan 15, 2023. It is now read-only.

Enables Use of .NET Global Tools in Build Image#156

Merged
mhart merged 1 commit intolambci:masterlambci/docker-lambda:masterfrom
chrisoverzero:feature/dotnet-global-toolsCopy head branch name to clipboard
May 17, 2019
Merged

Enables Use of .NET Global Tools in Build Image#156
mhart merged 1 commit intolambci:masterlambci/docker-lambda:masterfrom
chrisoverzero:feature/dotnet-global-toolsCopy head branch name to clipboard

Conversation

@chrisoverzero
Copy link
Contributor

@chrisoverzero chrisoverzero commented Jan 9, 2019

Adding the environment variable DOTNET_ROOT and the tools path enables use of .NET Global Tools.

Most useful is the Amazon.Lambda.Tools global tool, enabling commands beginning with dotnet lambda, such as dotnet lambda package. I considered installing this tool by default, but decided it may be too aggressive. If that sounds desirable, I can add another commit to this PR.

dotnetcore2.1/build/Dockerfile Show resolved Hide resolved
Tools currently can't find `dotnet` because it's installed in a
nonstandard location. The environment variable `DOTNET_ROOT` tells
tools where to look.

This and adding the tools path to `PATH` enables use of .NET Global
Tools.
@chrisoverzero
Copy link
Contributor Author

This PR accumulated some merge conflicts, which I've resolved.

@mhart
Copy link
Member

mhart commented May 17, 2019

Nice, I like this, thanks! I'd actually be ok with adding dotnet lambda to the build image too if you want to open a new PR for that 👍

@mhart mhart merged commit e738ca8 into lambci:master May 17, 2019
@mhart
Copy link
Member

mhart commented May 17, 2019

Will let you know when I've built and pushed the new image

@mhart
Copy link
Member

mhart commented May 17, 2019

Having some issues with the way the PATH has changed actually, the Dockerfile won't build for me. Will need to make some changes.

@chrisoverzero
Copy link
Contributor Author

chrisoverzero commented May 17, 2019

Oh, I must've botched the rebase, ugh. Apologies. First two meaningful lines should be:

ENV DOTNET_ROOT=/var/lang/bin
ENV PATH=$DOTNET_ROOT:~/.dotnet/tools:$PATH \

Would you like another PR from me for that, or is it trivial enough to copy-paste from here?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.