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/control: Remove unneeded debconf pre-dependency #1245

Open
wants to merge 1 commit into
base: debian/sid
Choose a base branch
from

Conversation

gioele
Copy link

@gioele gioele commented Feb 15, 2025

exabgp does not make use of debconf at all, so this (outdated) pre-dependency can be removed. An appropriate dependency will be added by dh_installdebconf if needed.

@gioele gioele changed the title d/control: Remove outdated debconf version constraint debian/control: Remove unneeded debconf pre-dependency Feb 15, 2025
debconf 1.5.34 is older than Debian squeeze, thus no longer needed.
Also, debconf is not needed as Pre-Depends, as it is not called from
`preinst`.

An appropriate dependency will be added by `dh_installdebconf`.

Closes: #1096113
@thomas-mangin
Copy link
Member

thomas-mangin commented Feb 18, 2025

@vincentbernat
Hi Vincent, I would welcome your feedback on this PR. Also, I was wondering if I should spend some time to change the debian packaging on main to create and install a zipapp? Any thought?

@thomas-mangin
Copy link
Member

@gioele Thank you for the PR. I have not build a debian packages in years, hopefully Vincent can just ack. your patch, otherwise I will investigate.

@vincentbernat
Copy link
Member

Ack for me on this one.

If you add a debian/ directory in the master branch, it may create conflicts and make it more difficult to work in debian/sid. But this is a minor inconvenience.

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.

3 participants