Bug 785869

Summary: R2spec-4.0.0 uses Python 2.5+ syntax on el5 (Python 2.4)
Product: [Fedora] Fedora EPEL Reporter: Shawn Lower <shawn>
Component: R2specAssignee: Pierre-YvesChibon <pingou>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: high Docs Contact:
Priority: unspecified    
Version: el5CC: pingou
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: R2spec-4.1.0-1.fc17 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-03-10 17:31: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:
Attachments:
Description Flags
replaces the os.getlogin() call with the use of the $LOGNAME environment variable, or pwd.getpwduid(os.getuid())[0] if that fails.
none
friendlier error message when R2spec is called without any arguments
none
change tries os.getlogin() but falls back to using either the $LOGNAME environment variable, or pwd.getpwduid(os.getuid())[0] none

Description Shawn Lower 2012-01-30 19:59:56 UTC
Description of problem:

R2spec 4.0.0 uses the 'with' statement, which was introduced in Python 2.5.

el5 comes with Python 2.4. This causes errors when running either R2spec or R2rpm.


Version-Release number of selected component (if applicable):

R2spec-4.0.0-1.el5

How reproducible:

Always fails on el5

Steps to Reproduce:
1. Install latest R2spec package on el5
2. run either R2spec/R2rpm
3. receive traceback below:
  
Actual results:

[ip-xxx ~/projec..ackages/mockBuilt/pracma (1070)] $ R2spec
Traceback (most recent call last):
  File "/usr/bin/R2spec", line 22, in ?
    from r2spec.r2spec_obj import R2spec, setup_parser
  File "/usr/lib/python2.4/site-packages/r2spec/r2spec_obj.py", line 111
    with open(configfile, 'w') as conf:
            ^
SyntaxError: invalid syntax
[ip-xxx ~/projec..ackages/mockBuilt/pracma (1071)] $ R2rpm
Traceback (most recent call last):
  File "/usr/bin/R2rpm", line 22, in ?
    from r2spec.r2spec_obj import R2rpm, setup_parser
  File "/usr/lib/python2.4/site-packages/r2spec/r2spec_obj.py", line 111
    with open(configfile, 'w') as conf:
            ^
SyntaxError: invalid syntax



Expected results:


Additional info:

R2spec-3.0.3-1.el5.noarch works fine. I guess R2spec-4.0.0 shouldn't be in the el5 branch of EPEL?

Comment 1 Pierre-YvesChibon 2012-02-05 19:28:58 UTC
Thanks for the report.

I think I can fix this, would you be willing to test from the git version once I have pushed a patch ?
I do not have an EL5 at the moment.

Comment 2 Shawn Lower 2012-02-06 15:39:29 UTC
Sure. I'd be happy to test it from git. Just let me know when you've pushed the patch and I'll try it out.

Comment 3 Pierre-YvesChibon 2012-02-07 09:51:34 UTC
I just pushed something that should fix the problem, however I see the tests are not passing at the moment, I believe I forgot to push some commits :-(

Comment 4 Shawn Lower 2012-02-09 20:50:41 UTC
Created attachment 560705 [details]
replaces the os.getlogin() call with the use of the $LOGNAME     environment variable, or pwd.getpwduid(os.getuid())[0] if that fails.

Comment 5 Shawn Lower 2012-02-09 20:51:25 UTC
Created attachment 560706 [details]
friendlier error message when R2spec is called without any arguments

Comment 6 Shawn Lower 2012-02-09 20:53:51 UTC
Great, that works! .. with caveats. I ran across a strange issue when running it under tmux (a terminal multiplexer similar to GNU Screen). It seems that when running under tmux, an entry isn't created in /var/run/utmp, so the call to os.getuser() raises an exception. I made a change (patch 0001) to just use the $LOGUSER variable instead (if it's set), and that seems to have fixed it.

I also added a friendlier error message for when R2spec/R2rpm are called without any arguments (p0002).


The error under tmux looks like this:


[ip-xxxxx ~/projects/R2spec/r2spec (1215)] $ ./R2spec 
Traceback (most recent call last):
  File "./R2spec", line 22, in ?
    from r2spec.r2spec_obj import R2spec, setup_parser
  File "/h/slower/projects/R2spec/r2spec/r2spec/r2spec_obj.py", line 128, in ?
    class Settings(object):
  File "/h/slower/projects/R2spec/r2spec/r2spec/r2spec_obj.py", line 131, in Settings
    packager = os.getlogin()
OSError: [Errno 2] No such file or directory

Comment 7 Pierre-YvesChibon 2012-02-09 21:18:41 UTC
(In reply to comment #6)
> Great, that works! ..

So it did work ? Cool :)

