Skip to content
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

breathe on windows fails due to missing path to doxygen #271

Closed
psifertex opened this issue Jul 28, 2016 · 3 comments
Closed

breathe on windows fails due to missing path to doxygen #271

psifertex opened this issue Jul 28, 2016 · 3 comments

Comments

@psifertex
Copy link
Contributor

psifertex commented Jul 28, 2016

Breathe/sphinx/doxygen on Windows (at least under a Jenkins build environment) fails to run because https://github.com/michaeljones/breathe/blob/master/breathe/process.py#L87 does not find doxygen in the path without an additionalshell=True option to the subprocess call. The change that worked for me was:

        self.run_process(['doxygen', cfgfile], cwd=build_dir, shell=True)

Note that shell=True is considered a potential security risk, however in this case all arguments to the command-line are from local data and so it shouldn't be a concern.

@vitaut
Copy link
Contributor

vitaut commented Jul 28, 2016

Thanks for catching this. Could you submit a PR?

psifertex added a commit to psifertex/breathe that referenced this issue Sep 18, 2016
Windows requires the shell true option for the path to work.
@psifertex
Copy link
Contributor Author

Certainly -- just did so, sorry, I missed this reply originally.

@michaeljones
Copy link
Collaborator

Should be fixed in with #277

vermeeren pushed a commit that referenced this issue Jun 4, 2018
Windows requires the shell true option for the path to work.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants