Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 167829 - app-antivirus/clamav-0.90: fails to compile with uclibc-0.9.28.1 (C_LINUX/NLS handling)
Summary: app-antivirus/clamav-0.90: fails to compile with uclibc-0.9.28.1 (C_LINUX/NLS...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Antivirus Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-02-21 03:21 UTC by solar (RETIRED)
Modified: 2007-03-08 17:00 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
lamav-0.90-uclibc-nls.patch (clamav-0.90-uclibc-nls.patch,285 bytes, patch)
2007-02-21 03:22 UTC, solar (RETIRED)
Details | Diff
clamav-0.90-nls.patch (clamav-0.90-nls.patch,284 bytes, patch)
2007-02-21 03:44 UTC, solar (RETIRED)
Details | Diff
clamav-0.90-r1.ebuild (clamav-0.90-r1.ebuild,3.95 KB, patch)
2007-02-21 03:46 UTC, solar (RETIRED)
Details | Diff
clamav-0.90-nls.patch (clamav-0.90-nls.patch,965 bytes, patch)
2007-03-03 20:09 UTC, solar (RETIRED)
Details | Diff
clamav-0.90-r1.ebuild (clamav-0.90-r1.ebuild,3.97 KB, text/plain)
2007-03-03 20:10 UTC, solar (RETIRED)
Details
updated nls patch (clamav-nls2.patch,2.17 KB, patch)
2007-03-08 16:09 UTC, Andrej Kacian (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description solar (RETIRED) gentoo-dev 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) gentoo-dev 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) gentoo-dev 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) gentoo-dev 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) gentoo-dev 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) gentoo-dev 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) gentoo-dev 2007-03-03 19:15:56 UTC
Sorry. let me go fine an autoconf macro for nls.
Comment 7 solar (RETIRED) gentoo-dev 2007-03-03 20:09:54 UTC
Created attachment 111961 [details, diff]
clamav-0.90-nls.patch

Updated nls patch
Comment 8 solar (RETIRED) gentoo-dev 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) gentoo-dev 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) gentoo-dev 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) gentoo-dev 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) gentoo-dev 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) gentoo-dev 2007-03-08 17:00:16 UTC
Patch applied in 0.90.1-r1. Thanks!