Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 229935 - net-im/jabberd2-2.2.0-r1 unnecessary dependencies
Summary: net-im/jabberd2-2.2.0-r1 unnecessary dependencies
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Server (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: Thomas Anderson (tanderson) (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-06-28 20:02 UTC by Thomas Arnett
Modified: 2008-06-29 11:11 UTC (History)
0 users

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


Attachments
implementation of comment 1 option 1 (ebuild.diff,2.43 KB, patch)
2008-06-28 20:10 UTC, Thomas Arnett
Details | Diff
remove pipe from use.local.desc (use.diff,456 bytes, patch)
2008-06-28 20:14 UTC, Thomas Arnett
Details | Diff
re-enable backends (ebuild.diff,2.46 KB, patch)
2008-06-28 22:18 UTC, Thomas Arnett
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Arnett 2008-06-28 20:02:19 UTC
jabberd2-2.2.0-r1.ebuild requires choosing a storage backend from {berkdb, postgres, mysql, sqlite} and an auth/reg backend from {berkdb, mysql, postgres, pam, ldap}.  This leaves out {oracle, fs} for storage and {oracle, anon, pipe} for auth/reg.  "pipe" is a local USE flag.

I suggest either:
1. Enable the backends without external dependencies (fs, anon, pipe).  Remove the backend checks and the "pipe" flag.
2. Have USE flags for all backends.
Comment 1 Thomas Arnett 2008-06-28 20:10:55 UTC
Created attachment 158799 [details, diff]
implementation of comment 1 option 1

This patch also reorders DEPEND and removes obsolete references to the resolver component.
Comment 2 Thomas Arnett 2008-06-28 20:14:04 UTC
Created attachment 158801 [details, diff]
remove pipe from use.local.desc
Comment 3 Thomas Anderson (tanderson) (RETIRED) gentoo-dev 2008-06-28 20:44:31 UTC
Thomas, yes, I agree with you now about the pipe USE flag...I didn't read configure.ac closely enough I guess.

I'm favoring option 1 at the moment....Though I think I'd have to revision bump again though, which I'm trying to avoid at the moment.
Comment 4 Thomas Arnett 2008-06-28 21:48:44 UTC
I think most people using unstable ebuilds understand they'll be bumped more often.  Also, the sooner you do it, fewer people will have installed r1 or even 2.2.0.

Sometimes, ebuilds are updated without changing the revision, but I don't know the policy about that.
Comment 5 Thomas Arnett 2008-06-28 22:17:33 UTC
This will make the decision easier: r1 is missing a backslash in econf, disabling every backend except db.
Comment 6 Thomas Arnett 2008-06-28 22:18:57 UTC
Created attachment 158821 [details, diff]
re-enable backends
Comment 7 Thomas Anderson (tanderson) (RETIRED) gentoo-dev 2008-06-28 22:19:49 UTC
Ok, I'm going with option 1...but I certainly can't get to this at least until tomorrow afternoon/night
Comment 8 Thomas Arnett 2008-06-28 22:53:34 UTC
Comment on attachment 158821 [details, diff]
re-enable backends

Sorry, I should've opened a separate bug.  The problem in comment 0 comes with a clear error message and is easy to work around.  The problem in comment 5 doesn't and isn't.
Comment 9 Thomas Arnett 2008-06-28 23:03:45 UTC
Created bug 229957
Comment 10 Thomas Anderson (tanderson) (RETIRED) gentoo-dev 2008-06-29 11:11:03 UTC
Ok, Fixed in CVS. Thanks for the report, it's appreciated.