Opened 9 years ago

Closed 9 years ago

Last modified 5 years ago

#85 closed enhancement (fixed)

Show the user any errors that the graphviz programs may produce.

Reported by: pkropf Owned by: pkropf
Priority: normal Component: GraphvizPlugin
Severity: normal Keywords:
Cc: kilian.cavalotti@… Trac Release: 0.8


Attachments (0)

Change History (9)

comment:1 Changed 9 years ago by kilian

I began to think about this, and naturally came to use the new subprocess module (to easily get stdout/stderr/returncode), but it only exists for python 2.4. What's the policy about python version requirements? May we use subprocess, or a more compatible os.system() function?

comment:2 Changed 9 years ago by kilian

  • Cc kilian.cavalotti@… added

comment:3 Changed 9 years ago by pkropf@…

I don't have a problem with using 2.4+ specific modules.

comment:4 Changed 9 years ago by kilian

Well, I asked because of Trac requirements:

To install Trac, the following software packages must be installed: 
Python, version >= 2.3. 
Python 2.4 is not supported on Windows since there are no Subversion bindings available for it.

So, if we introduce 2.4-isms in the plugin code, it won't work on Windows and 2.3 installations. Perhaps it's worth to work on a 2.3 compatible version of the GraphViz plugin, what do you think?

comment:5 Changed 9 years ago by pkropf

I suppose that I spoke too soon. If we can avoid it, the Graphviz module shouldn't impose a newer version of Python than Trac.

It's been a while since I've done any cross platform process control. Any reason why we can't use one of the popen series?

comment:6 Changed 9 years ago by kilian

I don't think so, I just have to familiarize with those. :)
Feel free to go ahead on this if you want.

comment:7 Changed 9 years ago by pkropf

  • Resolution set to fixed
  • Status changed from new to closed

Release v0.4 now used popen2 to control and communicate with the graphviz program processes. As a result, the code checks the return code and any text that the graphviz program sends to stderr is displayed to the user.

comment:8 follow-up: Changed 5 years ago by k0s

(In [6543]) display single region on /query page if selected; refs #85 ; 2.5 hours

comment:9 in reply to: ↑ 8 Changed 5 years ago by k0s

  • Trac Release set to 0.8

Replying to k0s:

(In [6543]) display single region on /query page if selected; refs #85 ; 2.5 hours

sorry, wrongtrac

Add Comment

Modify Ticket

as closed .
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from pkropf. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.

E-mail address and user name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.