[Bug 402821] Review Request: phpwapmail - WAP-based e-mail client

bugzilla at redhat.com bugzilla at redhat.com
Thu Dec 6 03:12:56 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: phpwapmail - WAP-based e-mail client


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


bugzilla at redhat.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|devel                       |rawhide

limb at jcomserv.net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|nobody at fedoraproject.org    |limb at jcomserv.net
             Status|NEW                         |ASSIGNED
               Flag|                            |fedora-review?




------- Additional Comments From limb at jcomserv.net  2007-12-05 22:12 EST -------
rpmlint:
phpwapmail.src: W: mixed-use-of-spaces-and-tabs (spaces: line 81, tab: line 52)
The specfile mixes use of spaces and tabs for indentation, which is a
cosmetic annoyance.  Use either spaces or tabs for indentation, not both.

Minor

phpwapmail.noarch: E: non-standard-gid /etc/phpwapmail/config.php apache
A file in this package is owned by a non standard group.
Standard groups are:
root, bin, daemon, sys, adm, tty, disk, lp, mem, kmem, wheel, mail,
news, uucp, man, games, gopher, dip, ftp, lock, nobody, users

phpwapmail.noarch: E: non-readable /etc/phpwapmail/config.php 0640
The file can't be read by everybody. If this is expected (for security
reasons), contact your rpmlint distributor to get it added to the list of
exceptions for your distro (or add it to your local configuration if you
installed rpmlint from the source tarball).

Probably fine.

More to follow.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.




More information about the Fedora-package-review mailing list