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

Fails to build from source on Debian unstable (amd64) #80

Closed
ryanakca opened this issue Aug 3, 2020 · 7 comments
Closed

Fails to build from source on Debian unstable (amd64) #80

ryanakca opened this issue Aug 3, 2020 · 7 comments

Comments

@ryanakca
Copy link
Contributor

ryanakca commented Aug 3, 2020

gophernicus currently fails to build from source on Debian stable:

cc -Wl,-z,relro -Wl,-z,now  gophernicus.o file.o menu.o string.o platform.o session.o options.o -lwrap -o gophernicus
/usr/bin/ld: file.o:/usr/include/tcpd.h:115: multiple definition of `deny_severity'; gophernicus.o:/usr/include/tcpd.h:115: first defined here
/usr/bin/ld: file.o:/usr/include/tcpd.h:114: multiple definition of `allow_severity'; gophernicus.o:/usr/include/tcpd.h:114: first defined here
/usr/bin/ld: menu.o:/usr/include/tcpd.h:115: multiple definition of `deny_severity'; gophernicus.o:/usr/include/tcpd.h:115: first defined here
/usr/bin/ld: menu.o:/usr/include/tcpd.h:114: multiple definition of `allow_severity'; gophernicus.o:/usr/include/tcpd.h:114: first defined here
/usr/bin/ld: string.o:/usr/include/tcpd.h:115: multiple definition of `deny_severity'; gophernicus.o:/usr/include/tcpd.h:115: first defined here
/usr/bin/ld: string.o:/usr/include/tcpd.h:114: multiple definition of `allow_severity'; gophernicus.o:/usr/include/tcpd.h:114: first defined here
/usr/bin/ld: platform.o:/usr/include/tcpd.h:115: multiple definition of `deny_severity'; gophernicus.o:/usr/include/tcpd.h:115: first defined here
/usr/bin/ld: platform.o:/usr/include/tcpd.h:114: multiple definition of `allow_severity'; gophernicus.o:/usr/include/tcpd.h:114: first defined here
/usr/bin/ld: session.o:/usr/include/tcpd.h:115: multiple definition of `deny_severity'; gophernicus.o:/usr/include/tcpd.h:115: first defined here
/usr/bin/ld: session.o:/usr/include/tcpd.h:114: multiple definition of `allow_severity'; gophernicus.o:/usr/include/tcpd.h:114: first defined here
/usr/bin/ld: options.o:/usr/include/tcpd.h:115: multiple definition of `deny_severity'; gophernicus.o:/usr/include/tcpd.h:115: first defined here
/usr/bin/ld: options.o:/usr/include/tcpd.h:114: multiple definition of `allow_severity'; gophernicus.o:/usr/include/tcpd.h:114: first defined here
collect2: error: ld returned 1 exit status

This is Debian bug #966853. The full build log is available from: http://qa-logs.debian.net/2020/08/02/gophernicus_3.0.1-1_unstable.log

@augfab
Copy link
Contributor

augfab commented Aug 4, 2020

I see it failed yesterday as well: http://qa-logs.debian.net/2020/08/03/gophernicus_3.0.1-1_unstable.log

Couple questions:

  • What commit was built? Is it upstream master or salsa's debian/sid? Something else?
  • I can't reproduce locally (Ubuntu 20.04), with either of the mentioned commits. I have an idea for a fix but I can't test if it works.
    Is there a way to test changes manually with Debian's QA?
  • Would you like to upstream the changes you made in gophernicus's Debian packaging scripts?

@fosslinux
Copy link
Collaborator

fosslinux commented Aug 4, 2020

I think the better option is to remove the Debian packaging scripts from the repo entirely.

This is the route most other packages follow, the Debian packaging scripts were only kept as an option for people to build their own packages. If they still want to do that they should do it the official Debian way. I think I will remove the scripts with 3.1.

Anyway, I will investigate the failure.

@ryanakca
Copy link
Contributor Author

ryanakca commented Aug 4, 2020

The Debian package builds the 3.0.1 release (your v3.0.1 tag). It (plus Debian-specific packaging changes) can be found on debian/sid branch on salsa.

If you want to test locally on Ubuntu, you should be able to use the "schroot" tool to create a Debian unstable chroot on Ubuntu without too much trouble. Alternatively, you're welcome to send me the patch and I can test it for you.

And fosslinux is right, the best option is to remove the Debian packaging scripts from the repo entirely.

@fosslinux
Copy link
Collaborator

This bug is already fixed as of the current commit, very recently. The patch is rather large, however, and I plan to release 3.1 soon.

I am unfamiliar with Debian's building process. If it needs to be built sooner rather than later I can release a hotfix 3.0.2.

@ryanakca
Copy link
Contributor Author

ryanakca commented Aug 6, 2020

Thanks, and sorry: I should have done the legwork of checking if it had been fixed in Git.

There is no particular rush. The archive software will automatically remove the package from the "testing" distribution (which becomes the "stable" distribution), probably in a month or so. That said, the stable freeze isn't scheduled to happen until early 2021, so we have time yet to get a fix uploaded if we're to get the package included in the next stable release.

@ryanakca
Copy link
Contributor Author

As a heads up, Debian will enter a freeze on January 12, 2021. That means that for gophernicus to be included in the next Debian release, I need to upload it a week or more before that.

@fosslinux
Copy link
Collaborator

Thanks for the notice. I will have a release by mid-december.

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

3 participants