>with caveats. I ran across a strange issue when running
> it under tmux (a terminal multiplexer similar to GNU Screen). It seems that
> when running under tmux, an entry isn't created in /var/run/utmp, so the call
> to os.getuser() raises an exception. I made a change (patch 0001) to just use
> the $LOGUSER variable instead (if it's set), and that seems to have fixed it.

hm what about doing a try/except around the os.getuser() ?
As I expect os.getuser() to work most often.

Otherwise I like the patch :)

> I also added a friendlier error message for when R2spec/R2rpm are called
> without any arguments (p0002).

This is one of the things which I have on my laptop but haven't pushed yet, I'll see which of our solution is best, your might be simpler than what I tried.

Thanks for all this !

Comment 8 Pierre-YvesChibon 2012-02-10 22:37:29 UTC
Ok the latest version in git the unit-tests are finally all working (tested on Fedora and EL6).

The R2spec/R2rpm call without argument provides a nicer message.

If you could provide a new version of your first patch (on getuser()) rebased on the new code I could integrate it in the git :-)
(Maybe with the try/except mentioned above)

I would like to release 4.1.0 with all these changes soon (like this week-end or early next week), so feel free to poke at it and see if there are some release blocker remaining.

Thanks!

Comment 9 Shawn Lower 2012-02-11 01:21:28 UTC
Ok, I tried it on a couple RHEL5 machines. I was able to generate a specfile or RPM package outside of tmux with no patches, and in tmux with the attached patch.

On one very old machine, I had trouble with the following:

$ ./R2spec --sources ~/rpmbuild/SOURCES/RUnit_0.4.22.tar.gz                                                                                                                                            
Traceback (most recent call last):
  File "./R2spec", line 29, in ?
    R2spec().main(ARG)
  File "/h/slower/projects/R2spec/r2spec/r2spec/r2spec_obj.py", line 276, in main
    pack.extract_sources()
  File "/h/slower/projects/R2spec/r2spec/r2spec/rpackage.py", line 142, in extract_sources
    tar.extractall()
AttributeError: 'TarFile' object has no attribute 'extractall'


It seems 'extactall' was added in Python 2.5, but it was backported to 2.4 during a security update that Red Hat pushed out in RHBA-2010:0724. The filename is:

python-2.4.3-27.el5_5.2.x86_64.rpm

Again, that only affects systems that haven't had python updated since that errata was issued, back in September 2010. All the semi-recently updated systems worked fine. Other than that, looks good!

https://rhn.redhat.com/rhn/errata/details/Details.do?eid=10249

Thanks

Comment 10 Shawn Lower 2012-02-11 01:22:29 UTC
Created attachment 561027 [details]
change tries os.getlogin() but falls back to using either the $LOGNAME environment variable, or pwd.getpwduid(os.getuid())[0]

Comment 11 Pierre-YvesChibon 2012-02-11 10:21:36 UTC
Ok I have pushed your patch and started to prepare the release.

I added the following line to the spec file to circumvent the problem you mentioned:
Requires:       python >= python-2.4.3-27


Thanks for all your testing!

Comment 12 Fedora Update System 2012-02-14 20:52:31 UTC
R2spec-4.1.0-1.el5 has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/R2spec-4.1.0-1.el5

Comment 13 Fedora Update System 2012-02-14 20:52:40 UTC
R2spec-4.1.0-1.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/R2spec-4.1.0-1.fc16

Comment 14 Fedora Update System 2012-02-14 20:52:49 UTC
R2spec-4.1.0-1.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/R2spec-4.1.0-1.fc15

Comment 15 Fedora Update System 2012-02-14 20:52:58 UTC
R2spec-4.1.0-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/R2spec-4.1.0-1.el6

Comment 16 Fedora Update System 2012-02-14 20:53:06 UTC
R2spec-4.1.0-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/R2spec-4.1.0-1.fc17

Comment 17 Fedora Update System 2012-02-15 01:20:23 UTC
Package R2spec-4.1.0-1.el6:
* should fix your issue,
* was pushed to the Fedora EPEL 6 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=epel-testing R2spec-4.1.0-1.el6'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-EPEL-2012-0468/R2spec-4.1.0-1.el6
then log in and leave karma (feedback).

Comment 18 Fedora Update System 2012-03-10 17:31:54 UTC
R2spec-4.1.0-1.el6 has been pushed to the Fedora EPEL 6 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 19 Fedora Update System 2012-03-10 17:32:04 UTC
R2spec-4.1.0-1.el5 has been pushed to the Fedora EPEL 5 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 20 Fedora Update System 2012-03-10 21:52:07 UTC
R2spec-4.1.0-1.fc15 has been pushed to the Fedora 15 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 21 Fedora Update System 2012-03-10 21:55:56 UTC
R2spec-4.1.0-1.fc16 has been pushed to the Fedora 16 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 22 Fedora Update System 2012-03-11 17:03:40 UTC
R2spec-4.1.0-1.fc17 has been pushed to the Fedora 17 stable repository.  If problems still persist, please make note of it in this bug report.