Modify

Opened 4 years ago

Closed 21 months ago

#7621 closed defect (fixed)

graphviz errors ignored without logging

Reported by: Hal Owned by: rjollos
Priority: normal Component: MasterTicketsPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11

Description

ticket:7415 shows a case where it was difficult to debug, as the error is silently ignored.

In this case, the error comes from execution of an external program (dot) via the subprocess.Popen.communicate method

It would be helpful to log, at DEBUG level:

  • return code (at least if non-zero)
  • stderr output

My personal preference would be for all other functionality of master ticket to continue to work, but raising an exception is also reasonable.

Attachments (1)

patch.diff (3.1 KB) - added by Hal 4 years ago.
Patch to add logging of dot command errors

Download all attachments as: .zip

Change History (5)

Changed 4 years ago by Hal

Patch to add logging of dot command errors

comment:1 Changed 21 months ago by rjollos

  • Owner changed from coderanger to rjollos
  • Status changed from new to assigned

Patch can be simplified a bit, but otherwise looks good.

comment:2 Changed 21 months ago by rjollos

Example of error message:

08:44:06 PM Trac[graphviz] ERROR: dot dot failed with code 1: Format:
"png2" not recognized. Use one of: canon cmap cmapx cmapx_np dot eps fig
gd gd2 gif gv imap imap_np ismap jpe jpeg jpg pdf plain plain-ext png ps
ps2 svg svgz tk vml vmlz vrml wbmp x11 xdot xlib

comment:3 Changed 21 months ago by rjollos

(In [12946]) Refs #7621: Log errors from calling GraphViz.

comment:4 Changed 21 months ago by rjollos

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

Add Comment

Modify Ticket

Action
as closed The owner will remain rjollos.
The resolution will be deleted. Next status will be 'reopened'.
Author


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

 
Note: See TracTickets for help on using tickets.