Bug 227127 - www/squid-devel: update to 4.0.24, add BDB option
Summary: www/squid-devel: update to 4.0.24, add BDB option
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Luca Pizzamiglio
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-03-30 22:25 UTC by ncrogers
Modified: 2018-04-13 09:34 UTC (History)
3 users (show)

See Also:
timp87: maintainer-feedback+


Attachments
update to 4.0.24, add BDB option (6.13 KB, patch)
2018-03-30 22:25 UTC, ncrogers
no flags Details | Diff
update to 4.0.24, add BDB option (disabled by default) (5.72 KB, patch)
2018-04-02 19:31 UTC, ncrogers
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description ncrogers 2018-03-30 22:25:10 UTC
Created attachment 191994 [details]
update to 4.0.24, add BDB option

This patch updates squid-devel to the latest version 4.0.24.

http://squid-web-proxy-cache.1019090.n4.nabble.com/squid-announce-Squid-4-0-24-beta-is-available-td4685043.html

I've also included an option to exclude Berkeley DB support, as my organization has concerns over its license. bdb was recently added to USES in this commit:
https://svnweb.freebsd.org/ports?view=revision&revision=438126

The "BDB" option seems to follow the general ports convention of how bdb support is included or not. Only the ext_session_acl and ext_time_quota_acl require Berkeley DB support, so it seems safe to optionally exclude it. To be consistent with the new behavior of this port I've enabled BDB by default. Perhaps there is a more appropriate way to configure this?

I've also tried including support for the ext_session acl helper, which also relies on BDB.

Thanks for your consideration.
Comment 1 ncrogers 2018-03-30 22:32:56 UTC
poudriere testport output

https://gist.github.com/courtland/3bd5e6f38f678c57688ab6a2b3f8feb5
Comment 2 Pavel Timofeev 2018-04-01 19:16:03 UTC
Comment on attachment 191994 [details]
update to 4.0.24, add BDB option

Great! Thanks a lot!
I would have no objectiona if BDB was turned off by default.
Comment 3 ncrogers 2018-04-02 19:30:31 UTC
BDB off by default seems reasonable to me since neither of the squid ports (3.x and 4.x) have required that before. I will update the diff.
Comment 4 ncrogers 2018-04-02 19:31:15 UTC
Created attachment 192116 [details]
update to 4.0.24, add BDB option (disabled by default)
Comment 5 Pavel Timofeev 2018-04-02 19:35:46 UTC
I wanted to mark the latest patch with '+', but bugzilla ignores my attempts
Comment 6 ncrogers 2018-04-02 20:47:17 UTC
I marked the first one as obsolete.
Comment 7 commit-hook freebsd_committer freebsd_triage 2018-04-13 09:22:28 UTC
A commit references this bug:

Author: pizzamig
Date: Fri Apr 13 09:21:52 UTC 2018
New revision: 467220
URL: https://svnweb.freebsd.org/changeset/ports/467220

Log:
  www/squid-devel: update to 4.0.24, add BDB option

  PR:		227127
  Submitted by:	ncrogers@gmail.com
  Approved by:	timp87@gmail.com (maintainer)

Changes:
  head/www/squid-devel/Makefile
  head/www/squid-devel/distinfo
  head/www/squid-devel/files/patch-src_ipc_Kid.cc
  head/www/squid-devel/pkg-plist
Comment 8 Luca Pizzamiglio freebsd_committer freebsd_triage 2018-04-13 09:34:25 UTC
Committed, thanks!