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

debian: depend on brz instead of bzr #507

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

bluca
Copy link
Contributor

@bluca bluca commented Jan 9, 2025

bzr is a deprecated, transitional empty package, pointingn to brz, since Debian 11

@bluca
Copy link
Contributor Author

bluca commented Jan 9, 2025

Dropping it entirely as per #505 it's even better of course

@M0ses M0ses self-requested a review January 22, 2025 08:40
M0ses
M0ses previously approved these changes Jan 22, 2025
Copy link
Collaborator

@M0ses M0ses left a comment

Choose a reason for hiding this comment

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

LGTM, approved

@M0ses
Copy link
Collaborator

M0ses commented Jan 22, 2025

I think I was a bit to fast with approving.

Wouldn't it make sense to use Suggests: instead of Depends for brz?

Even if I'm reluctant to drop the bzr code entirely, I would like get rid of hard dependencies on it (both brz and bzr).

bzr is a deprecated, transitional empty package, pointingn to brz,
since Debian 11, so switch to brz.
Also downgrade from depend to recommend, as it is not used very widely.
@bluca
Copy link
Contributor Author

bluca commented Jan 22, 2025

Sure, downgraded from depends: to recommends: same as mercurial

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.

2 participants