Bug 448788 - LANG="" repodiff can create unicode errors
Summary: LANG="" repodiff can create unicode errors
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Fedora
Classification: Fedora
Component: yum-utils
Version: rawhide
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Seth Vidal
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-05-28 19:57 UTC by Jesse Keating
Modified: 2013-01-10 03:20 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2008-06-03 19:07:54 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Jesse Keating 2008-05-28 19:57:21 UTC
LANG="" repodiff
--old=file:///mnt/koji/mash/rawhide-20080527/development/source/SRPMS/
--new=file:///mnt/koji/mash/rawhide-20080528/development/source/SRPMS/

kdeutils-4.0.80-1.fc10
----------------------
* Mon May 26 18:00:00 2008 Than Ngo <than> 4.0.80-1
- 4.1 beta1


Traceback (most recent call last):
  File "/usr/bin/repodiff", line 227, in <module>
    main(sys.argv[1:])
  File "/usr/bin/repodiff", line 211, in main
    print msg
UnicodeEncodeError: 'ascii' codec can't encode character u'\xf8' in position
236: ordinal not in range(128)

Calling it with LANG=en_US works.  This comes about because cron doesn't set a
LANG when it runs, so anytime repodiff is ran via cron you can get a traceback.

Comment 1 Tim Lauridsen 2008-05-29 11:09:40 UTC
Jesse, please test if this one is working better.

http://devel.linux.duke.edu/gitweb/?p=yum-utils.git;a=blob_plain;f=repodiff.py;hb=9f23ddc1681ef1b9cc74a97fa90f5c76f3d3c00f

Comment 2 Jesse Keating 2008-05-29 14:54:02 UTC
As stated on list, forcing LANG=C is probably a bad thing here.

Comment 5 Tim Lauridsen 2008-06-03 10:50:58 UTC
Jesse, just let me know, if the latest version work for you.

Comment 6 Jesse Keating 2008-06-03 17:46:55 UTC
Yes, the revision in comment #4 works.

Comment 7 Tim Lauridsen 2008-06-03 19:07:54 UTC
thanks, i will put out a new yum-utils version soon including this one


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