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

generate individual c++ makefiles in target dir #47801

Merged
merged 1 commit into from
Nov 27, 2023

Conversation

jkarneges
Copy link
Member

@jkarneges jkarneges commented Nov 27, 2023

This generates the c++ makefiles in the target dir rather than under src/cpp, so no files are written under src/cpp during build, and so cargo clean will remove all intermediate build files including generated makefiles. As a further simplification, the makefiles are generated individually via multiple qmake invocations rather than by relying on a qmake subdirs project. Also fix make install and move the location of postbuild's pri file to avoid warnings when running make clean.

@jkarneges jkarneges force-pushed the jkarneges/makefiles-in-target branch from b20f9c3 to 56d508a Compare November 27, 2023 22:50
/src/runner/certs/*.crt
/src/runner/certs/*.key
/postbuild/conf.pri
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if we want all conf.pri files to be ignore we can add conf.pri once in .gitignore

@jkarneges jkarneges merged commit 8637d37 into main Nov 27, 2023
@jkarneges jkarneges deleted the jkarneges/makefiles-in-target branch November 27, 2023 23:44
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

Successfully merging this pull request may close these issues.

2 participants