Summary: | www/dillo2 includes PORTSDIR/Mk file directly | ||
---|---|---|---|
Product: | Ports & Packages | Reporter: | Chris Rees <utisoft> |
Component: | Individual Port(s) | Assignee: | Brendan Fabeny <bf> |
Status: | Closed FIXED | ||
Severity: | Affects Only Me | CC: | des |
Priority: | Normal | ||
Version: | Latest | ||
Hardware: | Any | ||
OS: | Any |
Description
Chris Rees
2011-09-25 17:00:21 UTC
Responsible Changed From-To: freebsd-ports-bugs->bf Over to maintainer (via the GNATS Auto Assign Tool) Chris Rees <utisoft@hotmail.com> writes: >>Description: > des@ noticed that dillo2 is one of the many ports that directly includes bsd.openssl.mk rather than using USE_OPENSSL. > > While I was there, I also switched pre.mk for options.mk. I've While? AFAIK, USE_* cannot be included *after* pre.mk because there is no OPTIONSFILE processing in post.mk. Another example is ports/148637. But unlike USE_* flags OPTIONSFILE (the value) cannot depend on the *contents* of foo.mk before options.mk. The former can be fixed by using options.mk instead of pre.mk, the latter by adding yet another bunch of .include directives *before* options.mk and including OPTIONSFILE one more time inside pre.mk for a case when USE_* is defined conditionally *after* options.mk. > done some basic tests, and I don't think I broke any OPTIONS, but > perhaps you should run your own tests too. It passes mine, too. $ make check-sanity -dd |& sed -n '/\.include/!d; /options$/p; /openssl/p' .include /var/db/ports/dillo2/options .include /usr/ports/Mk/bsd.openssl.mk But if I do *not* change pre.mk to options.mk $ make check-sanity -dd |& sed -n '/\.include/!d; /options$/p; /openssl/p' .include /var/db/ports/dillo2/options $ make check-sanity USE_OPENSSL= -dd |& sed -n '/\.include/!d; /options$/p; /openssl/p' .include /var/db/ports/dillo2/options .include /usr/ports/Mk/bsd.openssl.mk > > - Use USE_OPENSSL and bsd.port.options.mk On 25 September 2011 18:25, h h <aakuusta@gmail.com> wrote: > Chris Rees <utisoft@hotmail.com> writes: > >>>Description: >> =A0 =A0 =A0 des@ noticed that dillo2 is one of the many ports that direc= tly includes bsd.openssl.mk rather than using USE_OPENSSL. >> >> =A0 =A0 =A0 While I was there, I also switched pre.mk for options.mk. I'= ve > > While? AFAIK, USE_* cannot be included *after* pre.mk because there is > no OPTIONSFILE processing in post.mk. Another example is ports/148637. > But unlike USE_* flags OPTIONSFILE (the value) cannot depend on the > *contents* of foo.mk before options.mk. Don't worry, I've examined the situation closely, and I assure you I've got this bit right :) I think you may have my sentence the wrong way round-- it was pre.mk, it's now options.mk. Have a look at my braindump and complaints at: http://wiki.freebsd.org/SimplifyingMkIncludes > The former can be fixed by using options.mk instead of pre.mk, the > latter by adding yet another bunch of .include directives *before* > options.mk and including OPTIONSFILE one more time inside pre.mk for a > case when USE_* is defined conditionally *after* options.mk. > >> done some basic tests, and I don't think I broke any OPTIONS, but >> perhaps you should run your own tests too. > > It passes mine, too. > > =A0$ make check-sanity -dd |& sed -n '/\.include/!d; /options$/p; /openss= l/p' > =A0.include /var/db/ports/dillo2/options > =A0.include /usr/ports/Mk/bsd.openssl.mk > > But if I do *not* change pre.mk to options.mk > > =A0$ make check-sanity -dd |& sed -n '/\.include/!d; /options$/p; /openss= l/p' > =A0.include /var/db/ports/dillo2/options > > =A0$ make check-sanity USE_OPENSSL=3D -dd |& sed -n '/\.include/!d; /opti= ons$/p; /openssl/p' > =A0.include /var/db/ports/dillo2/options > =A0.include /usr/ports/Mk/bsd.openssl.mk > Thanks hugely for checking for me though :) Chris On 9/25/11, h h <aakuusta@gmail.com> wrote: > The following reply was made to PR ports/161011; it has been noted by GNATS. > > From: h h <aakuusta@gmail.com> > To: Chris Rees <crees@FreeBSD.org> > Cc: bug-followup@FreeBSD.org, des@FreeBSD.org > Subject: Re: ports/161011: www/dillo2 includes PORTSDIR/Mk file directly > Date: Sun, 25 Sep 2011 17:25:28 +0000 > > Chris Rees <utisoft@hotmail.com> writes: > > >>Description: > > des@ noticed that dillo2 is one of the many ports that directly includes > bsd.openssl.mk rather than using USE_OPENSSL. > > > > While I was there, I also switched pre.mk for options.mk. I've > > While? AFAIK, USE_* cannot be included *after* pre.mk because there is > no OPTIONSFILE processing in post.mk. Another example is ports/148637. > But unlike USE_* flags OPTIONSFILE (the value) cannot depend on the > *contents* of foo.mk before options.mk. I'm a little confused by the above. The openssl handling must be done after the options are read, so that WITH_SSL is properly (un)defined (as properly, that is, as is possible with the current fragile options-handling in Ports), but before the pre-makefile section, where USE_OPENSSL is acted upon -- I don't see what these other details, or the other PR, have to do with this case. b. Hey, Would you be OK to commit/approve this? I'll get another committer to review it if it's a little contentious, but it concerns me that people copy/paste stuff like this! Chris -- This message has been scanned for viruses and dangerous content by MailScanner, and is believed to be clean. bf 2011-10-04 10:07:59 UTC FreeBSD ports repository Modified files: security/tor Makefile security/tor-devel Makefile www/dillo2 Makefile Log: use a more common form of openssl handling PR: 161011 Submitted by: crees Revision Changes Path 1.117 +4 -3 ports/security/tor-devel/Makefile 1.76 +4 -3 ports/security/tor/Makefile 1.54 +3 -3 ports/www/dillo2/Makefile _______________________________________________ cvs-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/cvs-all To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org" State Changed From-To: open->closed Committed. Thanks! |