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

Partial update #110

Closed
wants to merge 3 commits into from
Closed

Partial update #110

wants to merge 3 commits into from

Conversation

jkufner
Copy link

@jkufner jkufner commented Jan 10, 2014

This updates and enhances oker1's pull request #84.

Repository configuration from satis.json is now used. URL is used only to find appropriate configuration in satis.json. If URL is not found, an exception is thrown.

It also renames --repository option to --repository-url to better describe expected value.

It performs full update if cache file is not found.

Fixes #40.

oker1 and others added 3 commits August 5, 2013 20:22
Conflicts:
	src/Composer/Satis/Command/BuildCommand.php
Option is now --repository-url, because it better describes expected
value.

If repository URL is not found in configuration, an exception is thrown,
to ensure consistency of partial builds with full build.
@oker1
Copy link
Contributor

oker1 commented Jan 13, 2014

Thanks a lot for taking on this task!

Unfortunately I didn't have time to work on this.

@jkufner
Copy link
Author

jkufner commented Feb 3, 2014

That is old commit. I fixed it already in 4b5f4b0. Looks like Github has old ref somewhere...

@oker1
Copy link
Contributor

oker1 commented Feb 3, 2014

My mistake, sorry. Thanks again for this one!

@oker1
Copy link
Contributor

oker1 commented Feb 8, 2014

I've tried this patch, it works great apart from an issue with dumping dists. If we are only updating one repository it should dump only that repo's dists, but instead it dumps every one.

@mathroc
Copy link
Contributor

mathroc commented Jan 8, 2015

👍

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.

Selective update
3 participants