Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 773547 - www-apps/davical: 'ldap' USE flag has no function
Summary: www-apps/davical: 'ldap' USE flag has no function
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Till Schäfer
URL:
Whiteboard:
Keywords: PullRequest
Depends on:
Blocks:
 
Reported: 2021-03-01 10:29 UTC by Joonas Niilola
Modified: 2021-03-04 12:26 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Joonas Niilola gentoo-dev 2021-03-01 10:29:50 UTC
'ldap' does nothing more than control optional RDEPEND, I suggest moving it to pkg_postint. optfeature will be informational enough.
  https://projects.gentoo.org/qa/policy-guide/dependencies.html#pg0001
Comment 1 Till Schäfer 2021-03-03 09:59:52 UTC
THX for the advice. Pull request is on its way.
Comment 2 Larry the Git Cow gentoo-dev 2021-03-04 12:26:14 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=358e1409706d1aa3e078e27014a6efdc0a3d8bef

commit 358e1409706d1aa3e078e27014a6efdc0a3d8bef
Author:     Till Schäfer <till2.schaefer@uni-dortmund.de>
AuthorDate: 2021-03-03 09:55:27 +0000
Commit:     Joonas Niilola <juippis@gentoo.org>
CommitDate: 2021-03-04 12:26:08 +0000

    www-apps/davical: remove ldap dep and useflag
    
    remove ldap dependency, since it is a runtime dependency only.
    remove obsolete comment about already removed PHP compatibility restriction
    
    Closes: https://bugs.gentoo.org/773547
    Package-Manager: Portage-3.0.13, Repoman-3.0.2
    Signed-off-by: Till Schäfer <till2.schaefer@uni-dortmund.de>
    Closes: https://github.com/gentoo/gentoo/pull/19750
    Signed-off-by: Joonas Niilola <juippis@gentoo.org>

 www-apps/davical/davical-1.1.9.3-r1.ebuild | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)