Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

restore the references just for the project #294

Merged
merged 1 commit into from Oct 26, 2014

Conversation

ctaggart
Copy link
Contributor

Thanks to #267 and the below changes, our build time decreased by only downloading required nuget packages. Our build.cmd now looks like:

.paket\paket.bootstrapper.exe
.paket\paket.exe restore --references-files paket.references
packages\FAKE\tools\FAKE.exe build.fsx %*

@forki
Copy link
Member

forki commented Oct 23, 2014

Yes I forgot about that. /cc @agross this will also help a bit with your parallel restore pain.

@agross
Copy link
Contributor

agross commented Oct 23, 2014

Ok, I'll test it out!

@@ -17,7 +17,8 @@
<PaketBootStrapperCommand Condition=" '$(OS)' == 'Windows_NT'">"$(PaketBootStrapperExePath)"</PaketBootStrapperCommand>
<PaketBootStrapperCommand Condition=" '$(OS)' != 'Windows_NT' ">mono --runtime=v4.0.30319 $(PaketBootStrapperExePath)</PaketBootStrapperCommand>
<!-- Commands -->
<RestoreCommand>$(PaketCommand) restore</RestoreCommand>
<PaketReferences>$(MSBuildProjectDirectory)\paket.references</PaketReferences>
Copy link
Member

Choose a reason for hiding this comment

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

we have a second version of this file myproject.csproj.paket.references which is a special version if you have more than one project in the folder - can you also check for this?

forki added a commit that referenced this pull request Oct 26, 2014
restore the references just for the project
@forki forki merged commit f01ac43 into fsprojects:master Oct 26, 2014
@forki
Copy link
Member

forki commented Oct 26, 2014

I merged this since it's a big improvement, but created #301

dog

@ctaggart ctaggart deleted the justprojectrefs branch June 6, 2015 23:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants