19039 2003-04-09 11:09 0000 sys-apps/man-1.5l-r2 fails to build with man.c:1107: `q' undeclared (first use in this function) 2003-04-28 05:26:58 0000 1 1 1 Unclassified Gentoo Linux Core system unspecified x86 Linux RESOLVED FIXED P2 normal --- 1 bevdv@yahoo.com azarah@gentoo.org seemant@gentoo.org bevdv@yahoo.com 2003-04-09 11:09:11 0000 Emerge man-1.5l-r2 appears to apply patches and configure normally, but compilation fails on man.c: colorgcc -c -Wall -Wstrict-prototypes -Wmissing-prototypes -march=athlon-mp -O3 -pipe -fforce-addr -fomit-frame-pointer -funroll-loops -falign-functions=4 -I. -DSTDC_HEADERS -DPOSIX -DDO_COMPRESS -DGREPSILENT="'q'" man.c man.c: In function `do_global_apropos': man.c:1107: `q' undeclared (first use in this function) Reproducible: Always Steps to Reproduce: No out of the ordinary setup to my knowledge. full output from emerge in attachment bevdv@yahoo.com 2003-04-09 11:11:58 0000 Created an attachment (id=10437) output from emerge seemant@gentoo.org 2003-04-10 04:05:17 0000 does this happen without colorgcc as well? bevdv@yahoo.com 2003-04-10 10:03:37 0000 Yes it happens even if colorgcc is unmerged. azarah@gentoo.org 2003-04-10 15:27:12 0000 I dont know what to do about that 'q' thing anymore. I have tried to fix it a dozen ways, and each fails somewhere else. What version of sed installed (emerge epm; epm -q sed), and attach output of 'emerge info'. bevdv@yahoo.com 2003-04-11 10:23:47 0000 please:/tmp# epm -q sed sed-4.0.6 please:/tmp# emerge info Portage 2.0.47-r10 (default-x86-1.4, gcc-3.2.2, glibc-2.3.1-r4) ================================================================= System uname: 2.4.20 i686 AMD Athlon(tm) MP 2000+ GENTOO_MIRRORS="http://gentoo.oregonstate.edu/ http://distro.ibiblio.org/pub/Linux/distributions/gentoo" CONFIG_PROTECT="/etc /var/qmail/control /usr/kde/2/share/config /usr/kde/3/share/config /usr/X11R6/lib/X11/xkb /usr/kde/3.1/share/config /usr/share/config" CONFIG_PROTECT_MASK="/etc/gconf /etc/env.d" PORTDIR="/usr/portage" DISTDIR="/usr/portage/distfiles" PKGDIR="/usr/portage/packages" PORTAGE_TMPDIR="/home/portage" PORTDIR_OVERLAY="/usr/local/portage" USE="x86 oss apm cups mikmod nls xml2 gdbm slang ruby libwww motif X 3dnow mmx directfb sse dga opengl xv fbcon kde qt qtmt arts tcltk aalib imlib ncurses readline sdl svga lcms gif jpeg png tiff gd avi mpeg quicktime esd gtk -gnome alsa ggi dvd xmms oggvorbis encode pam ssl crypt tcpd mozilla spell truetype xml pdflib plotutils tetex guile perl python libg++ atlas pic berkdb mysql postgres odbc samba gpm zlib java" COMPILER="gcc3" CHOST="i686-pc-linux-gnu" CFLAGS="-march=athlon-mp -O3 -pipe -fforce-addr -fomit-frame-pointer -funroll-loops -falign-functions=4" CXXFLAGS="-march=athlon-mp -O3 -pipe -fforce-addr -fomit-frame-pointer -funroll-loops -falign-functions=4" ACCEPT_KEYWORDS="x86" MAKEOPTS="-j4" AUTOCLEAN="yes" SYNC="rsync://rsync.gentoo.org/gentoo-portage" FEATURES="sandbox ccache" bevdv@yahoo.com 2003-04-11 10:38:30 0000 Juts FYI, I tried with a pared down CFLAGS and USE commented out completely, and it still fails in the same way. Also, If I replace GREPSILENT in man.c with the char constant 'q' by hand and it compiles fine. bevdv@yahoo.com 2003-04-14 14:11:01 0000 OK, so I don't know why the substitution is not working, but I gave up trying to be subtle. I change the patch: /usr/portage/sys-apps/man/files/man-1.5l-wrong-quotes.patch which has the offending assignment to just use the hard coded hex ascii vallue for 'q' --- man-1.5k/src/Makefile.in.orig 2002-12-26 06:43:39.000000000 +0200 +++ man-1.5k/src/Makefile.in 2002-12-26 06:48:18.000000000 +0200 @@ -16,7 +16,7 @@ pager = @pager@ -GS = -DGREPSILENT=\'@grepsilent@\' +GS = -DGREPSILENT=0x71 DEFS = @DEFS@ $(GS) CWARN = -Wall -Wstrict-prototypes -Wmissing-prototypes CWARNNP = -Wall and it builds fine now. I don't know what a better solution is. azarah@gentoo.org 2003-04-14 15:04:10 0000 Can you merge supersed, and then cp /bin/sed to /bin/sed.old for example, and then link ssed to sed, and see if it works unpatched then ? bevdv@yahoo.com 2003-04-14 15:20:41 0000 Well, man has always had the "wrong quotes" patch, so I am going to assume you mean the *original* patch, before I modified it, i.e. the one with +GS = -DGREPSILENT="'@grepsilent@'" Anyhow, I tried what you suggested and it still fails, with the same error. The actual gcc command issued looks fine, so I think maybe the problem is after automake/sed do their business. azarah@gentoo.org 2003-04-27 19:19:58 0000 Right. I do not want to hardcode 'q', and thus is why I have not simply changed it in the past. I went with your hexdecimal idea, and changed the patch to: ----------------------------------------------------------------- --- man-1.5k/src/Makefile.in.orig 2002-12-26 06:43:39.000000000 +0200 +++ man-1.5k/src/Makefile.in 2002-12-26 06:48:18.000000000 +0200 @@ -16,7 +16,7 @@ pager = @pager@ -GS = -DGREPSILENT=\'@grepsilent@\' +GS = -DGREPSILENT=$(shell echo "@grepsilent@" | hexdump -d -n 1 | gawk '{ printf("0x%x", $$2); exit 0 }') DEFS = @DEFS@ $(GS) CWARN = -Wall -Wstrict-prototypes -Wmissing-prototypes CWARNNP = -Wall ---------------------------------------------------------------- Which fixes it, and still will handle 'grepsilent' changing. jyrki@kruu.org 2003-04-28 05:26:58 0000 breaks building stage2, see bug #20095 10437 2003-04-09 11:11 0000 output from emerge out.txt text/plain cGxlYXNlOn4jIGVtZXJnZSBtYW4KQ2FsY3VsYXRpbmcgZGVwZW5kZW5jaWVzIC4uLmRvbmUhCj4+ PiBlbWVyZ2UgKDEgb2YgMSkgc3lzLWFwcHMvbWFuLTEuNWwtcjIgdG8gLwo+Pj4gbWQ1IDstKSBt YW4tMS41bC50YXIuZ3oKPj4+IFVucGFja2luZyBzb3VyY2UuLi4KPj4+IFVucGFja2luZyBtYW4t MS41bC50YXIuZ3ogdG8gL2hvbWUvcG9ydGFnZS9wb3J0YWdlL21hbi0xLjVsLXIyL3dvcmsKICog QXBwbHlpbmcgbWFuLTEuNWwtc2VhcmNoLW9yZGVyLnBhdGNoLi4uICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIFsgb2sgXQogICog QXBwbHlpbmcgbWFuLTEuNWwtZ3JvZmYtMS4xOC5wYXRjaC4uLiAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIFsgb2sgXQogICAq IEFwcGx5aW5nIG1hbi0xLjVsLXdyb25nLXF1b3Rlcy5wYXRjaC4uLiAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBbIG9rIF0KICAg ICogQXBwbHlpbmcgbWFuLTEuNWwtdXRpbF9jLXNlZ2ZhdWx0LnBhdGNoLi4uICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIFsgb2sgXQog ICAgICogQXBwbHlpbmcgbWFuLTEuNWwtcmVkaGF0LXBhdGNoZXMucGF0Y2guLi4gICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIFsgb2sg XQogICAgICAqIEFwcGx5aW5nIG1hbi0xLjVsLWxvY2FsZS1kZWJ1Zy1pbmZvLnBhdGNoLi4uICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBb IG9rIF0KICAgICAgPj4+IFNvdXJjZSB1bnBhY2tlZC4KICAgICAgbmxzCiAgICAgIGNoZWNraW5n IGZvciBnY2MKICAgICAgY2hlY2tpbmcgZm9yIEFOU0kgQyBoZWFkZXIgZmlsZXMKICAgICAgY2hl Y2tpbmcgZm9yIFBPU0lYLjEgaGVhZGVyIGZpbGVzCiAgICAgIGNoZWNraW5nIGZvciBCU0Qgc3Ry aW5nIGFuZCBtZW1vcnkgZnVuY3Rpb25zCiAgICAgIGNoZWNraW5nIHdoZXRoZXIgc3lzL3R5cGVz LmggZGVmaW5lcyB1aWRfdAogICAgICBjaGVja2luZyBmb3IgWGVuaXgKICAgICAgY2hlY2tpbmcg aG93IHRvIGdldCBhbGxvY2EKICAgICAgY2hlY2tpbmcgZm9yIG5scwogICAgICBjaGVja2luZyBm b3IgZ2V0b3B0LmgKICAgICAgQ2hlY2tpbmcgZWNobyB0byBzZWUgaG93IHRvIHN1cHByZXNzIG5l d2xpbmVzLi4uCiAgICAgIC4uLnVzaW5nIC1uLgogICAgICBUZXN0aW5nIHdoZXRoZXIgZ3JlcCBu ZWVkcyAtcyBvciAtcSAuLi4gLXEKCiAgICAgIEFzc3VtaW5nIHlvdSB3YW50IHRvIGJlIGFibGUg dG8gaGFuZGxlIHRoZSBucm9mZiBzb3VyY2UgZm9yIG1hbiBwYWdlcy4KCiAgICAgIEFzc3VtaW5n IHRoZSBmb2xsb3dpbmcgd2F5cyB0byBpbnZva2UgdGhlIHZhcmlvdXMgY29tbWFuZHMgYW5kIGZp bHRlcnM6CiAgICAgIHRyb2ZmIGNvbW1hbmQ6ICAvdXNyL2Jpbi9ncm9mZiAtVHBzIC1tYW5kb2MK ICAgICAgbnJvZmYgY29tbWFuZDogIC91c3IvYmluL25yb2ZmIC1UbGF0aW4xIC1jIC1tYW5kb2MK ICAgICAgam5yb2ZmIGNvbW1hbmQ6IC91c3IvYmluL2dyb2ZmIC1UbmlwcG9uIC1tYW5kb2NqCiAg ICAgIGVxbiBjb21tYW5kOiAgICAvdXNyL2Jpbi9nZXFuIC1UcHMKICAgICAgbmVxbiBjb21tYW5k OiAgIC91c3IvYmluL2dlcW4gLVRsYXRpbjEKICAgICAgam5lcW4gY29tbWFuZDogIC91c3IvYmlu L2dlcW4gLVRuaXBwb24KICAgICAgdGJsIGNvbW1hbmQ6ICAgIC91c3IvYmluL2d0YmwKICAgICAg Y29sIGNvbW1hbmQ6ICAgIC91c3IvYmluL2NvbAogICAgICB2Z3JpbmQgY29tbWFuZDogbWlzc2lu ZwogICAgICByZWZlciBjb21tYW5kOiAgL3Vzci9iaW4vcmVmZXIKICAgICAgZ3JhcCBjb21tYW5k OiAgIG1pc3NpbmcKICAgICAgcGljIGNvbW1hbmQ6ICAgIC91c3IvYmluL3BpYwogICAgICBwYWdl ciBjb21tYW5kOiAgL3Vzci9iaW4vbGVzcyAtaXMKICAgICAgY21wIGNvbW1hbmQ6ICAgIC91c3Iv YmluL2NtcCAtcwogICAgICBjYXQgY29tbWFuZDogICAgL3Vzci9iaW4vY2F0CiAgICAgIGF3ayBj b21tYW5kOiAgICAvdXNyL2Jpbi9hd2sKCiAgICAgIEl0IHNlZW1zIHRoYXQgbm8gY29sIGlzIG5l ZWRlZCwgc2luY2UgeW91IHVzZSBncm9mZi4KCiAgICAgIEFzc3VtaW5nIHRoYXQgYmluYXJpZXMg Z28gdG8gL3Vzci9iaW4sIHRoZSBjb25maWd1cmF0aW9uIGZpbGUKICAgICAgZ29lcyB0byAvZXRj LCBtYWtld2hhdGlzIHRvIC91c3Ivc2JpbiwgYW5kIHRoZSBtYW4gcGFnZXMKICAgICAgdG8gL3Vz ci9zaGFyZS9tYW4vbWFuKi4KICAgICAgLi9jb25maWd1cmU6IGxpbmUgMTogcHJlZml4OiBjb21t YW5kIG5vdCBmb3VuZAoKICAgICAgVXNpbmcgbG9jYWxlZGlyIC91c3Ivc2hhcmUvbG9jYWxlLyVM LyVOCgogICAgICBTZXR0aW5nIHRoZSBkZWZhdWx0IHNlY3Rpb24gc2VhcmNoIG9yZGVyIHRvIDEg OCAyIDMgNCA1IDYgNyA5IHRjbCBuIGwgcCBvLgoKICAgICAgQXNzdW1pbmcgdGhhdCBmb3JtYXR0 ZWQgcGFnZXMgc2hvdWxkIGJlIGNvbXByZXNzZWQuCiAgICAgIFVzaW5nIC9iaW4vYnppcDIgZm9y IGNvbXByZXNzaW9uLgoKICAgICAgQXNzdW1pbmcgdGhlIGZvbGxvd2luZyB3YXlzIHRvIGludm9r ZSB0aGUgdmFyaW91cyBkZWNvbXByZXNzb3JzOgogICAgICBndW56aXAgY29tbWFuZDogICAgICAg ICAvYmluL2d1bnppcCAtYwogICAgICBiemlwMiBjb21tYW5kOiAgICAgICAgICAvYmluL2J6aXAy IC1jIC1kCiAgICAgIHBjYXQgY29tbWFuZDogICAgICAgICAgIG1pc3NpbmcKICAgICAgemNhdCBj b21tYW5kOiAgICAgICAgICAgL2Jpbi96Y2F0CiAgICAgIGZjYXQgY29tbWFuZDogICAgICAgICAg IG1pc3NpbmcKICAgICAgdW55YWJiYSBjb21tYW5kOiAgICAgICAgbWlzc2luZwoKICAgICAgQ3Jl YXRlZCBNYWtlZmlsZSBhbmQgY29uZl9zY3JpcHQuIE5vdyBkbyAibWFrZSIgYW5kICJtYWtlIGlu c3RhbGwiLgoKICAgICAgY2Qgc3JjOyAuLi9jb25mX3NjcmlwdCBNYWtlZmlsZQogICAgICBDcmVh dGluZyBNYWtlZmlsZSBmcm9tIE1ha2VmaWxlLmluCiAgICAgIGNkIHNyYzsgbWFrZQogICAgICBt YWtlWzFdOiBFbnRlcmluZyBkaXJlY3RvcnkgYC9ob21lL3BvcnRhZ2UvcG9ydGFnZS9tYW4tMS41 bC1yMi93b3JrL21hbi0xLjVsL3NyYycKICAgICAgY29sb3JnY2MgLW1hcmNoPWF0aGxvbi1tcCAt TzMgLXBpcGUgLWZmb3JjZS1hZGRyIC1mb21pdC1mcmFtZS1wb2ludGVyIC1mdW5yb2xsLWxvb3Bz IC1mYWxpZ24tZnVuY3Rpb25zPTQgIC1zICBtYWtlbXNnLmMgICAtbyBtYWtlbXNnCiAgICAgIC4v bWFrZW1zZyAuLi9tc2dzL21lc3MuZW4gZ3JpcGVkZWZzLmggbXNnLmMKICAgICAgY29sb3JnY2Mg LWMgLVdhbGwgLVdzdHJpY3QtcHJvdG90eXBlcyAtV21pc3NpbmctcHJvdG90eXBlcyAtbWFyY2g9 YXRobG9uLW1wIC1PMyAtcGlwZSAtZmZvcmNlLWFkZHIgLWZvbWl0LWZyYW1lLXBvaW50ZXIgLWZ1 bnJvbGwtbG9vcHMgLWZhbGlnbi1mdW5jdGlvbnM9NCAtSS4gLURTVERDX0hFQURFUlMgLURQT1NJ WCAtRERPX0NPTVBSRVNTIC1ER1JFUFNJTEVOVD0iJ3EnIiBtYW4uYwogICAgICBtYW4uYzogSW4g ZnVuY3Rpb24gYGRvX2dsb2JhbF9hcHJvcG9zJzoKICAgICAgbWFuLmM6MTEwNzogYHEnIHVuZGVj bGFyZWQgKGZpcnN0IHVzZSBpbiB0aGlzIGZ1bmN0aW9uKQogICAgICBtYW4uYzoxMTA3OiAoRWFj aCB1bmRlY2xhcmVkIGlkZW50aWZpZXIgaXMgcmVwb3J0ZWQgb25seSBvbmNlCiAgICAgIG1hbi5j OjExMDc6IGZvciBlYWNoIGZ1bmN0aW9uIGl0IGFwcGVhcnMgaW4uKQogICAgICBtYWtlWzFdOiAq KiogW21hbi5vXSBFcnJvciAxCiAgICAgIG1ha2VbMV06IExlYXZpbmcgZGlyZWN0b3J5IGAvaG9t ZS9wb3J0YWdlL3BvcnRhZ2UvbWFuLTEuNWwtcjIvd29yay9tYW4tMS41bC9zcmMnCiAgICAgIG1h a2U6ICoqKiBbc291cmNlXSBFcnJvciAyCgogICAgICAhISEgRVJST1I6IHN5cy1hcHBzL21hbi0x LjVsLXIyIGZhaWxlZC4KICAgICAgISEhIEZ1bmN0aW9uIHNyY19jb21waWxlLCBMaW5lIDcxLCBF eGl0Y29kZSAyCiAgICAgICEhISAobm8gZXJyb3IgbWVzc2FnZSkKICAgICAgCg==