Skip to content
This repository has been archived by the owner on Jul 10, 2022. It is now read-only.

Cleanup delete / add #59

Closed
wants to merge 1 commit into from
Closed

Conversation

guilhem
Copy link
Contributor

@guilhem guilhem commented Sep 30, 2014

⚠️ big changes

@guilhem
Copy link
Contributor Author

guilhem commented Sep 30, 2014

@krobertson this is a new way to manage #57

It's pretty big and lacks tests / verify / all for the moment. But manage pool in a good way.

I will update PR this week to complete missing features.

Feel free to comment, it's pretty big work ^^

@htmldoug
Copy link

htmldoug commented Oct 1, 2014

👍 for adding a Logger.

@htmldoug
Copy link

htmldoug commented Oct 1, 2014

Looks like a lot of refactoring and introduction of a logger. It's going to be a few days before I'd be able to review this in depth.

@guilhem
Copy link
Contributor Author

guilhem commented Oct 2, 2014

Yes... Wow such refactor much logger ^^

But I try to cleanup every parts.

I think it's really easier to read now than before ;)

@guilhem guilhem force-pushed the testpool branch 2 times, most recently from 42d6f77 to f0d963b Compare October 2, 2014 16:28
@guilhem
Copy link
Contributor Author

guilhem commented Oct 2, 2014

Job done!

- Add distribution codename to url_filename Fix krobertson#55
- remove file from pool when delete
- remove package when NOT preserve-version
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.

Package overwrite in pool between distribution
3 participants