Bug 167829 – app-antivirus/clamav-0.90: fails to compile with uclibc-0.9.28.1 (C_LINUX/NLS handling)

• Product: Gentoo Linux
• Status: RESOLVED FIXED

[Description solar (RETIRED) 2007-02-21 03:21:29 UTC]

i386-gentoo-linux-uclibc-gcc -Os -pipe -fforce-addr -fomit-frame-pointer -Wstack-protector -Wformat=2 -o .libs/clamscan output.o getopt.o memory.o cfgparser.o misc.o options.o clamscan.o others.o manager.o treewalk.o  ../libclamav/.libs/libclamav.so -lpthread
output.o: In function `logg':
output.c:(.text+0x453): undefined reference to `libintl_gettext'
output.c:(.text+0x506): undefined reference to `libintl_gettext'
output.c:(.text+0x5ba): undefined reference to `libintl_gettext'
collect2: ld returned 1 exit status
make[2]: *** [clamscan] Error 1
make[2]: Leaving directory `/var/tmp/portage/app-antivirus/clamav-0.90/work/clamav-0.90/clamscan'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/var/tmp/portage/app-antivirus/clamav-0.90/work/clamav-0.90'
make: *** [all] Error 2

!!! ERROR: app-antivirus/clamav-0.90 failed.
Call stack:
  ebuild.sh, line 1614:   Called dyn_compile
  ebuild.sh, line 971:   Called qa_call 'src_compile'
  environment, line 3404:   Called src_compile
  clamav-0.90.ebuild, line 73:   Called die

!!! (no error message)
!!! If you need support, post the topmost build error, and the call stack if relevant.
!!! A complete build log is located at '/var/tmp/portage/app-antivirus/clamav-0.90/temp/build.log'.




[ebuild     U ] app-antivirus/clamav-0.90 [0.88.7-r2] USE="bzip2%* -crypt -curl% -gmp% -logrotate -mailwrapper -milter (-selinux) (-onaccess%)" 0 kB

[Comment 1 solar (RETIRED) 2007-02-21 03:22:53 UTC]

Created attachment 110783 [details, diff]
lamav-0.90-uclibc-nls.patch

quick work around for uclibc users w/o +nls support which is USE= masked.

[Comment 2 solar (RETIRED) 2007-02-21 03:44:38 UTC]

Created attachment 110784 [details, diff]
clamav-0.90-nls.patch

This patch is not limited to uclibc alone. pretty much the patch type used in grep and friends.

[Comment 3 solar (RETIRED) 2007-02-21 03:46:42 UTC]

Created attachment 110785 [details, diff]
clamav-0.90-r1.ebuild

clamav-0.90-r1.ebuild with (+|-) nls flag support.

[Comment 4 Andrej Kacian (RETIRED) 2007-02-21 03:52:51 UTC]

Thanks for the patch. Feel free to apply it and add the -r1 into the tree - I
just woke up and will be at work for next 8-10 hours, and I won't have chance
to do it before I get back.

I will send the patch upstream later, so please don't close this bug if you bump the ebuild.

[Comment 5 Andrej Kacian (RETIRED) 2007-02-25 12:51:20 UTC]

solar, clamav's configure doesn't have --{en,dis}able-nls option, so the patch won't help - or am I missing something here?

[Comment 6 solar (RETIRED) 2007-03-03 19:15:56 UTC]

Sorry. let me go fine an autoconf macro for nls.

[Comment 7 solar (RETIRED) 2007-03-03 20:09:54 UTC]

Created attachment 111961 [details, diff]
clamav-0.90-nls.patch

Updated nls patch

[Comment 8 solar (RETIRED) 2007-03-03 20:10:56 UTC]

Created attachment 111962 [details]
clamav-0.90-r1.ebuild

Updated ebuild. Calls autoreconf to for the new nls option.

[Comment 9 solar (RETIRED) 2007-03-03 20:14:16 UTC]

I see clamav-0.90.1 exists now. If you want me to attach new patches and ebuilds just yell.

[Comment 10 Andrej Kacian (RETIRED) 2007-03-03 21:31:00 UTC]

Reported upstream as bug 387. For now, let's apply the patch in 0.90 and 0.90.1 ebuilds.

[Comment 11 Natanael Copa 2007-03-08 15:25:56 UTC]

Comfirming the ebuild and patch works. I tried to just renamed it to clamav-0.90.1-r1 and it compiled just fine too.

[Comment 12 solar (RETIRED) 2007-03-08 16:01:36 UTC]

Sadly for whatever reason I can't view upstream bug 387 even after signing up for clamav bugzilla. 

[Comment 13 Andrej Kacian (RETIRED) 2007-03-08 16:09:23 UTC]

Created attachment 112575 [details, diff]
updated nls patch

Basically, upstream responded on the bug:

We'll need to review more code that uses gettext(). shared/getopt.c is using
gettext() too.

I uploaded another patch (attached).

No response since then.

[Comment 14 Andrej Kacian (RETIRED) 2007-03-08 17:00:16 UTC]

Patch applied in 0.90.1-r1. Thanks!