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

GitLab CI broken: Lintian failure because of binary in /etc (profstats) #4733

Closed
kmk3 opened this issue Dec 1, 2021 · 4 comments
Closed
Labels
bug Something isn't working

Comments

@kmk3
Copy link
Collaborator

kmk3 commented Dec 1, 2021

Likely caused by commit 8f93df9 ("install profstats in /etc/firejail
directory - undocumented, used only for development", 2021-11-30).

Failed pipelines for that commit:

  • build_debian_package

https://gitlab.com/Firejail/firejail_ci/-/jobs/1833634608

*****************************************
install size 6368	firejail-0.9.67/debian
*****************************************
dpkg-deb: building package 'firejail' in 'debian.deb'.
warning: the authors of lintian do not recommend running it with root privileges!
E: firejail: binary-in-etc etc/firejail/profstats
W: firejail: executable-is-not-world-readable usr/lib/firejail/fcopy 0711
W: firejail: executable-is-not-world-readable usr/lib/firejail/fldd 0711
W: firejail: executable-is-not-world-readable usr/lib/firejail/fnet 0711
W: firejail: executable-is-not-world-readable usr/lib/firejail/fnetfilter 0711
W: firejail: executable-is-not-world-readable usr/lib/firejail/fsec-optimize 0711
W: firejail: executable-is-not-world-readable usr/lib/firejail/fsec-print 0711
W: firejail: executable-is-not-world-readable usr/lib/firejail/fseccomp 0711
W: firejail: executable-is-not-world-readable usr/lib/firejail/fshaper.sh 0711
N: 1 tag overridden (1 warning)
Makefile:224: recipe for target 'deb' failed
make: *** [deb] Error 1
Cleaning up project directory and file based variables 00:01
ERROR: Job failed: exit code 2
  • debian_ci

https://gitlab.com/Firejail/firejail_ci/-/jobs/1833634618

make[1]: Entering directory '/builds/Firejail/firejail_ci'
dh_missing -pfirejail --fail-missing
dh_missing: warning: etc/firejail/profstats exists in debian/tmp but is not installed to anywhere 
dh_missing: error: missing files, aborting
	The following debhelper tools have reported what they installed (with files per package)
	 * dh_install: firejail (44), firejail-profiles (1174)
	 * dh_installdocs: firejail (5), firejail-profiles (0)
	 * dh_installman: firejail (0), firejail-profiles (0)
	If the missing files are installed by another tool, please file a bug against it.
	When filing the report, if the tool is not part of debhelper itself, please reference the
	"Logging helpers and dh_missing" section from the "PROGRAMMING" guide for debhelper (10.6.3+).
	  (in the debhelper package: /usr/share/doc/debhelper/PROGRAMMING.gz)
	Be sure to test with dpkg-buildpackage -A/-B as the results may vary when only a subset is built
	If the omission is intentional or no other helper can take care of this consider adding the
	paths to debian/not-installed.
make[1]: *** [debian/rules:34: override_dh_missing] Error 25
make[1]: Leaving directory '/builds/Firejail/firejail_ci'
make: *** [debian/rules:8: binary] Error 2
dpkg-buildpackage: error: debian/rules binary subprocess returned exit status 2
debuild: fatal error at line 1182:
dpkg-buildpackage -us -uc -ui -i -I failed
gbp:error: 'debuild -i -I -uc -us' failed: it exited with 29
Cleaning up project directory and file based variables 00:00
ERROR: Job failed: exit code 1

Cc: @netblue30 @reinerh

@rusty-snake
Copy link
Collaborator

rpmlint: firejail.x86_64: E: binary-in-etc /etc/firejail/profstats

Probably every distro will need to patch this out because it violates packaging-guidelines.

@reinerh
Copy link
Collaborator

reinerh commented Dec 1, 2021

Yes, this should really not get installed there. LIB(EXEC)DIR is probably a better place. Maybe it can also be "hidden" behind the configure flag that enables installation of contrib scripts?

@netblue30 netblue30 added the bug Something isn't working label Dec 8, 2021
@netblue30
Copy link
Owner

I'll put a fix in, thanks.

netblue30 added a commit that referenced this issue Dec 10, 2021
@kmk3
Copy link
Collaborator Author

kmk3 commented Dec 10, 2021

Fixed by commit 30e7546 ("profstats fix (#4733)", 2021-12-10).

The GitLab pipeline runs successfully now:

@kmk3 kmk3 closed this as completed Dec 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done (RELNOTES N/A)
Development

No branches or pull requests

4 participants