Page MenuHomeFreeBSD

www/nginx: some style fix; Makefile cleanup
ClosedPublic

Authored by joneum on Nov 2 2017, 5:09 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jun 6, 6:53 PM
Unknown Object (File)
Apr 26 2024, 2:37 AM
Unknown Object (File)
Apr 26 2024, 2:37 AM
Unknown Object (File)
Apr 26 2024, 2:37 AM
Unknown Object (File)
Apr 25 2024, 8:30 PM
Unknown Object (File)
Apr 23 2024, 7:49 AM
Unknown Object (File)
Feb 13 2024, 9:06 PM
Unknown Object (File)
Dec 22 2023, 10:52 PM
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.