Bug 448788

Summary: LANG="" repodiff can create unicode errors
Product: [Fedora] Fedora Reporter: Jesse Keating <jkeating>
Component: yum-utilsAssignee: Seth Vidal <skvidal>
Status: CLOSED UPSTREAM QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: low Docs Contact:
Priority: low    
Version: rawhideCC: dcantrell, james.antill, pmatilai, tim.lauridsen
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-06-03 19:07:54 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

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