New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

-computeurl does not work with -pipe #56

Closed
alexbestul opened this Issue May 13, 2017 · 2 comments

Comments

Projects
None yet
2 participants
@alexbestul

alexbestul commented May 13, 2017

It appears that the -computeurl/-encodeurl arguments are not honored when using the -pipe argument.

I read through some of the source code, and it looks like -pipe triggers a code path that completely ignores the -computeurl option.

Can you make -pipe honor -computeurl? If you can't do that, can you at least update the help content to say that they don't work together?

@arnaudroques

This comment has been minimized.

Show comment
Hide comment
@arnaudroques

arnaudroques May 13, 2017

Contributor

Thanks for the report.
Sure, this is fixed in last beta https://www.dropbox.com/s/koo42q3d9gxw288/plantuml.jar?dl=0
Now -pipe and -computeurl can be used together.
This will be released in next official release.

Contributor

arnaudroques commented May 13, 2017

Thanks for the report.
Sure, this is fixed in last beta https://www.dropbox.com/s/koo42q3d9gxw288/plantuml.jar?dl=0
Now -pipe and -computeurl can be used together.
This will be released in next official release.

@alexbestul

This comment has been minimized.

Show comment
Hide comment
@alexbestul

alexbestul May 13, 2017

Wow, that was fast.

I tried your beta build, and confirmed that -pipe and -computeurl now work together as I expected.

Thanks, and keep up the good work!

alexbestul commented May 13, 2017

Wow, that was fast.

I tried your beta build, and confirmed that -pipe and -computeurl now work together as I expected.

Thanks, and keep up the good work!

@alexbestul alexbestul closed this May 13, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment