Bug 800744

Summary:	subscription-manager depends on python-simplejson that is not needed basically
Product:	Red Hat Enterprise Linux 7	Reporter:	Satoru SATOH <ssato>
Component:	subscription-manager	Assignee:	Adrian Likins <alikins>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Entitlement Bugs <entitlement-bugs>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	7.0	CC:	alikins, bkearney, fsharath, jgalipea, jsefler
Target Milestone:	beta
Target Release:	7.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	1.10.5-1	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2014-06-13 12:01:39 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:	863175

Description Satoru SATOH 2012-03-07 06:39:24 UTC

Description of problem:
Same as rhbz#800732, subscription-manager does not need to depends on
python-simplejson as json (simplejson) module is in python itself in RHEL 6.


Version-Release number of selected component (if applicable):
Confirmed with subscription-manager-0.96.17-1.el6.x86_64


Additional info:
Like python-rhsm, I made an exeprimental (not tested) patch.

Comment 4 RHEL Program Management 2012-07-10 08:48:01 UTC

This request was not resolved in time for the current release.
Red Hat invites you to ask your support representative to
propose this request, if still desired, for consideration in
the next release of Red Hat Enterprise Linux.

Comment 5 RHEL Program Management 2012-07-11 01:48:31 UTC

This request was erroneously removed from consideration in Red Hat Enterprise Linux 6.4, which is currently under development.  This request will be evaluated for inclusion in Red Hat Enterprise Linux 6.4.

Comment 6 RHEL Program Management 2012-12-14 08:48:01 UTC

This request was not resolved in time for the current release.
Red Hat invites you to ask your support representative to
propose this request, if still desired, for consideration in
the next release of Red Hat Enterprise Linux.

Comment 7 Adrian Likins 2013-02-08 21:14:27 UTC

Yup, looks like that python-simplejson dep can go away for rhel7.

We'll need some code changes since we "import simplejson as json" 
in a few places (since simplejson exists on rhel5/rhel6, and json
only exists on rhel6/7).

Comment 8 Adrian Likins 2013-02-08 21:31:58 UTC

We should be fine without a change, since python-simplejson is still there on rhel7, but making it conditional makes the deps a little cleaner.

Comment 9 Bryan Kearney 2013-10-28 17:40:31 UTC

fixed in commit 5875bae79d93196bf7abcb411ba23873fd73f03f
Merge: 7c7a966 ee34aef
Author: Michael Stead <michael.stead>
Date:   Thu Oct 24 10:36:03 2013 -0700

    Merge pull request #796 from candlepin/alikins/1006748_ourjson
    
    1006748: replace simplejson with 'ourjson'


Available in 1.10.5-1

Comment 11 Sharath Dwaral 2013-11-05 23:23:40 UTC

Verified in 

https://bugzilla.redhat.com/show_bug.cgi?id=1006748#c3


Marking VERIFIED

Comment 12 Ludek Smid 2014-06-13 12:01:39 UTC

This request was resolved in Red Hat Enterprise Linux 7.0.

Contact your manager or support representative in case you have further questions about the request.