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 directory improvements #21

Closed
slavkoja opened this issue Jan 24, 2019 · 3 comments
Closed

debian directory improvements #21

slavkoja opened this issue Jan 24, 2019 · 3 comments

Comments

@slavkoja
Copy link

slavkoja commented Jan 24, 2019

Hi,

i tried to build debian package on current testing (buster) and it provides empty binary package. After small investigation, i have some suggestions to improve it:

Make its build dependencies to debhelper>10 (it is available in stable and via backport on oldstable too and it depends on dh_autoreconf, which is needed) and add bats into it (all test fails without it) into debian/control:

Build-Depends: debhelper (>= 10~), bats

Do not use native format, it is not recommended and it is a problem with it, if upstream (you) provides debian directory, you can read more at mentors, then i suggest to change debian/source/format to:

3.0 (quilt)

This requires to add debian's version into debian/changelog, eg.:

netcalc (2.1.4-1) unstable; urgency=medium

And finally, please do not install ipcalc symlink and remove conflict with it. From my point of view (as exmaintainer some official debian packages) it is too intrusive to force people to uninstall package, which can coexistent without any problems (i did it internally).

@slavkoja
Copy link
Author

I forget to mention, that call to dh_autoreconf have to be added into debian/rules:

%:
    dh $@ --with autoreconf

@troglobit
Copy link
Owner

Packaging preferences seem to be different for every Debian developer I talk to. The native form we use here is what is also used by the Inadyn project, and the pros and cons of native vs quilt were discussed at length there. So I'm going to deny your request to change to quilt.

The other points you bring up are interesting though:

  • Step debhelper version, yup that's a big todo for most of my projects
  • Symlink to ipcalc, the best way I guess is to provide a separate package for it. E.g. netcalc-ipcalc_foo.deb
  • Regarding autoreconf in rules, thanks!

@slavkoja
Copy link
Author

slavkoja commented Feb 3, 2019

Ad ipcalc, IMO it is best to manage it as alternative (via update-alternatives) but without cooperation with ipcalc maintainer, you will need to bother with with dpkg-divert and i never tried it for this. If someone want to use netcalc as ipcalc, it can simple make alias for it ;-)

Ad native/quilt, the main difference is, when package maintains someone other than you. In quilt type package, when i do source update automatically via uscan, my debian directory changes are retained and relevant things are updated by uscan. With native packages i need to reapply all my changes in debian directory manually after any update. AFAIK this is main reason, why quilt type is recommended, while it is a bit complicated to work with it for accidental/unexperienced users. It is, of course, your decision, but if you want, you can see and try it from my personal repo https://debs.slavino.sk/pool/main/n/netcalc/

troglobit added a commit that referenced this issue May 24, 2020
Fix issue #21

Signed-off-by: Joachim Nilsson <troglobit@gmail.com>
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

No branches or pull requests

2 participants