Summary: | mail/milter-greylist: Missing run-dependencies | ||||||
---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | Harald Schmalzbauer <bugzilla.freebsd> | ||||
Component: | Individual Port(s) | Assignee: | Bartek Rutkowski <robak> | ||||
Status: | Closed FIXED | ||||||
Severity: | Affects Many People | CC: | m.tsatsenko, robak | ||||
Priority: | --- | Flags: | bugzilla.freebsd:
maintainer-feedback?
(m.tsatsenko) |
||||
Version: | Latest | ||||||
Hardware: | Any | ||||||
OS: | Any | ||||||
Attachments: |
|
Description
Harald Schmalzbauer
2014-09-28 12:56:36 UTC
Maintainer CC'd Hi, To submitter: is there any specific reason for using RUN_DEPS instead of LIB_DEPENDS in the patch? There's no reason for using RUN_DEPS instead of LIB_DEPENDS besides brain-o. I guess I was stuck with finding reasons for BUILD_DEPENDS usage, possibly related to new pkg, which has some kind of auto-lib-depends function. But haven't paid any more attention to it, just wanted to report that it's not working like it is for me. Created attachment 148048 [details]
patch
- Use LIB_DEPS for curl, spf and geoip
- Use ${PORTSDIR} instead of {.CURDIR} for bsd.milter.mk location
- Do not bump PORTREVISION as all of curl, spf and geoip are disabled by default
maintainer provided patch which looks decent on inspection, moving to patch-ready. A commit references this bug: Author: robak Date: Mon Oct 27 07:24:19 UTC 2014 New revision: 371532 URL: https://svnweb.freebsd.org/changeset/ports/371532 Log: mail/milter-greylist: minor fixes - Use LIB_DEPS for curl, spf and geoip - Use ${PORTSDIR} instead of {.CURDIR} for bsd.milter.mk location - Do not bump PORTREVISION as all of curl, spf and geoip are disabled by defaul PR: 193993 Submitted by: Mikhail Tsatsenko <m.tsatsenko@gmail.com> Approved by: mentors (implicit) Changes: head/mail/milter-greylist/Makefile Committed, thanks for your work, both of you! |