Page MenuHomeFreeBSD

www/nginx: some style fix; Makefile cleanup
ClosedPublic

Authored by joneum on Nov 2 2017, 5:09 PM.
Tags
None
Referenced Files
F109028493: D12915.diff
Thu, Jan 30, 8:18 PM
Unknown Object (File)
Dec 26 2024, 10:59 AM
Unknown Object (File)
Nov 29 2024, 9:44 PM
Unknown Object (File)
Nov 13 2024, 2:24 AM
Unknown Object (File)
Oct 16 2024, 6:11 AM
Unknown Object (File)
Oct 5 2024, 11:50 PM
Unknown Object (File)
Sep 24 2024, 8:25 AM
Unknown Object (File)
Sep 13 2024, 12:00 AM
Subscribers

Details

Summary

Do the port clean-up:

-replace '!empty(PORT_OPTIONS:M...)' with '${PORT_OPTIONS:M...}';
-http_accesskey was removed in r439844, cleanup it.

PR: 222337
Reported by: m1cro <rum1cro@yandex.ru>

Approved by: xxx (mentor)
#Differential Revision: https://reviews.freebsd.org/Dxxxxx

Test Plan

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Can't you use

MDSO_CONFIGURE_ON+=[...]
MDSO_CONFIGURE_OFF+=[...]

^ that should obviously be DSO ^^ .. sorry, I see myself to bed.

mfg Tobias

This comment was removed by tcberner.

Can't you use

MDSO_CONFIGURE_ON+=[...]
MDSO_CONFIGURE_OFF+=[...]

mfg Tobias

www/nginx-devel has been updated with it. That should be the same if nginx on version 1.13. replaced. Therefore, I would like to have it like this. Thank you

Here is the link: https://svnweb.freebsd.org/ports?view=revision&revision=450471

The ports should stay the same. I therefore ask for release. Thank you.

Please double check the inline comment. If there is nothing there. Go ahead.

www/nginx/Makefile
1206 ↗(On Diff #34684)

does this apply to 1.12 too or was this removed by mistake?

This revision is now accepted and ready to land.Nov 2 2017, 9:33 PM
www/nginx/Makefile
1206 ↗(On Diff #34684)

http_accesskey was removed in r439844

This revision was automatically updated to reflect the committed changes.