Bug 226118 – Merge Review: mailx

Bugzilla will be upgraded to version 5.0 on a still to be determined date in the near future. The original upgrade date has been delayed.

Bug 226118 - Merge Review: mailx

Summary:	Merge Review: mailx

Status:	CLOSED NEXTRELEASE

Aliases:	None

Product:	Fedora
Classification:	Fedora
Component:	Package Review (Show other bugs)
Sub Component:
Version:	rawhide
Hardware:	All Linux

Priority	medium Severity medium
Target Milestone:	---
Target Release:	---
Assigned To:	Marcela Mašláňová
QA Contact:	Fedora Package Reviews List
Docs Contact:

URL:
Whiteboard:
Keywords:

Depends On:
Blocks:
	Show dependency tree / graph

Reported:	2007-01-31 14:36 EST by Nobody's working on this, feel free to take it
Modified:	2007-11-30 17:11 EST (History)
CC List:	3 users (show)

See Also:
Fixed In Version:
Doc Type:	Bug Fix
Doc Text:
Story Points:	---
Clone Of:
Environment:
Last Closed:	2007-08-23 08:07:47 EDT
Type:	---
Regression:	---
Mount Type:	---
Documentation:	---
CRM:
Verified Versions:
Category:	---
oVirt Team:	---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---

Flags:	mmaslano: fedora‑review+

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

Groups: None (edit)

Description Nobody's working on this, feel free to take it 2007-01-31 14:36:35 EST

Fedora Merge Review: mailx

http://cvs.fedora.redhat.com/viewcvs/devel/mailx/
Initial Owner: varekova@redhat.com

Comment 1 Marcela Mašláňová 2007-02-22 07:11:16 EST

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

Comment 2 Ivana Varekova 2007-02-23 09:46:49 EST

Fixed in mailx-8.1.1-45.fc7.

Comment 3 Marcela Mašláňová 2007-02-26 07:24:56 EST

APPROVED

Comment 4 Robert Scheck 2007-03-03 19:50:58 EST

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.

Comment 5 Ivana Varekova 2007-03-05 03:48:21 EST

Thanks for your notice, you are right - the directory is added in
mailx-8.1.1-46.fc7.

Comment 6 Marcela Mašláňová 2007-04-06 02:55:13 EDT

APPROVED

Comment 7 Patrice Dumas 2007-07-07 10:09:39 EDT

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.

Note You need to log in before you can comment on or make changes to this bug.