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

qbx.cpp should be located in the temp folder, or not reference the temp folder #43

Open
mkilgore opened this issue May 7, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@mkilgore
Copy link
Contributor

mkilgore commented May 7, 2022

qbx.cpp is located outside of ./internal/temp but is modified during the build. It also directly includes most of the files located inside of ./internal/temp, which locks down the location of temp. Overall it greatly complicates any attempt to move the temp folder somewhere else. I think we have two options for what to do with it:

  1. Copy it into ./internal/temp during the build. Change the paths, and give the compiler a -I flag back to ./internal/c/ so that qbx.cpp can see the common.h header without actually being in there.
  2. Modify the building such that qbx.cpp does not directly include anything from ./internal/temp. This is more work but theoretically possible by restructuring how the .txt files work and compile them separately on their own, and then link the whole thing together afterward.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Development

No branches or pull requests

1 participant