-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Unable to build 1.9.2 #8855
Comments
|
With a build on Gentoo we get (see issue doxygen#8855) ``` /var/tmp/portage/app-doc/doxygen-1.9.2/work/doxygen-1.9.2/src/doxygen.cpp: In function ‘const char* getArg(int, char**, int&)’: /var/tmp/portage/app-doc/doxygen-1.9.2/work/doxygen-1.9.2/src/doxygen.cpp:10271:53: warning: declaration of ‘optind’ shadows a globa10271 | static const char *getArg(int argc,char **argv,int &optind) | ~~~~~^~~~~~ In file included from /usr/include/bits/getopt_posix.h:27, from /usr/include/unistd.h:900, from /usr/include/bits/sigstksz.h:24, from /usr/include/signal.h:328, from /var/tmp/portage/app-doc/doxygen-1.9.2/work/doxygen-1.9.2/src/doxygen.cpp:121: /usr/include/bits/getopt_core.h:50:12: note: shadowed declaration is here 50 | extern int optind; | ^~~~~~ /var/tmp/portage/app-doc/doxygen-1.9.2/work/doxygen-1.9.2/src/doxygen.cpp: In function ‘void readConfiguration(int, char**)’: /var/tmp/portage/app-doc/doxygen-1.9.2/work/doxygen-1.9.2/src/doxygen.cpp:10434:7: warning: declaration of ‘optind’ shadows a global10434 | int optind=1; | ^~~~~~ In file included from /usr/include/bits/getopt_posix.h:27, from /usr/include/unistd.h:900, from /usr/include/bits/sigstksz.h:24, from /usr/include/signal.h:328, from /var/tmp/portage/app-doc/doxygen-1.9.2/work/doxygen-1.9.2/src/doxygen.cpp:121: /usr/include/bits/getopt_core.h:50:12: note: shadowed declaration is here 50 | extern int optind; | ^~~~~~ ``` by changing the name of the local variable / argument this shadowing behavior can be overcome.
Hm, I thought I had attached the build.log? Attaching it uploads it to some magic github site and gave me the link in that form. EDIT: Ah, I've now put it inline with a collapsible section. Happy to attach whatever you need.
... so I seem to have .so, but not .a.
Just to be clear: this wasn't something I'm doing in Gentoo, this was something a similar (Funtoo) distribution seems to have used to workaround the problem on their end. Was just sharing in case it's helpful.
Hm, I don't think so. I couldn't see one in /usr/lib{,64}.
No difference:
|
Pity.
Regarding the log file. |
All other libraries have been declared STATIC so better to declare the cmls and mscgen libraries also STATIC.
I've just pushed a proposed patch, pull request #8862 |
Code has been integrated in master on GitHub (please don't close the issue as this will be done at the moment of an official release). |
Trying patch now (sorry for delay), but I suspect it'll do the trick. ACK, won't close the bug. |
I suspect this was exposed in Gentoo defaulting to BUILD_SHARED_LIBS=ON (in our CMake tooling). Confirmed, your fix works great, thank you! |
This issue was previously marked 'fixed but not released', |
Describe the bug
Doxygen 1.9.2 doesn't seem to be able to build in Gentoo. I was preparing a new ebuild for packaging (from 1.9.1 which builds fine) when I noticed it.
Fails as:
Compiling Doxygen myself using the Gentoo ebuild. My failed build log is here:
build.log
I noticed that the same issue was reported in Funtoo and they seem to have worked around(?) the problem by linking in libxml statically:
Expected behavior
Successful build.
Version
The text was updated successfully, but these errors were encountered: