[Bug 225575] Review Request: roundcubemail - Round Cube Webmail is a browser-based multilingual IMAP client

bugzilla at redhat.com bugzilla at redhat.com
Wed May 9 17:39:55 UTC 2007


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: roundcubemail - Round Cube Webmail is a browser-based multilingual IMAP client


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225575





------- Additional Comments From bjohnson at symetrix.com  2007-05-09 13:39 EST -------
(In reply to comment #37)
> E: roundcubemail non-standard-dir-perm /etc/roundcubemail 0640

This is probably an error in your spec file.  You probably want this:
%dir %{_sysconfdir}/%{name}
%attr(0644,root,apache) %config(noreplace) %{_sysconfdir}/*

> E: roundcubemail non-readable /etc/roundcubemail/db.inc.php.dist 0640
> E: roundcubemail non-standard-gid /var/log/roundcubemail apache
> E: roundcubemail non-standard-dir-perm /var/log/roundcubemail 0775

These are expected and can be safely ignored.

> E: roundcubemail non-readable /etc/httpd/conf.d/roundcubemail.conf 0640

There is no particular reason that that you can't have this as 0644.

> W: roundcubemail spurious-executable-perm

Change the mode on these files to 0644 with chmod or install -m in the %install
section.
 
> E: roundcubemail non-readable /etc/roundcubemail/main.inc.php.dist 0640

Expected. 

W: roundcubemail conffile-without-noreplace-flag /etc/roundcubemail/db.inc.php.dist
W: roundcubemail conffile-without-noreplace-flag
/etc/roundcubemail/main.inc.php.dist

It's complaining that these files are not marked with %config(noreplace). 
However, I thought you said you were now installing them as
/etc/roundcubemail/db.inc.php and /etc/roundcubemail/main.inc.php respectively.  



-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.




More information about the Fedora-package-review mailing list