Bug 508984

Summary: Remove rhpl usage
Product: [Fedora] Fedora Reporter: Jeremy Katz <katzj>
Component: system-config-usersAssignee: Nils Philippsen <nphilipp>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: rawhideCC: nphilipp
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: 2009-09-03 12:29:21 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:
Bug Depends On:    
Bug Blocks: 508951    
Attachments:
Description Flags
rhpl.translate -> gettext
none
Switch to unicode() instead of rhpl.iconv
none
rhpl.executil -> subprocess none

Description Jeremy Katz 2009-06-30 18:20:28 UTC
system-config-users uses rhpl for a few things.  I've gone ahead and worked up patches to remove the dependency for all three cases

Comment 1 Jeremy Katz 2009-06-30 18:21:31 UTC
Created attachment 349994 [details]
rhpl.translate -> gettext

Comment 2 Jeremy Katz 2009-06-30 18:22:05 UTC
Created attachment 349995 [details]
Switch to unicode() instead of rhpl.iconv

This should be good -- rhpl.iconv was much more needed in the pre-python2 days when bfox wrote the code :)

Comment 3 Jeremy Katz 2009-06-30 18:23:06 UTC
Created attachment 349996 [details]
rhpl.executil -> subprocess

This is a direct conversion.  I think it'd be better, though, to switch to using os.unlink and shutil.rmtree instead of this method.

Comment 4 Nils Philippsen 2009-09-03 12:29:21 UTC
fixed in version 1.2.88