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


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


Patrice Dumas <[EMAIL PROTECTED]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[EMAIL PROTECTED]




--- Comment #2 from Patrice Dumas <[EMAIL PROTECTED]>  2008-10-15 04:03:58 EDT 
---
The mail transcript should be directly in the package as asource, so

Source2: http://marcbradshaw.co.uk/packages/review/sl/sl-license-mail.txt

would work, with cp and adding to %doc.

I suggest removing the /bin/ and let the commands be searched on the path.

You should use cp -p and install -p (for the manpage) to keep timestamps.

Also I suggest doing something along:

iconv -f iso-2022-jp README -t utf8 > README.conv README.conv && \
 touch -c -r README && \
 mv README.conv README

Also the -f of rm is not needed, it is always the default in rpm.

-- 
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.

_______________________________________________
Fedora-package-review mailing list
[email protected]
http://www.redhat.com/mailman/listinfo/fedora-package-review

Reply via email to