Graphviz Issue Tracker
Mantis Bug Tracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0002354graphvizBuild/Installpublic2013-09-04 06:212013-10-15 22:58
Reporterphisama 
Assigned Toellson 
PrioritynormalSeveritynormalReproducibilityalways
StatusresolvedResolutionfixed 
PlatformOSopenSUSEOS Version12.3
Summary0002354: An alternative way to get the python version
DescriptionI use the attached patch for a IMO better way to get the python version.
TagsNo tags attached.
AUXILLARY-FILES
DATE-FIXED
FIX-COMMENTpatch applied
FORMER-ID
INPUT-FILE
OUTPUT-FILE
STATUS-COMMENT
VERSION
Attached Filespatch file icon graphviz-python_version.patch [^] (679 bytes) 2013-09-04 06:21 [Show Content]
patch file icon graphviz-python3_version.patch [^] (794 bytes) 2013-09-12 10:29 [Show Content]

- Relationships

-  Notes
User avatar (0000476)
ellson (administrator)
2013-09-04 09:45

Patch applied. Thank you.
User avatar (0000515)
phisama (reporter)
2013-09-12 07:17

One small change should be applied. Change 'print sys.version[3]' to
'print(sys.version[3])'. While Python < 3.X accepts both, Python 3.X only accepts the latter.
User avatar (0000516)
ellson (administrator)
2013-09-12 09:18

Sorry, I can't find "print sys.version[3]"
I'm guessing, but am not confident, that you mean to change "print '%d.%d'", in config/config_python.py ?

Can you attach a patch?
User avatar (0000517)
phisama (reporter)
2013-09-12 10:36

I've attached the patch. This will work for both 2.6 and up as well as 3.x. If you want to support python versions before 2.6 you would have to test first if python groks the code before deciding which to use. In that case one could leave the code as is for < 3.x and only use the function form for >= 3.0.

And just forget comment 515, I got things mixed up in my mind and didn't check before posting. Sorry for that.
User avatar (0000518)
ellson (administrator)
2013-09-12 11:10

Patch applied. Thank you.

- Issue History
Date Modified Username Field Change
2013-09-04 06:21 phisama New Issue
2013-09-04 06:21 phisama File Added: graphviz-python_version.patch
2013-09-04 09:45 ellson Note Added: 0000476
2013-09-04 09:47 ellson FIX-COMMENT => patch applied
2013-09-04 09:47 ellson Status new => resolved
2013-09-04 09:47 ellson Resolution open => fixed
2013-09-04 09:47 ellson Assigned To => ellson
2013-09-12 07:17 phisama Note Added: 0000515
2013-09-12 07:17 phisama Status resolved => feedback
2013-09-12 07:17 phisama Resolution fixed => reopened
2013-09-12 09:18 ellson Note Added: 0000516
2013-09-12 10:29 phisama File Added: graphviz-python3_version.patch
2013-09-12 10:36 phisama Note Added: 0000517
2013-09-12 10:36 phisama Status feedback => assigned
2013-09-12 11:10 ellson Note Added: 0000518
2013-10-15 22:58 ellson Status assigned => resolved
2013-10-15 22:58 ellson Resolution reopened => fixed


MantisBT 1.2.5[^]
Copyright © 2000 - 2011 MantisBT Group
Powered by Mantis Bugtracker