Bug 1344015

Summary: Python support defective
Product: [Fedora] Fedora EPEL Reporter: Nima Saed-Samii <nima>
Component: rootAssignee: Mattias Ellert <mattias.ellert>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: epel7CC: mattias.ellert, steve.traylen
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
URL: https://sft.its.cern.ch/jira/browse/ROOT-8221
Whiteboard:
Fixed In Version: root-6.06.04-4.fc24 root-6.06.04-4.el7 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-06-29 12:55:46 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Nima Saed-Samii 2016-06-08 14:07:47 UTC
Description of problem:
ROOT Python bindings are defective

Version-Release number of selected component (if applicable):
root-python-6.06.04-1.el7.x86_64

Steps to Reproduce:
$ python
>>> import ROOT
>>> ROOT.kKey_Shift

Actual results:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib64/python2.7/site-packages/ROOT.py", line 410, in __getattr1
    return getattr( self, name )
  File "/usr/lib64/python2.7/site-packages/ROOT.py", line 449, in __getattr2
    raise AttributeError( name )
AttributeError: kKey_Shift

Expected results:
4128

Comment 1 Mattias Ellert 2016-06-09 11:32:36 UTC
Forwarded upstream:
https://sft.its.cern.ch/jira/browse/ROOT-8221

Comment 2 Mattias Ellert 2016-06-13 18:45:45 UTC
Upstream's reply to the bug report so far:

"KeySymbols.h is not in the PCH; I'd like to hear Bertrand's opinion whether it should be."

So this change in behaviour between root 5 and root 6 could be intentional, or not. So applying a patch to change the behaviour in Fedora might not be a good idea before upstream makes a decision.

The following works though:

$ python
Python 2.7.11 (default, Feb  5 2016, 01:53:41) 
[GCC 6.0.0 20160201 (Red Hat 6.0.0-0.9)] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import ROOT
>>> ROOT.gROOT.ProcessLine('#include "KeySymbols.h"')
0L
>>> ROOT.kKey_Shift
4128
>>>

Comment 3 Nima Saed-Samii 2016-06-14 14:35:23 UTC
We do have an in-house application that bases heavily on ROOT/PyROOT. The entire GUI is written using ROOT's primitives. So for us it is vital to have those symbols exported in PyROOT.

However, I do think that this is related to a change in the build-system. If I compile the current release 6.06.04 via the configure script it works.

Comment 4 Fedora Update System 2016-06-20 22:52:03 UTC
root-6.06.04-4.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-0bc674d384

Comment 5 Fedora Update System 2016-06-20 22:52:09 UTC
root-6.06.04-4.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-0bc674d384

Comment 6 Fedora Update System 2016-06-20 22:54:36 UTC
root-6.06.04-4.fc24 has been submitted as an update to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2016-f5dc66d209

Comment 7 Fedora Update System 2016-06-22 02:19:41 UTC
root-6.06.04-4.el7 has been pushed to the Fedora EPEL 7 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-0bc674d384

Comment 8 Fedora Update System 2016-06-22 02:27:06 UTC
root-6.06.04-4.fc24 has been pushed to the Fedora 24 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-f5dc66d209

Comment 9 Nima Saed-Samii 2016-06-22 20:18:34 UTC
The new package does fix the bug for me. Thank You.

Comment 10 Fedora Update System 2016-06-29 12:55:43 UTC
root-6.06.04-4.fc24 has been pushed to the Fedora 24 stable repository. If problems still persist, please make note of it in this bug report.

Comment 11 Fedora Update System 2016-07-10 02:22:09 UTC
root-6.06.04-4.el7 has been pushed to the Fedora EPEL 7 stable repository. If problems still persist, please make note of it in this bug report.