[#TAV-267] Graphviz not found when running in windows. - myGrid

Details

Type: Bug
Status: Closed
Priority: Major
Resolution: Fixed
Affects Version/s: 1.5
Fix Version/s: 1.5
Component/s: None
Labels:
None

Description

The release built when doing assembly:directory doesn't work in Windows. It complains about being unable to find dot when rendering the diagram.

Issue Links

This issue related to:
TAV-271 Allow overriding MyGridConfiguration properties on command line

Activity

Ascending order - Click to sort in descending order

Hide

Permalink

Stian Soiland-Reyes added a comment - 2006-11-20 14:20

Solved by adding set ARGS=%ARGS% "-Dtaverna.dotlocation=%~dp0\bin\win32i386\dot.exe" using some 1337 windows batch scripting skillz. (%~dp0 is the path of the directory containing runme.bat)

Show

Stian Soiland-Reyes added a comment - 2006-11-20 14:20 Solved by adding set ARGS=%ARGS% "-Dtaverna.dotlocation=%~dp0\bin\win32i386\dot.exe" using some 1337 windows batch scripting skillz. (%~dp0 is the path of the directory containing runme.bat)

Hide

Permalink

Stian Soiland-Reyes added a comment - 2006-11-20 14:33

Note that taverna.dotlocation is read as a system properties instead of MyGridConfiguration.getProperty() by ScuflSVGDiagram and ScuflDiagramPanel.

This is to support setting taverna.dotlocation on the commandline, as now done in Windows (runme.bat) and in OS X (Taverna.app/blah/blah/Info.plist).

A better fix would be to let MyGridConfiguration.getProperties() to read overriden properties from system properties, like we do with raven.properties in Bootstrap. However, this causes a challenge as currently the MyGrid properties are also exported as system properties by loadMyGridProperties() for backward compatibility.

Possibly Graphviz should be an artifact with binaries included for known OS-es (also Linux), unpacked to some location (like .taverna/lib/graphviz), and run from there.

Show

Stian Soiland-Reyes added a comment - 2006-11-20 14:33 Note that taverna.dotlocation is read as a system properties instead of MyGridConfiguration.getProperty() by ScuflSVGDiagram and ScuflDiagramPanel. This is to support setting taverna.dotlocation on the commandline, as now done in Windows (runme.bat) and in OS X (Taverna.app/blah/blah/Info.plist). A better fix would be to let MyGridConfiguration.getProperties() to read overriden properties from system properties, like we do with raven.properties in Bootstrap. However, this causes a challenge as currently the MyGrid properties are also exported as system properties by loadMyGridProperties() for backward compatibility. Possibly Graphviz should be an artifact with binaries included for known OS-es (also Linux), unpacked to some location (like .taverna/lib/graphviz), and run from there.

People

Assignee:

Stian Soiland-Reyes

Reporter:

Stuart Owen

Vote (0)

Watch (0)

Dates

Created:

2006-11-20 11:10

Updated:

2008-03-20 15:21

Resolved:

2006-11-20 14:33