Bug 189903 - [stage] Add staging support to www/squid33
Summary: [stage] Add staging support to www/squid33
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: John Marino
URL:
Keywords:
: 191680 (view as bug list)
Depends on:
Blocks: 180353 182177 186378 186379 188846 192093 192119
  Show dependency treegraph
 
Reported: 2014-05-18 01:40 UTC by joe
Modified: 2014-09-06 05:16 UTC (History)
3 users (show)

See Also:


Attachments
file.diff (2.82 KB, patch)
2014-05-18 01:40 UTC, joe
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description joe 2014-05-18 01:40:00 UTC
Adds staging support, added shebangfixes, and USES tar:bzip2 to www/squid33.

Fix: Patch attached with submission follows:
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2014-05-18 01:40:04 UTC
Responsible Changed
From-To: freebsd-ports-bugs->tmseck

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 John Marino freebsd_committer freebsd_triage 2014-08-11 13:24:31 UTC
*** Bug 191680 has been marked as a duplicate of this bug. ***
Comment 3 John Marino freebsd_committer freebsd_triage 2014-08-11 13:26:23 UTC
maintainer reset, moving this to patch-ready (untested, take at your own risk)
Comment 4 Martin Birgmeier 2014-08-13 17:18:45 UTC
Hmmm... maybe Joe could give an evaluation of whether he thinks the fix in the duplicate https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=191680 is better than this, so as to decide which one should be used in the end.

-- Martin
Comment 5 John Marino freebsd_committer freebsd_triage 2014-08-17 11:28:57 UTC
i was hoping somebody else would grab these. :(
Comment 6 John Marino freebsd_committer freebsd_triage 2014-08-17 11:33:35 UTC
(In reply to Martin Birgmeier from comment #4)
> Hmmm... maybe Joe could give an evaluation of whether he thinks the fix in
> the duplicate https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=191680 is
> better than this, so as to decide which one should be used in the end.

As tempting as a dynamic plist is, especially after going through a very obnoxious squid32 conversion with the same plist issues, I think we need to keep a "static" plist.
Comment 7 commit-hook freebsd_committer freebsd_triage 2014-08-17 12:47:46 UTC
A commit references this bug:

Author: marino
Date: Sun Aug 17 12:47:42 UTC 2014
New revision: 365181
URL: http://svnweb.freebsd.org/changeset/ports/365181

Log:
  Stage www/squid33

  This port needed a *lot* of love, and it needs a lot more.  It has a
  ton of options (Do we need so many options rather than enabled by
  default?) and I suspect many options aren't staged properly.  Also the
  way the plist is put together is crazy.  This needs a proper static
  pkg-plist with PLIST_SUB switches.  This port badly needs a full-time
  maintainer.  (www/squid32 and www/squid are just as bad)

  I also added patch from PR 192119 while I was here.

  PR:		189903
  Submitted by:	joe (thralling.com)
  Lotsa TLC:	marino

Changes:
  head/www/squid33/Makefile
  head/www/squid33/files/patch-compat_compat.h
Comment 8 John Marino freebsd_committer freebsd_triage 2014-08-17 12:49:43 UTC
These squid port makefile are terrible, especially where plist is involved.

There are likely LOTS of problems with the options.  It's stage-safe with default options though.

squid33 needs a full-time maintainer that knows what he/she is doing.