Created attachment 192150 [details] Patch adding support for the new ${name}_limits mechanics (databases/percona55-server) Thanks to the recent change to rc.subr(8) it is no longer necessary to define a resource limiting logic in service scripts. limits(1) is now run by default and its configuration is handled via the "${name}_limits" variable. Unfortunately, there is a collision of variable names. The service scripts are broken on FreeBSD version with that new change (12.0-CURRENT, 11-STABLE and soon 11.2-RELEASE). It has been reported and diagnosed here.[2] I'm submitting a patch fixing the service script for this port. A similar patch has already been accepted by a maintainer of another database port.[2] The patch preserves the old behavior on the systems without the modification. It would be great if you could review the patch and test it. :) [1]: https://svnweb.freebsd.org/base?view=revision&revision=r328331 [2]: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=226907
Created attachment 192229 [details] Patch adding support for the new ${name}_limits mechanics (databases/percona55-server, revision: 1) The previous patch would fail to trigger the new ${name}_limits semantics for __FreeBSD_version 1101514. The new patch fixes it (thanks to mmokhi@).
Created attachment 192415 [details] Patch adding support for the new ${name}_limits mechanics (databases/percona55-server, revision: 4) Due to an increasing number of problems caused by the new ${name}_limits behavior, the revision, which was backported from 12-CURRENT to 11-STABLE, has now been reverted.[1] As a result, the new behavior will not be present in any FreeBSD version < 12. This new patch adapts the port to those latest changes. [1]: https://svnweb.freebsd.org/base?view=revision&revision=r332363
A commit references this bug: Author: flo Date: Fri May 18 19:51:30 UTC 2018 New revision: 470324 URL: https://svnweb.freebsd.org/changeset/ports/470324 Log: Fix rc scripts after r328331 in head. PR: 227234, 227235 Submitted by: 0mp Changes: head/databases/percona55-server/Makefile head/databases/percona55-server/files/mysql-server.in head/databases/percona56-server/Makefile head/databases/percona56-server/files/mysql-server.in
Committed. Thanks.