Page MenuHomeFreeBSD

Mk/Uses/apache.mk: Port from Mk/bsd.apache.mk
AbandonedPublic

Authored by brnrd on Sep 10 2017, 7:55 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Mar 21, 10:50 AM
Unknown Object (File)
Mon, Mar 18, 7:44 PM
Unknown Object (File)
Mon, Mar 18, 7:44 PM
Unknown Object (File)
Mon, Mar 18, 7:44 PM
Unknown Object (File)
Mon, Mar 18, 7:44 PM
Unknown Object (File)
Mon, Mar 18, 7:44 PM
Unknown Object (File)
Mon, Mar 18, 7:44 PM
Unknown Object (File)
Mon, Mar 18, 7:44 PM
Subscribers

Details

Summary
Mk/Uses/apache.mk: Port from Mk/bsd.apache.mk

 - Move to new Uses framework

Differential_Revision: 
Reviewed_by:
Approved by:
Test Plan

This has been running on my system for a while now (bar the 22/24 -> 2.2/2.4 fix in bsd.ports.mk) during active porting of mod_h2-devel and mod_md-devel.

Just rebuilt all my Apache and PHP ports, these were all OK
See also D12398 for the PHP 7.1 rework

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 12868
Build 13130: arc lint + arc unit

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes
brnrd added reviewers: ohauer, pgollucci.

Please deprecate all the USE_APACHE*.

In D12308#255661, @mat wrote:

Please deprecate all the USE_APACHE*.

Isn't that what line 86 does, or am I missing something?

In D12308#255661, @mat wrote:

Please deprecate all the USE_APACHE*.

Isn't that what line 86 does, or am I missing something?

Well, all deprecation happen in Mk/bsd.sanitiy.mk, don't start putting some in other places.

Move warnings to Mk/bsd.sanity.mk

  • Warnings don't belong in Mk/Uses/*.mk
  • The APACHE_PORT directive has long been replaced by DEFAULT_VERSIONS
  • the www/apache-<mpm>-mpm ports have been deleted
Mk/Uses/apache.mk
371–375

This code will never be used, you never set _USES_POST+=apache. See Mk/Uses/libtool.mk for an example of how to write a USES with b.p.post.m support.

Mk/bsd.port.mk
895–896

why this change ?

1939–1940

Are there any ports that have a USE_APACHE* defined after bsd.port.pre.mk ?
Because you cannot set USES after bsd.port.pre.mk, so this bit should rather go away, and the ports be fixed.

brnrd marked an inline comment as done.

Fix _POST usage

  • Revert spurious change on USE_LDCONFIG
  • Find ports with USE_APACHE after bsd.port.pre.mk

Open item: Check list of ports with incorrect USE_APACHE behavior

brnrd added inline comments.
Mk/Uses/apache.mk
371–375

Looking at Mk/Uses/libtool.mk makes it clearer how that works

Mk/bsd.port.mk
895–896

Stale file in my tree? Fixed!

1939–1940
lang/php70
lang/php56
lang/php71
www/blogsum
www/otrs
security/shibboleth2-sp

Will investigate

brnrd edited the test plan for this revision. (Show Details)
brnrd added inline comments.
Mk/bsd.port.mk
1939–1940

The PHP ports are actually 3-in-1. The main port, the mod_php port and the extensions. So the USE_APACHE isn't out of order
I've updated PHP 7.1 port nonetheless as there were some improvements possible with Options framework, saved about 100 lines of Makefile.

www/blogsum: Misinterpreted, order OK
www/otrs: Pending mainainer approval
security/shibboleth-sp: Pending maintainer approval

This revision was automatically updated to reflect the committed changes.
brnrd marked an inline comment as done.
brnrd added inline comments.
Mk/bsd.port.mk
1939–1940

These are now fixed/commited/closed

brnrd marked an inline comment as done.

Further checks and simplification

  • common22 is an alias for server,2.2
  • Re-order argumnents-checks (run,2.2+ returned an error)
  • apache_ARGS are already split in bsd.port.mk
  • Remove duplicate incorrect USES usage check
  • Remove conditional slated for removal in 2012
  • Remove duplicate MIN/MAX version determination
  • Simplify MIN/MAX determination
  • WITH(OUT)_MODULES should've long been removed in favor of OPTIONS framework
  • Simplify STATIC handling
Mk/Uses/apache.mk
79

Left-over from testing with an additional httpd-trunk port, removed locally

Several ports directly include Mk/bsd.apache.mk

  • Mk/Uses/php.mk
  • lang/php56
  • lang/php70
  • lang/php71

Any remarks?
I've done a full exp-run on the ports tree and haven't found failures due to this change.

Add CHANGES and direct includes changes

Add missing Mk/bsd.port.mk

antoine requested changes to this revision.Nov 19 2017, 8:32 PM
antoine added a subscriber: antoine.

The php stuff may work but it's only by luck. Setting USE_APACHE=22+ or USE_APACHE_RUN=22+ before including ${PORTSDIR}/Mk/Uses/apache.mk doesn't have any effect (${PORTSDIR}/Mk/Uses/apache.mk doesn't do anything from those variables)

This revision now requires changes to proceed.Nov 19 2017, 8:32 PM

Also, has anyone from apache@ reviewed this ?

Mk/Uses/apache.mk
132

Would probably be a good idea to make it fatal. At one point, old glue should be removed.

Mk/bsd.port.mk
1402–1406

:S/

In D12308#274191, @mat wrote:

Also, has anyone from apache@ reviewed this ?

Nope. The apache@ team seems to have insufficient time to fix things.
NB: I'm part of apache@

In D12308#274191, @mat wrote:

Also, has anyone from apache@ reviewed this ?

Nope. The apache@ team seems to have insufficient time to fix things.
NB: I'm part of apache@

It would be nice if someone else from apache@ had a look.

brnrd marked an inline comment as done.

Rework patches

Mk/bsd.sanity.mk

  • APACHE_PORT is fatal

Mk/bsd.port.mk

  • Properly transform versions 2.2-24 is now 2.2-2.4

Mk/Uses/apache.mk

  • DEFAULT_APACHE_VER is now fatal
  • Version checks updated and fixed
  • Version selection improved
  • Properly default to APACHE_DEFAULT
  • Prep for version 2.5
In D12308#276609, @mat wrote:
In D12308#274191, @mat wrote:

Also, has anyone from apache@ reviewed this ?

Nope. The apache@ team seems to have insufficient time to fix things.
NB: I'm part of apache@

It would be nice if someone else from apache@ had a look.

Can't agree more, but I haven't had sufficient rapport on my requests for review sofar. See https://lists.freebsd.org/pipermail/freebsd-apache/2017-November/004408.html
I've had this in use for quite some time now and have

Mk/Uses/apache.mk
79

Re-added for addition of www/apache25-devel (see D13111)

Mk/bsd.port.mk
1402–1406

Got it. Improved for use-case of USE_APACHE= 22-24 using :C

brnrd marked an inline comment as done.

Add ports checkning APACHE_VERSION to diff

Mk/Uses/apache.mk
313

Must appear before the previous APACHE_VERSION check

413

This must be the .else clause _APACHE_PORT_IS_MODULE is no longer used

Update diff after bulk build

  • Define APACHE_VERSION before using it
  • Use .else not .elif for do-build target

The php stuff may work but it's only by luck. Setting USE_APACHE=22+ or USE_APACHE_RUN=22+ before including ${PORTSDIR}/Mk/Uses/apache.mk doesn't have any effect (${PORTSDIR}/Mk/Uses/apache.mk doesn't do anything from those variables)

Hi Antoine,

This has been long resolved, but the review is still blocked on this. The bulk run was successful, how do we proceed?

I started testing today and do a test on all ports (amd64/i386 FreeBSD 11.1) this weekend

This can be closed. Fixed in #464175
Forgot to add the DiffRev to the commit message :'(

This revision is now accepted and ready to land.Apr 8 2018, 12:39 PM
This revision now requires review to proceed.Apr 8 2018, 12:39 PM
This revision is now accepted and ready to land.Apr 8 2018, 12:39 PM
This revision now requires review to proceed.Apr 8 2018, 12:39 PM
This revision is now accepted and ready to land.Apr 8 2018, 12:40 PM
This revision now requires review to proceed.Apr 8 2018, 12:40 PM

Abandoning as I can't close it and no response