app-admin/ulogd-1.23-r1 introduced "DEPEND ( ... postgres? virtual/postgresql-server )W --> virtual/postgresql-base should be enough here. - maybe introduce new USE=-minimal for DEPEND on full postgresql-server..? Reproducible: Always Steps to Reproduce: (trivial)
hmm, i'm fine with just postgresql-base. Have you tested that and does it work fine?
actually not, yet ;-) but it used to function some ebuilds before with just dev-db/libpq installed ;-) apart from that - as described above, you'll still need DEPEND=virtual/postgresql-server when your log target is not localhost..
(In reply to comment #2) > actually not, yet ;-) but it used to function some ebuilds before with just > dev-db/libpq installed ;-) > > apart from that - as described above, you'll still need > DEPEND=virtual/postgresql-server when your log target is not localhost.. > ugh. I'd rather not introduce another use flag on something that I don't maintain myself.
so I'll test if: DEPEND="net-firewall/iptables mysql? ( virtual/mysql ) postgres? ( virtual/postgresql-base )" is enough for a remote database. And we'll keep DEPEND="virtual/postgresql-server" to the user that knows best if [s]he want's a local database as target?
(In reply to comment #4) > so I'll test if: > > DEPEND="net-firewall/iptables > mysql? ( virtual/mysql ) > postgres? ( virtual/postgresql-base )" > > is enough for a remote database. > And we'll keep DEPEND="virtual/postgresql-server" to the user that knows best > if [s]he want's a local database as target? Yea, sounds good. Or we could just do: postgres? || ( virtual/postgresql-base virtual/postgresql-server ) That way it will prefer the -base but accept -server as well. I can only assume that if the user has a remote log target then they would already have -server installed. What do you think?
(In reply to comment #5) > Yea, sounds good. Or we could just do: > > postgres? || ( virtual/postgresql-base virtual/postgresql-server ) > dev-db/postgresql-server has RDEPEND=dev-db/postgresql-base set so *-server pulls *-base anyway > That way it will prefer the -base but accept -server as well. I can only assume > that if the user has a remote log target then they would already have -server > installed. What do you think? > I've tested for proper functioning with: - postgres? ( virtual/postgresql-server )" + postgres? ( virtual/postgresql-base )" -> WORKSFORME (=ulogd-1.23-r3)
> I've tested for proper functioning with: > - postgres? ( virtual/postgresql-server )" > + postgres? ( virtual/postgresql-base )" > > -> WORKSFORME (=ulogd-1.23-r3) Thanks for testing. Does not warrant a revbump though. Committed.
