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

Fix lowest_matching constraint out of sync #1882

Merged
merged 1 commit into from Aug 25, 2016

Conversation

Projects
None yet
2 participants
@klettier
Contributor

klettier commented Aug 24, 2016

Lowest_matching constraint isn't working when you set it at group level.

Cause : The lowest_matching constraint isn't wrote into paket.lock file when the paket install occurs

You can reproduce the out of sync error with this simple paket.dependencies content

source https://www.nuget.org/api/v2

lowest_matching: true

In paket verion 2.66.9.0 the command succeed but does not populate the paket.lock file

--------------------------------In paket version 2.66.9.0--------------------------------
$ paket install
Paket version 2.66.9.0
Resolving packages for group Main:
Locked version resolution written to ...\paket.lock
0 seconds - ready.

$ paket restore
Paket version 2.66.9.0
0 seconds - ready.

In current paket version the command failed

--------------------------------In paket version 3.16.0.0--------------------------------

$ .paket/paket install
Paket version 3.16.0.0
Resolving packages for group Main:
...\paket.lock is already up-to-date
0 seconds - ready.


$ .paket/paket restore
Paket version 3.16.0.0
paket.dependencies and paket.lock are out of sync in ...
Please run 'paket install' or 'paket update' to recompute the paket.lock file.
0 seconds - ready.

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Aug 25, 2016

Member

cool thanks!

Member

forki commented Aug 25, 2016

cool thanks!

@forki forki merged commit 0aefdba into fsprojects:master Aug 25, 2016

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment