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=509160

--- Comment #17 from David A. Wheeler <dwhee...@dwheeler.com> 2010-01-27 
12:49:09 EST ---
Thanks!

Sadly, the new SRPM creates a new rpmlint error:
 rpmlint mine_detector.spec ../RPMS/x86_64/mine_detector-6.0-3.fc12.x86_64.rpm
../SRPMS/mine_detector-6.0-3.fc12.src.rpm 
mine_detector.x86_64: W: file-not-utf8
/usr/share/doc/mine_detector-6.0/license_clarification.mbox

The guidelines require fixing rpmlint errors unless rpmlint is wrong:
  http://fedoraproject.org/wiki/Packaging/Guidelines#rpmlint

I suggest changing .mbox to .txt, and invoking "iconv" during %prep or %build
to convert the file to UTF-8.  That way, people can get the original file in
its original encoding if they want to (it'll be in the SRPM), but the installed
file will be in UTF-8.  I suspect few people will use mail readers to read this
license clarification material.  Instead, they'll probably pull up their
favorite pager or text editor, which won't really pay attention the character
encoding markup in this file.  Switching to UTF-8 and using a .txt extension
will it easier on those users.

-- 
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.
_______________________________________________
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to