Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 167260 - app-admin/ulogd-1.23-r1 does not respect USE=-postgres if libpq is installed
Summary: app-admin/ulogd-1.23-r1 does not respect USE=-postgres if libpq is installed
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Jeremy Olexa (darkside) (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-02-16 19:57 UTC by Natanael Copa
Modified: 2008-08-09 03:46 UTC (History)
2 users (show)

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


Attachments
Rework of configure.in patch from Bug 144154 (ulogd-1.23-r1-configure.in.patch,5.36 KB, patch)
2008-01-25 19:49 UTC, Andrey Bondarenko
Details | Diff
Difference file for the ebuild to apply configure.in patch (ulogd-1.23-r1.ebuild.diff,1.81 KB, patch)
2008-01-25 20:00 UTC, Andrey Bondarenko
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.