Bug 357313 – app-misc/pax-utils-0.2.3 fails tests with Python 3

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Steffen Bergner 2011-03-04 08:50:44 UTC]

steps to reproduce:

linux01 ~ # eselect python list
Available Python interpreters:
  [1]   python2.7
  [2]   python3.1 *

works  (Python 2.7): 
--------------------
linux01 ~ # eselect python set 1
linux01 ~ # FEATURES="test ccache" emerge -1v app-misc/pax-utils-0.2.3

...
make[3]: Entering directory `/var/tmp/portage/app-misc/pax-utils-0.2.3/work/pax-utils-0.2.3/tests/source'
./dotest
PASS: src.typos
PASS: src.obsolete.funcs
PASS: src.bad.constants
PASS: src.obsolete.headers
PASS: src.use.xfuncs
...


fails (Python 3.x):
-------------------
linux01 ~ # eselect python set 2
linux01 ~ # FEATURES="test ccache" emerge -1v app-misc/pax-utils-0.2.3

...
make[3]: Entering directory `/var/tmp/portage/app-misc/pax-utils-0.2.3/work/pax-utils-0.2.3/tests/source'
./dotest
PASS: src.typos
PASS: src.obsolete.funcs
PASS: src.bad.constants
PASS: src.obsolete.headers
PASS: src.use.xfuncs
PASS: src.style
  File "/var/tmp/portage/app-misc/pax-utils-0.2.3/work/pax-utils-0.2.3/tests/source/space.py", line 18
    print line
             ^
SyntaxError: invalid syntax
  File "/var/tmp/portage/app-misc/pax-utils-0.2.3/work/pax-utils-0.2.3/tests/source/space.py", line 18
    print line
             ^
...

[Comment 1 Steffen Bergner 2011-03-04 08:52:09 UTC]

works  (Python 2.7): (src.style part)
--------------------
...
PASS: src.obsolete.headers
PASS: src.use.xfuncs
PASS: src.style
PASS: src.space
make[3]: Leaving directory `/var/tmp/portage/app-misc/pax-utils-0.2.3/work/pax-utils-0.2.3/tests/source'
make -C scanelf all
...

[Comment 2 Steffen Bergner 2011-03-04 08:53:45 UTC]

right line (it's friday!):
FEATURES="test ccache" emerge -1v =app-misc/pax-utils-0.2.3

[Comment 3 Arfrever Frehtes Taifersar Arahesis (RETIRED) 2011-03-04 16:09:10 UTC]

Created attachment 264729 [details, diff]
pax-utils-0.2.3-python3.patch

This patch adds support for Python 3 and fixes ResourceWarnings with Python >=3.2 and -Wd.

[Comment 4 Arfrever Frehtes Taifersar Arahesis (RETIRED) 2011-03-04 16:15:54 UTC]

Additionally app-misc/pax-utils ebuilds should have IUSE="... test" and DEPEND="... test? ( dev-lang/python )"

[Comment 5 SpanKY 2011-03-04 21:02:47 UTC]

yeah, no.  we're not depending on python.

[Comment 6 SpanKY 2011-03-04 21:04:43 UTC]

Comment on attachment 264729 [details, diff]
pax-utils-0.2.3-python3.patch

thanks, ive applied this to portage-utils/pax-utils

http://sources.gentoo.org/gentoo-projects/pax-utils/tests/source/space.py?r1=1.1&r2=1.2
http://sources.gentoo.org/gentoo-projects/portage-utils/tests/source/space.py?r1=1.1&r2=1.2

[Comment 7 SpanKY 2011-03-17 03:58:59 UTC]

pax-utils no longer has space.py at all.  ive converted it to sed/awk.