-
Notifications
You must be signed in to change notification settings - Fork 27
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
Add install/build options to requirements.txt #74
Comments
Yes, this is already supported, see here. |
Thanks. It seems it is not getting the defines in this requirements.txt file :
(but it works if I call it like Any clues why? |
You cant set those flags because they are usually set by the toolchain. Although there should be a way to add additional flags for some projects. However, for your example, you can add |
I see. But then why does it work if set those flags when I call cget?
I.e., this:
cget install -Detc
works in this case.
…On Thu, Oct 17, 2019 at 3:23 PM Paul Fultz II ***@***.***> wrote:
You cant set those flags because they are usually set by the toolchain.
Although there should be a way to add additional flags for some projects.
However, for your example, you can add
-DCMAKE_POSITION_INDEPENDENT_CODE=On to add the -fPIC flag.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#74?email_source=notifications&email_token=AAEYNNFRTYQPBJSCOGPHGWDQPBYNBA5CNFSM4JAF74F2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBQI35I#issuecomment-543198709>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAEYNNHRAHIDEH5QY67TOSDQPBYNBANCNFSM4JAF74FQ>
.
|
Variables like |
Is there a way to add install options to the requirements.txt file, such as:
package1 -DOption1=1
package2 -DOption2=2
?
Thanks
The text was updated successfully, but these errors were encountered: