Modify

Opened 9 years ago

Closed 9 years ago

Last modified 8 years ago

#90 closed defect (fixed)

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

Reported by: pkropf Owned by: pkropf
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 9 years ago by pkropf

  • Cc gotoh@… added

comment:2 Changed 9 years ago by gotoh

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 9 years ago by pkropf

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 9 years ago by pkropf

  • Resolution set to fixed
  • Status changed from new to closed
  • Trac Release set to 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...

Add Comment

Modify Ticket

Action
as closed The owner will remain pkropf.
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.