FS#53378 - [nginx-mainline] Enable dynamic module compatibility

Attached to Project: Arch Linux
Opened by Kevin Dodd (Jesin) - Sunday, 19 March 2017, 23:15 GMT
Last edited by Bartłomiej Piotrowski (Barthalion) - Monday, 27 March 2017, 11:03 GMT
Task Type Feature Request
Category Packages: Extra
Status Closed
Assigned To Bartłomiej Piotrowski (Barthalion)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Starting in version 1.11.5, nginx has a new configure option "--with-compat", which enables the executable to load new dynamic modules without being rebuilt.

Maxim Dounin (the upstream developer who signed the last release) recommends that package-maintainers enable this option:
https://mailman.nginx.org/pipermail/nginx-devel/2016-October/008920.html

Please add "--with-compat" to the "_mainline_flags" array in the PKGBUILD.
This task depends upon

Closed by  Bartłomiej Piotrowski (Barthalion)
Monday, 27 March 2017, 11:03 GMT
Reason for closing:  Fixed
Comment by Massimiliano Torromeo (mtorromeo) - Monday, 20 March 2017, 08:04 GMT
+1
I would also change the build to compile every possible module as dynamic.
Maybe it's time for a split pkgbuild?
Comment by Bartłomiej Piotrowski (Barthalion) - Friday, 24 March 2017, 14:06 GMT
I will handle this after OpenSSl rebuild gets out of staging.

Massimiliano, I don't see many advantages of this. Space savings will be minimal, and current stance is that we rather avoid unnecessary splits.
Comment by Massimiliano Torromeo (mtorromeo) - Friday, 24 March 2017, 14:25 GMT
The split package would have only small advantages, I agree.
On the other hand, making the built-in modules dynamic allows the user to choose whether to enable them or not, so I still would like them to be dynamic if possibile.
Comment by Bartłomiej Piotrowski (Barthalion) - Monday, 27 March 2017, 11:03 GMT
Done for nginx-mainline. I will revisit splitting some modules (like geoip or mail proxy) after --with-compat becomes available in stable branch.

Loading...