Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 167260

Summary: app-admin/ulogd-1.23-r1 does not respect USE=-postgres if libpq is installed
Product: Gentoo Linux Reporter: Natanael Copa <natanael.copa>
Component: New packagesAssignee: Jeremy Olexa (darkside) (RETIRED) <darkside>
Status: RESOLVED FIXED    
Severity: normal CC: antarus, maintainer-needed
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: Rework of configure.in patch from Bug 144154
Difference file for the ebuild to apply configure.in patch

Description Natanael Copa 2007-02-16 19:57:19 UTC
Calculating dependencies... done!
[ebuild   R   ] app-admin/ulogd-1.23-r1  USE="-mysql -postgres" 0 kB 
[snip...]
checking for MySQL files... configure: warning: MySQL backend not used
checking for PGSQL files... found pgsql in /usr

This is very similar to:
https://bugs.gentoo.org/show_bug.cgi?id=144154
Comment 1 Jakub Moc (RETIRED) gentoo-dev 2007-02-16 20:01:51 UTC
+AC_ARG_WITH(pgsql, AS_HELP_STRING([--with-pgsql=<directory>],[pgsql installed in <directory>]),[with_mysql=$withval],[with_mysql=no])

Eh, the patch has a bunch of cut'n'paste typos (s/mysql/pgsql/g)

:)
Comment 2 Jakub Moc (RETIRED) gentoo-dev 2007-11-17 21:33:29 UTC
antarus, your commit for Bug 144154 :)
Comment 3 Andrey Bondarenko 2008-01-25 19:49:25 UTC
Created attachment 141781 [details, diff]
Rework of configure.in patch from Bug 144154
Comment 4 Andrey Bondarenko 2008-01-25 20:00:53 UTC
Created attachment 141782 [details, diff]
Difference file for the ebuild to apply configure.in patch

Basically this patch was also taken from Bug 144154. I've added use flags for sqlite3 support and logging of ip addresses as strings, mostly for postgres. It works for me and I tried to test it but I don't know much about autoconf and writing ebuilds, so it needs reviewing.
Comment 5 Mark Loeser (RETIRED) gentoo-dev 2008-04-06 01:04:47 UTC
treecleaner: want to vote on this?
Comment 6 Samuli Suominen (RETIRED) gentoo-dev 2008-05-15 08:58:53 UTC
Voting.. It seems fixable and ulogd seems to be used software..

-1
Comment 7 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2008-07-17 22:23:30 UTC
Looks good to me. This can be fixed and not be removed. I'll take care of this soon.

(assigning to myself to not forgot)
Comment 8 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2008-08-09 03:46:51 UTC
done. thanks for the patch.