Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 547442 - dev-db/firebird-2.5.3.26780.0-r1 installs logrotate file with wrong permissions
Summary: dev-db/firebird-2.5.3.26780.0-r1 installs logrotate file with wrong permissions
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-04-22 23:13 UTC by William L. Thomson Jr.
Modified: 2015-09-23 16:20 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description William L. Thomson Jr. 2015-04-22 23:13:16 UTC 
The logrotate file for Firebird gets installed with the wrong file permissions. Which leads to the following error.

Cron <root@wlt> test -x /usr/sbin/run-crons && /usr/sbin/run-crons
error: Ignoring firebird because of bad file mode - must be 0644 or 0444.


A simple chmod/fperms 0644 or 0444 of ebuild on install should resolve the problem.

ws wlt # 
diff -Naur firebird-2.5.3.26780.0-r1.ebuild firebird-2.5.3.26780.0-r2.ebuild
--- firebird-2.5.3.26780.0-r1.ebuild       2015-03-21 11:01:06.000000000 -0400
+++ firebird-2.5.3.26780.0-r2.ebuild       2015-04-22 19:11:24.278063510 -0400
@@ -212,6 +212,7 @@
 
        insinto /etc/logrotate.d
        newins "${FILESDIR}/${PN}.logrotate" ${PN}
+       fperms 0644 /etc/logrotate.d/${PN}
 
        diropts -m 755 -o firebird -g firebird
        dodir /var/log/${PN}
Comment 1 William L. Thomson Jr. 2015-09-22 06:21:36 UTC 
Attempted fix via PR, 1 line change turned into entire ebuild review and 16+ comments... Bug still not fixed... :(

https://github.com/gentoo/gentoo/pull/101
Comment 2 William L. Thomson Jr. 2015-09-23 16:20:57 UTC 
Thanks to monsieurp for processing https://github.com/gentoo/gentoo/pull/120