Fedora Merge Review: mailx http://cvs.fedora.redhat.com/viewcvs/devel/mailx/ Initial Owner: varekova
Source: W: mailx summary-ended-with-dot The /bin/mail program for sending e-mail messages. W: mailx no-url-tag E: mailx hardcoded-library-path in %{_prefix}/lib Check if hardcoded-library-path is really necessary for backward compatibility. URL: Missing Source: false, not sure what's upstream. Should it be? ftp://ftp.debian.org/debian/pool/main/m/mailx/ Buildroot: replace with new Release: too long format Uselles to define macro datadir. Defatter should have 4 parametres. Arch: W: mailx summary-ended-with-dot The /bin/mail program for sending e-mail messages. W: mailx no-url-tag E: mailx only-non-binary-in-usr-lib -> should be in /usr/share Debug: W: mailx-debuginfo no-url-tag
Fixed in mailx-8.1.1-45.fc7.
APPROVED
Not really...there was one regression introduced during the review: directory /usr/share/mailx is no longer owned by mailx which should be the case. Ivana, please fix this for the next rebuild.
Thanks for your notice, you are right - the directory is added in mailx-8.1.1-46.fc7.
There is still an error: warning: File listed twice: /usr/share/mailx/mail.help warning: File listed twice: /usr/share/mailx/mail.tildehelp More fundamentaly, the upstream selection seems a bit strange to me. It seems to me that the cleanest thing to do would be to package directly http://www.openbsd.org/cgi-bin/cvsweb/src/usr.bin/mail/ (using a cvs checkout, as explained here: http://fedoraproject.org/wiki/Packaging/SourceURL ) If you don't want the latest changes you can use an earlier checkout, but I guess that all the patches in openbsd mail cvs fix something. Seems like debian package the cvs, it seems to me that it would be cleaner to do that than to use the (old) repackaged debian source.