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
treewide: replace bazaar with breezy #85539
Conversation
7967049
to
fadb56c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I approve of this PR, less python2-only packages is a plus.
csv2svn only uses bazaar as checkInput
not much to stop working here as long as the tests succeed.
It would probably a good idea to add the deprecation notice into the release-notes and maybe even add some documentation to the manual, however i would not even know where to put it
Right, I now mentioned the change in the release notes. Thank you. |
Bazaar is deprecated in favour of Breezy, which is backward compatible Python 3 fork of the original VCS.
Rebased. I think it has received enough feedback, now: I'm going to merge. |
Motivation for this change
Fix #80740
Things done
./result/bin/
)nix-review report
Result of
nixpkgs-review
19 package built:
closure size increase
As @jtojnar noticed, breezy constitues a large increase in closure size due to an issue in
python3Packages.secretstorage
. The solution is trivial but I decided to deal with this in another PR not to mix up the two changes in the results of nix-review.edit: Fixed in #85571