[Buildroot] [PATCH v3] lighttpd: use standard web root directory
Maxime Ripard
maxime.ripard at free-electrons.com
Fri Jul 20 12:51:39 UTC 2012
Hi Simon,
Le 20/07/2012 14:38, Simon Dawson a écrit :
> On 20 July 2012 13:23, Maxime Ripard <maxime.ripard at free-electrons.com> wrote:
>>> define LIGHTTPD_INSTALL_CONFIG
>>> - mkdir -p $(TARGET_DIR)/etc/lighttpd
>>> - mkdir -p $(TARGET_DIR)/etc/lighttpd/conf.d
>>> - mkdir -p $(TARGET_DIR)/srv/www/htdocs
>>> + $(INSTALL) -d -m 0755 $(TARGET_DIR)/etc/lighttpd/conf.d
>>> + $(INSTALL) -d -m 0755 $(TARGET_DIR)/var/www
>>
>> Why do you remove the mkdirs here ?
>
> They are redundant, since install is called with the -d option for the
> directories in question. (/srv/www/htdocs is no longer used.)
Ok, my question was more about why you replaced them by install -d
actually, but mostly out of curiosity :)
>>> define LIGHTTPD_UNINSTALL_TARGET_CMDS
>>> - rm -f $(TARGET_DIR)/usr/sbin/lighttpd
>>> - rm -f $(TARGET_DIR)/usr/sbin/lighttpd-angel
>>> - rm -rf $(TARGET_DIR)/usr/lib/lighttpd
>>> + $(RM) $(TARGET_DIR)/usr/sbin/lighttpd
>>> + $(RM) $(TARGET_DIR)/usr/sbin/lighttpd-angel
>>> + $(RM) -r $(TARGET_DIR)/usr/lib/lighttpd
>>> endef
>>
>> Why do you drop the -f here ? Depending on the user's shell and its
>> configuration, rm might require the user to confirm this removal, which
>> is not the case with rm -f.
>
> The $(RM) make variable includes the -f option.
Ok, didn't know that.
Then you can add my
Acked-By: Maxime Ripard <maxime.ripard at free-electrons.com>
To your next iteration with your signed-off-by in the patch.
--
Maxime Ripard, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
More information about the buildroot
mailing list