Skip to content
This repository was archived by the owner on May 31, 2019. It is now read-only.

Added Dockerfile to download the nightly build of ASP.NET 5 #20

Merged
merged 1 commit into from
Dec 27, 2014
Merged

Added Dockerfile to download the nightly build of ASP.NET 5 #20

merged 1 commit into from
Dec 27, 2014

Conversation

vlesierse
Copy link
Contributor

When a developer would like to run against the latest build of ASP.NET 5 he can use the microsoft/aspnet:nightly image. This image will point to the https://www.myget.org/F/aspnetvnext/api/v2 feed to get the runtime and us an ONBUILD to get the latest version of KRE.

@giggio has brought this improvement in issue: Create container for aspnet latest dev version


RUN apt-get -qq update && apt-get -qqy install unzip

RUN curl -sSL https://raw.githubusercontent.com/aspnet/Home/master/kvminstall.sh | sh

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this use the kvminstall.sh from the dev branch as well?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I think it should... The master branch has the kvminstall from the latest NuGet.org ("official") release. Granted, they're probably not very different, but might as well use the best version.

@Eilon
Copy link
Contributor

Eilon commented Dec 22, 2014

@ahmetalpbalkan for review.

And CLA is confirmed.

@vlesierse
Copy link
Contributor Author

Good idea. I have also change the RUN command to ONBUILD RUN. This way it will always takes the latest version from the dev branch.

@Eilon
Copy link
Contributor

Eilon commented Dec 23, 2014

@ahmetalpbalkan if you can review and sign off then we can merge, thanks!

@ahmetb
Copy link
Contributor

ahmetb commented Dec 24, 2014

@Eilon I'm OOF until 1/12 sorry for the delay.

@vlesierse Thanks for the PR! Can you please verify if aspnet/Home@dev is really nightly and installs a newer version than master? Also, is env[KRE_FEED] value used by kvminstall.sh implicitly or did you forget to use it?

Edit: I realized kvm.sh uses it. Looks good to me, please merge. :shipit: Once merged I'll add the tag to Docker Hub image.

@vlesierse
Copy link
Contributor Author

Is there a need to squash the commits first?

@ahmetb
Copy link
Contributor

ahmetb commented Dec 24, 2014

@vlesierse yes, that'd be great. Squash them into one and git push -f, please. Thanks!

@vlesierse
Copy link
Contributor Author

Done

@ahmetb
Copy link
Contributor

ahmetb commented Dec 24, 2014

squash LGTM.

@Eilon
Copy link
Contributor

Eilon commented Dec 25, 2014

Cool!

@davidfowl if your Barbadian Interwebz are working, can you merge?

davidfowl added a commit that referenced this pull request Dec 27, 2014
Added Dockerfile to download the nightly build of ASP.NET 5
@davidfowl davidfowl merged commit a6e36e8 into aspnet:master Dec 27, 2014
@vlesierse vlesierse deleted the nightly branch March 6, 2015 12:14
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.

6 participants