Summary: | x11-apps/xconsole-1.0.6 : /.../xconsole.c:795:6: error: implicit declaration of function ‘openpty’ [-Werror=implicit-function-declaration] | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Thomas Haschka <raketenzwerg83> |
Component: | Current packages | Assignee: | Matt Turner <mattst88> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | x11, ysottre |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | 416069 | ||
Bug Blocks: | |||
Attachments: |
build.log
xconsole-1.0.6-r1.ebuild xconsole-1.0.6-update-configure.ac.patch xconsole-1.0.6-r1.ebuild #2 |
Description
Thomas Haschka
2015-06-06 17:14:17 UTC
Please attach the build log. Created attachment 404698 [details]
build.log
Something is going wrong in src_configure(). Compare his failed build: >checking for library containing openpty... -lutil >checking for openpty... yes >checking util.h usability... yes >checking util.h presence... yes >checking for util.h... yes >checking that generated files are newer than configure... done >configure: creating ./config.status With my successful build: >checking for library containing openpty... -lutil >checking for openpty... yes >checking util.h usability... no >checking util.h presence... no >checking for util.h... no >checking libutil.h usability... no >checking libutil.h presence... no >checking for libutil.h... no >checking pty.h usability... yes >checking pty.h presence... yes >checking for pty.h... yes >checking that generated files are newer than configure... done >configure: creating ./config.status Created attachment 404726 [details] xconsole-1.0.6-r1.ebuild Quick fix is to force it to use openpty from pty.h. @Thomas Haschka, can you please test this ebuild in a local overlay. See https://wiki.gentoo.org/wiki/Overlay/Local_overlay for instructions on setting one up. @Andrew Miller Your ebuild works for me! Thanks! (In reply to Thomas Haschka from comment #5) That's good news, but like I said it's a quick fix, not something we can put in the tree. A more robust fix is to add some logic to the configure script so it defaults to pty.h, and only tries to include util.h on mac/nbsd/obsd and libutil.h on fbsd. Created attachment 404756 [details, diff]
xconsole-1.0.6-update-configure.ac.patch
Created attachment 404758 [details]
xconsole-1.0.6-r1.ebuild #2
Please test this ebuild too.
(In reply to Andrew Miller from comment #8) > Created attachment 404758 [details] > xconsole-1.0.6-r1.ebuild #2 > > Please test this ebuild too. Works perfectly with the patch ! (In reply to Andrew Miller from comment #3) > Something is going wrong in src_configure(). Compare his failed build: I can confirm. My build looks like Andrew Miller's successful build. Thomas, are you still able to reproduce this? If so, could you check out the upstream sources and try those? $ git clone git://anongit.freedesktop.org/xorg/app/xconsole $ cd xconsole $ ./autogen.sh $ make If that succeeds, please run $ git clean -dxf $ git checkout HEAD~ $ ./autogen.sh $ make and report whether that works. I'm speculating that the top of the tree -- commit 649966c9f22e8 ("config: add AC_USE_SYSTEM_EXTENSIONS") at the time of this writing -- fixes the build for you. The attached patch does a superset of that, all of which I don't believe is necessary. I've made a 1.0.7 release and pushed it into Gentoo. |