Closed Russtopia closed 5 years ago
Intended to address issue https://github.com/TrueFurby/go-callvis/issues/36
Thank you for your contribution. Please fix the PR as mentioned in comments.
@Russtopia It would be really great if you could also update few examples that show command usage with parameters, because they don't currently show something that would work with latest version.
@Russtopia It would be really great if you could also update few examples that show command usage with parameters, because they don't currently show something that would work with latest version.
Sure, I should have done that with the original patch.
@Russtopia It would be really great if you could also update few examples that show command usage with parameters, because they don't currently show something that would work with latest version.
Newest PR updates examples, but I had to guess at the last example -- I'm unfamiliar with 'exo-open' but from the context it appears to take the SVG output and convert to JPG?
New version pushed. Implements the changes suggested:
output format no longer limited to png/svg; outputFormat passed to 'dot' -T arg allowing any supported format
server mode is activated by omitting -file arg, seems more sensible
Tested server mode with and without -skipbrowser, specifying -format jpg, svg etc. and with no -format specified (defaults to svg)
Tested -file mode with -format [png, svg, gif, jpg] OK - other invalid values cause dot to return error 1 (could be improved?) - go-callvis will exit with status 1 in this case
Addition of -output, -svg, -png options to run go-callvis in a non-server mode to just output image files. Useful for scripting eg., makefile targets to generate graphvis for modules non-interactively.