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

qt-cpp: Add the rest of the environment path to the path variable #32

Merged
merged 2 commits into from
Jan 14, 2025

Conversation

OrkunTokdemir
Copy link
Collaborator

qt-cpp: Add the rest of the environment path to the path variable
When Ninja doesn't exist in PATH and on a Unix-based system, Ninja might
be used from the Qt installation. In this case, the following output is
generated.

Output:
"PATH": "/home/orkun/Qt/Tools/Ninja"

The problem is that the above variable overrides the PATH variable
completely and it causes compilers not to be found. This commit adds the
rest of the environment path to the PATH variable for Unix-based systems
when Ninja is not found in PATH.

When Ninja doesn't exist in PATH and on a Unix-based system, Ninja might
be used from the Qt installation. In this case, the following output is
generated.

Output:
`"PATH": "/home/orkun/Qt/Tools/Ninja"`

The problem is that the above variable overrides the PATH variable
completely and it causes compilers not to be found. This commit adds the
rest of the environment path to the PATH variable for Unix-based systems
when Ninja is not found in PATH.
@OrkunTokdemir OrkunTokdemir force-pushed the orkun_ninja_path_14_01_2025 branch from 35f3ed9 to ba56def Compare January 14, 2025 14:29
@OrkunTokdemir OrkunTokdemir marked this pull request as ready for review January 14, 2025 14:31
@OrkunTokdemir OrkunTokdemir merged commit cf2ad6d into qt-labs:dev Jan 14, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants