Bug 232335 - sysutils/syslog-ng317: Requires libbson even if mongodb is not enabled
Summary: sysutils/syslog-ng317: Requires libbson even if mongodb is not enabled
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Cy Schubert
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-10-16 22:59 UTC by david
Modified: 2018-10-17 03:56 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (cy)

Attachments
Patch to sysutils/syslog-ng317/Makefile to make libbson a requirement only if mongodb is enabled (1.27 KB, patch)
2018-10-16 22:59 UTC, david 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description david 2018-10-16 22:59:30 UTC 
Created attachment 198241 [details]
Patch to sysutils/syslog-ng317/Makefile to make libbson a requirement only if mongodb is enabled

[I am submitting this on behalf of a colleague who does not have -- and does not want -- a Bugzilla ID. -- dhw]

From his (internal) commit message:

    libbson seems to only be necessary if mongodb is enabled. (we do not enable
    it in our builds).
    
    Move libbson pre-requisite from being a standard LIB_DEPENDS to being
    a MONGO_LIB_DEPENDS.
Comment 1 commit-hook freebsd_committer freebsd_triage 2018-10-17 03:46:10 UTC 
A commit references this bug:

Author: cy
Date: Wed Oct 17 03:45:29 UTC 2018
New revision: 482269
URL: https://svnweb.freebsd.org/changeset/ports/482269

Log:
  libbson is only needed whwn the mongodb option is selected.
  Otherwise it is needlessly requisite. Fix this.

  PR:		232335
  Submitted by:	dhw@

Changes:
  head/sysutils/syslog-ng310/Makefile
  head/sysutils/syslog-ng311/Makefile
  head/sysutils/syslog-ng312/Makefile
  head/sysutils/syslog-ng313/Makefile
  head/sysutils/syslog-ng314/Makefile
  head/sysutils/syslog-ng315/Makefile
  head/sysutils/syslog-ng317/Makefile
  head/sysutils/syslog-ng39/Makefile
Comment 2 Cy Schubert freebsd_committer freebsd_triage 2018-10-17 03:56:39 UTC 
Committed. Thanks for the patch.