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

Command Line Options - relative paths inconsistent #37

Open
sean-parent opened this issue Aug 6, 2019 · 2 comments
Open

Command Line Options - relative paths inconsistent #37

sean-parent opened this issue Aug 6, 2019 · 2 comments
Assignees

Comments

@sean-parent
Copy link
Contributor

Describe the bug
The path for -hyde-yaml-dir and -hyde-src-root is relative to the directory containing the file being processed. i.e.

hyde ./include/psutil/safe_cast.hpp -hyde-src-root=../../include -hyde-yaml-dir=../../docs/include

Here the path ../../include is relative to ./include/psutil/ and not the current working directory.

@fosterbrereton
Copy link
Contributor

Am I understanding correctly that you'd like the relative paths to be from the current working directory? That means you must navigate to a specific path before running the tool in order for it to function. I based the relative paths off the source file being processed because then the tool can be run regardless of the current working directory, which seemed the more flexible option.

@fosterbrereton fosterbrereton self-assigned this Aug 7, 2019
@sean-parent
Copy link
Contributor Author

Let's say I have:
include/library/file.hpp
include/library/sub/other.hpp

I would have to have two separate roots for each of those. That is a bit of a pain. Normally a path specified on the command line is relative to the current working directory, and a path in a file (like a config file, header file, html file, etc.) is relative to that file. Having the path be relative to some other file is weird and confusing.

fosterbrereton added a commit that referenced this issue Aug 30, 2019
* Fix for issue #37 - removing the cwd being set to the source file being processed.
* Adding `use-system-clang` flag (among 5 in total)
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

2 participants