[Bug 466997] Review Request: sl - Joke command for when you type 'sl' instead of 'ls'

bugzilla at redhat.com bugzilla at redhat.com
Wed Oct 15 08:03:59 UTC 2008


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 <pertusus at free.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pertusus at free.fr




--- Comment #2 from Patrice Dumas <pertusus at free.fr>  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.




More information about the package-review mailing list