-
Notifications
You must be signed in to change notification settings - Fork 589
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
CFLAGS error #58
Comments
Do you have the CFLAGS environment variable set? Try clearing it.
|
I received some assistance from the Solus community(kfrc4267) that resolved the issue.
|
Today I encountered this problem in FreeBSD, when compiling from ports. This patch helped: P.S. GitHub does not allow to upload a file without an extension. It needs to be removed (.txt). |
This patch simply removes the test for invalid flags in CFLAGS. You should really
fix the root problem instead by removing that invalid flag. That's what the
previous comment does in a brute-force manner, but better would be to figure
out what is adding -D_FORTIFY_SOURCE=2 and fix it at the source (the BSD ports
build system, maybe?)
|
This flag I added myself and other programs in the ports it does not prevent to build. I really do not understand why these tests exist. |
It might not cause the build to fail for you in this particular case, but
putting a preprocessor flag into CFLAGS could cause bad builds in other
circumstances. It's best to use the right variable for the right flag to avoid
any possibility of such issues.
|
There are no problems with using preprocessor flags in CFLAGS (except ideological ones). But with these tests there is definitely a problem where preprocessor flags are set in CFLAGS. https://stackoverflow.com/questions/2754966/cflags-vs-cppflags#2756681 |
It's untrue to say categorically that there are no problems with the use of
preprocessor flags in CFLAGS. An autoconf test that uses AC_PREPROC_IFELSE is
not going to use any flags in CFLAGS but only CPPFLAGS ones. A macro definition
flag in CFLAGS that radically changes the environment (like _POSIX_C_SOURCE or
-D_FILE_OFFSET_BITS) will cause such a test to act differently than the system
will act during a compile, and could cause incorrect code to be used.
|
Well, it looks like I got excited. Indeed, there is a case where only CPPFLAGS is used. It's funny, I just updated the ports. This patch does not seem to be needed anymore (in FreeBSD and forks). Because it has long been in the ports. |
Hey guys,
I am not proficient with building from source, but unfortunately I have to since there is no package in Solus repos.
I am getting this:
I am not sure if this is an issue from my side or yours. Would appreciate any feedback.
Addendum: Running on Solus 1.2. C-ares version 1.11.0.
The text was updated successfully, but these errors were encountered: