#85 closed enhancement (fixed)
Show the user any errors that the graphviz programs may produce.
| Reported by: | Peter Kropf | Owned by: | Peter Kropf |
|---|---|---|---|
| Priority: | normal | Component: | GraphvizPlugin |
| Severity: | normal | Keywords: | |
| Cc: | kilian.cavalotti@… | Trac Release: | 0.8 |
Description
Attachments (0)
Change History (9)
comment:1 Changed 20 years ago by
comment:2 Changed 20 years ago by
| Cc: | kilian.cavalotti@… added; anonymous removed |
|---|
comment:4 Changed 20 years ago by
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 20 years ago by
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 20 years ago by
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 20 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → 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: 9 Changed 16 years ago by
comment:9 Changed 16 years ago by
| Trac Release: | → 0.8 |
|---|



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?