[2/2] apache: pass explicit sbindir

Message ID 20180302180756.14315-2-chrismcc@gmail.com
State New
Headers show
Series
  • [1/2] memcached: bump to version 1.5.6
Related show

Commit Message

Christopher McCrory March 2, 2018, 6:07 p.m.
The apache configure script help indicates that the sbindir is
PREFIX/sbin , but it actually uses PREFIX/bin.  Most people
probably expect httpd to be located in /usr/sbin/httpd

Signed-off-by: Christopher McCrory <chrismcc@gmail.com>
---
 package/apache/apache.mk | 3 +++
 1 file changed, 3 insertions(+)

Comments

Christopher McCrory March 2, 2018, 6:11 p.m. | #1
Hello...

Thanks Thomas, I copied and pasted your recomendations



On Fri, Mar 2, 2018 at 10:07 AM, Christopher McCrory <chrismcc@gmail.com>
wrote:

> The apache configure script help indicates that the sbindir is
> PREFIX/sbin , but it actually uses PREFIX/bin.  Most people
> probably expect httpd to be located in /usr/sbin/httpd
>
> Signed-off-by: Christopher McCrory <chrismcc@gmail.com>
>

<snip>

Patch

diff --git a/package/apache/apache.mk b/package/apache/apache.mk
index e1b1a209cb..4fd5896d59 100644
--- a/package/apache/apache.mk
+++ b/package/apache/apache.mk
@@ -28,7 +28,10 @@  else ifeq ($(BR2_PACKAGE_APACHE_MPM_WORKER),y)
 APACHE_MPM = worker
 endif
 
+# Explicitly pass sbindir, because apache configure script defines it
+# to PREFIX/bin, which doesn't make much sense
 APACHE_CONF_OPTS = \
+	--sbindir=/usr/sbin \
 	--sysconfdir=/etc/apache2 \
 	--with-apr=$(STAGING_DIR)/usr \
 	--with-apr-util=$(STAGING_DIR)/usr \