Modify

Opened 12 years ago

Closed 11 years ago

Last modified 11 years ago

#90 closed defect (fixed)

Checking for executables on win32 needs to use the .exe extension.

Reported by: Peter Kropf Owned by: Peter Kropf
Priority: normal Component: GraphvizPlugin
Severity: normal Keywords:
Cc: gotoh@… Trac Release: 0.8

Description

When checking for the existance of the graphviz programs on win32 platforms, looking for just the program name isn't enough since windows uses .exe file name extension on executables.

Attachments (0)

Change History (4)

comment:1 Changed 12 years ago by Peter Kropf

Cc: gotoh@… added; anonymous removed

comment:2 Changed 12 years ago by Shun-ichi Goto

The problem of the file extension comes from the environment check which does the exact file existance check. How about "try execution then warn if failed" approach instead?

To execute command, extension is not required on windows. So you can make command line without extension like "/path/to/command/dot".

comment:3 Changed 12 years ago by Peter Kropf

This was fixed for release v0.3 but will remain open for a little while longer. I'm not certain that how the code was fixed is the right way to get the job done...

comment:4 Changed 11 years ago by Peter Kropf

Resolution: fixed
Status: newclosed
Trac Release: 0.8

Since there have been no additional comments that the fix doesn't work, I'm going to close this issue. Feel free to open it again if there's a problem...

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Peter Kropf.
The resolution will be deleted.

Add Comment


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

 
Note: See TracTickets for help on using tickets.