[Buildroot] [PATCH 0/3] Add ICAP server support

[Buildroot] [PATCH 0/3] Add ICAP server support

[Guillaume GARDET]

Hi,

This patch series enable ICAP support in squid and add c-icap and c-icap-modules packages.

Regards,

Guillaume

----------------------------------------------------------------
Guillaume GARDET (3):
  Enable c-icap server support in squid and define logdir, pidfile and  
      swapdir
  New package: c-icap     c-icap is an implementation of an ICAP server.
    Squid 3.x proxy server     supports it.
  New package: c-icap-modules     Additionals modules for c-icap server.

 package/Config.in                        |  2 ++
 package/c-icap-modules/Config.in         |  7 +++++++
 package/c-icap-modules/c-icap-modules.mk | 27 +++++++++++++++++++++++++++
 package/c-icap/Config.in                 | 10 ++++++++++
 package/c-icap/c-icap.mk                 | 24 ++++++++++++++++++++++++
 package/squid/squid.mk                   | 12 ++++++++++--
 6 files changed, 80 insertions(+), 2 deletions(-)
 create mode 100644 package/c-icap-modules/Config.in
 create mode 100644 package/c-icap-modules/c-icap-modules.mk
 create mode 100644 package/c-icap/Config.in
 create mode 100644 package/c-icap/c-icap.mk

-- 
1.8.4.5

[Buildroot] [PATCH 1/3] package/squid: enable ICAP server support in squid and define logdir, pidfile and swapdir

[Guillaume GARDET]

This patch adds ICAP server support to squid and also defines logdir, 
pidfile and swapdir.

Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>

---
 package/squid/squid.mk | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/package/squid/squid.mk b/package/squid/squid.mk
index 6dffae8..f68a402 100644
--- a/package/squid/squid.mk
+++ b/package/squid/squid.mk
@@ -26,7 +26,11 @@ SQUID_CONF_OPT =	--enable-async-io=8 --enable-linux-netfilter \
 			--enable-auth-negotiate="wrapper" \
 			--enable-auth-ntlm="fake" \
 			--disable-strict-error-checking \
-			--enable-external-acl-helpers="file_userip"
+			--enable-external-acl-helpers="file_userip" \
+			--with-logdir=/var/log/squid/ \
+			--with-pidfile=/var/run/squid.pid \
+			--with-swapdir=/var/cache/squid/ \
+			--enable-icap-client
 
 # On uClibc librt needs libpthread
 ifeq ($(BR2_TOOLCHAIN_HAS_THREADS)$(BR2_TOOLCHAIN_USES_UCLIBC),yy)
@@ -45,6 +49,10 @@ define SQUID_CLEANUP_TARGET
 		cachemgr.conf mime.conf.default squid.conf.default)
 endef
 
-SQUID_POST_INSTALL_TARGET_HOOKS += SQUID_CLEANUP_TARGET
+define SQUID_CREATE_MISSING_FOLDER
+	mkdir -p $(TARGET_DIR)/var/log/squid/
+endef
+
+SQUID_POST_INSTALL_TARGET_HOOKS += SQUID_CLEANUP_TARGET SQUID_CREATE_MISSING_FOLDER
 
 $(eval $(autotools-package))
-- 
1.8.4.5

[Buildroot] [PATCH 2/3] New package: c-icap c-icap is an implementation of an ICAP server. Squid 3.x proxy server supports it.

[Guillaume GARDET]

This patch adds c-icap server, a free ICAP server. It can be used with squid server.

Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>

---
 package/Config.in        |  1 +
 package/c-icap/Config.in | 10 ++++++++++
 package/c-icap/c-icap.mk | 24 ++++++++++++++++++++++++
 3 files changed, 35 insertions(+)
 create mode 100644 package/c-icap/Config.in
 create mode 100644 package/c-icap/c-icap.mk

diff --git a/package/Config.in b/package/Config.in
index 73ba86a..24a0a8e 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -950,6 +950,7 @@ menu "Networking applications"
 	source "package/boa/Config.in"
 	source "package/bridge-utils/Config.in"
 	source "package/bwm-ng/Config.in"
+	source "package/c-icap/Config.in"
 	source "package/can-utils/Config.in"
 	source "package/chrony/Config.in"
 	source "package/civetweb/Config.in"
diff --git a/package/c-icap/Config.in b/package/c-icap/Config.in
new file mode 100644
index 0000000..827bfbf
--- /dev/null
+++ b/package/c-icap/Config.in
@@ -0,0 +1,10 @@
+config BR2_PACKAGE_C_ICAP
+        bool "c-icap"
+        help
+          c-icap is an implementation of an ICAP server. It can be used with 
+          HTTP proxies that support the ICAP protocol to implement content 
+          adaptation and filtering services.
+          Most of the commercial HTTP proxies must support the ICAP protocol.
+          The open source Squid 3.x proxy server supports it.
+
+          http://c-icap.sourceforge.net/
\ No newline at end of file
diff --git a/package/c-icap/c-icap.mk b/package/c-icap/c-icap.mk
new file mode 100644
index 0000000..d35d958
--- /dev/null
+++ b/package/c-icap/c-icap.mk
@@ -0,0 +1,24 @@
+################################################################################
+#
+# c-icap
+#
+################################################################################
+
+C_ICAP_VERSION = 0.3.4
+C_ICAP_SOURCE = c_icap-$(C_ICAP_VERSION).tar.gz
+C_ICAP_SITE = http://downloads.sourceforge.net/c-icap/
+C_ICAP_INSTALL_STAGING = YES
+C_ICAP_INSTALL_TARGET = YES
+C_ICAP_CONF_OPT = --sysconfdir=/etc --with-process-mutexes=posix --enable-large-files
+C_ICAP_DEPENDENCIES = 
+
+define C_ICAP_INSTALL_TARGET_CONFIGS
+	mkdir -p $(TARGET_DIR)/var/run/c-icap/
+	mkdir -p $(TARGET_DIR)/tmp/c-icap/
+	mkdir -p $(STAGING_DIR)/var/run/c-icap/
+	mkdir -p $(STAGING_DIR)/tmp/c-icap/
+endef
+
+C_ICAP_POST_INSTALL_TARGET_HOOKS = C_ICAP_INSTALL_TARGET_CONFIGS
+
+$(eval $(autotools-package))
\ No newline at end of file
-- 
1.8.4.5

[Buildroot] [PATCH 3/3] New package: c-icap-modules Additionals modules for c-icap server.

[Guillaume GARDET]

This patch adds c-icap-modules, additionnal modules for c-icap.

Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>

---
 package/Config.in                        |  1 +
 package/c-icap-modules/Config.in         |  7 +++++++
 package/c-icap-modules/c-icap-modules.mk | 27 +++++++++++++++++++++++++++
 3 files changed, 35 insertions(+)
 create mode 100644 package/c-icap-modules/Config.in
 create mode 100644 package/c-icap-modules/c-icap-modules.mk

diff --git a/package/Config.in b/package/Config.in
index 24a0a8e..86a50ae 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -951,6 +951,7 @@ menu "Networking applications"
 	source "package/bridge-utils/Config.in"
 	source "package/bwm-ng/Config.in"
 	source "package/c-icap/Config.in"
+	source "package/c-icap-modules/Config.in"
 	source "package/can-utils/Config.in"
 	source "package/chrony/Config.in"
 	source "package/civetweb/Config.in"
diff --git a/package/c-icap-modules/Config.in b/package/c-icap-modules/Config.in
new file mode 100644
index 0000000..7946c17
--- /dev/null
+++ b/package/c-icap-modules/Config.in
@@ -0,0 +1,7 @@
+config BR2_PACKAGE_C_ICAP_MODULES
+        bool "c-icap-modules"
+	depends on BR2_PACKAGE_C_ICAP
+	help
+          Additionals modules for c-icap server.
+
+          http://c-icap.sourceforge.net/
\ No newline at end of file
diff --git a/package/c-icap-modules/c-icap-modules.mk b/package/c-icap-modules/c-icap-modules.mk
new file mode 100644
index 0000000..683dd4f
--- /dev/null
+++ b/package/c-icap-modules/c-icap-modules.mk
@@ -0,0 +1,27 @@
+################################################################################
+#
+# c-icap-modules
+#
+################################################################################
+
+C_ICAP_MODULES_VERSION = 0.3.2
+C_ICAP_MODULES_SOURCE = c_icap_modules-$(C_ICAP_MODULES_VERSION).tar.gz
+C_ICAP_MODULES_SITE = http://downloads.sourceforge.net/c-icap/
+C_ICAP_MODULES_INSTALL_STAGING = NO
+C_ICAP_MODULES_INSTALL_TARGET = YES
+C_ICAP_MODULES_CONF_OPT = --with-c-icap=$(STAGING_DIR)/usr/ --includedir=$(STAGING_DIR)/usr/include/  \
+		CPPFLAGS="-I $(STAGING_DIR)/usr/include/ -I $(STAGING_DIR)/usr/include/c_icap/" \
+		CFLAGS="$(CFLAGS) -I $(STAGING_DIR)/usr/include/ -I $(STAGING_DIR)/usr/include/c_icap/"
+C_ICAP_MODULES_DEPENDENCIES = c-icap
+
+define C_ICAP_MODULES_REMOVE_BROKEN_INCLUDE_PATH
+	$(SED) "s#-I/usr/include -I/usr/include/c_icap###" $(C_ICAP_MODULES_DIR)/Makefile
+	$(SED) "s#-I/usr/include -I/usr/include/c_icap###" $(C_ICAP_MODULES_DIR)/services/Makefile
+	$(SED) "s#-I/usr/include -I/usr/include/c_icap###" $(C_ICAP_MODULES_DIR)/services/url_check/Makefile
+	$(SED) "s#-I/usr/include -I/usr/include/c_icap###" $(C_ICAP_MODULES_DIR)/services/virus_scan/Makefile	
+endef
+
+
+C_ICAP_MODULES_POST_CONFIGURE_HOOKS = C_ICAP_MODULES_REMOVE_BROKEN_INCLUDE_PATH
+
+$(eval $(autotools-package))
\ No newline at end of file
-- 
1.8.4.5

[Buildroot] [PATCH 0/3] Add ICAP server support

[Guillaume GARDET - Oliséo]

Ping.

Guillaume


Le 18/09/2014 11:27, Guillaume GARDET a ?crit :
> Hi,
>
> This patch series enable ICAP support in squid and add c-icap and c-icap-modules packages.
>
> Regards,
>
> Guillaume
>
> ----------------------------------------------------------------
> Guillaume GARDET (3):
>    Enable c-icap server support in squid and define logdir, pidfile and
>        swapdir
>    New package: c-icap     c-icap is an implementation of an ICAP server.
>      Squid 3.x proxy server     supports it.
>    New package: c-icap-modules     Additionals modules for c-icap server.
>
>   package/Config.in                        |  2 ++
>   package/c-icap-modules/Config.in         |  7 +++++++
>   package/c-icap-modules/c-icap-modules.mk | 27 +++++++++++++++++++++++++++
>   package/c-icap/Config.in                 | 10 ++++++++++
>   package/c-icap/c-icap.mk                 | 24 ++++++++++++++++++++++++
>   package/squid/squid.mk                   | 12 ++++++++++--
>   6 files changed, 80 insertions(+), 2 deletions(-)
>   create mode 100644 package/c-icap-modules/Config.in
>   create mode 100644 package/c-icap-modules/c-icap-modules.mk
>   create mode 100644 package/c-icap/Config.in
>   create mode 100644 package/c-icap/c-icap.mk
>

[Buildroot] [PATCH 1/3] package/squid: enable ICAP server support in squid and define logdir, pidfile and swapdir

[Thomas Petazzoni]

Dear Guillaume GARDET,

On Thu, 18 Sep 2014 11:27:04 +0200, Guillaume GARDET wrote:
> This patch adds ICAP server support to squid and also defines logdir, 
> pidfile and swapdir.
> 
> Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>

Thanks for this contribution. A few comments below.

> 
> diff --git a/package/squid/squid.mk b/package/squid/squid.mk
> index 6dffae8..f68a402 100644
> --- a/package/squid/squid.mk
> +++ b/package/squid/squid.mk
> @@ -26,7 +26,11 @@ SQUID_CONF_OPT =	--enable-async-io=8 --enable-linux-netfilter \
>  			--enable-auth-negotiate="wrapper" \
>  			--enable-auth-ntlm="fake" \
>  			--disable-strict-error-checking \
> -			--enable-external-acl-helpers="file_userip"
> +			--enable-external-acl-helpers="file_userip" \
> +			--with-logdir=/var/log/squid/ \
> +			--with-pidfile=/var/run/squid.pid \
> +			--with-swapdir=/var/cache/squid/ \
> +			--enable-icap-client
>  
>  # On uClibc librt needs libpthread
>  ifeq ($(BR2_TOOLCHAIN_HAS_THREADS)$(BR2_TOOLCHAIN_USES_UCLIBC),yy)
> @@ -45,6 +49,10 @@ define SQUID_CLEANUP_TARGET
>  		cachemgr.conf mime.conf.default squid.conf.default)
>  endef
>  
> -SQUID_POST_INSTALL_TARGET_HOOKS += SQUID_CLEANUP_TARGET
> +define SQUID_CREATE_MISSING_FOLDER
> +	mkdir -p $(TARGET_DIR)/var/log/squid/
> +endef
> +
> +SQUID_POST_INSTALL_TARGET_HOOKS += SQUID_CLEANUP_TARGET SQUID_CREATE_MISSING_FOLDER
>  
>  $(eval $(autotools-package))

I believe the changes of adding the log directory, pidfile and swapdir
on one side, and adding the ICAP client option on the other side should
be two separate patches.

Also, how much does the ICAP client support adds to Squid size? I'm
trying to see if this should be optional instead of always enabled like
you're proposing.

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH 2/3] New package: c-icap c-icap is an implementation of an ICAP server. Squid 3.x proxy server supports it.

[Thomas Petazzoni]

Dear Guillaume GARDET,

The title of the commit should just be:

	c-icap: new package

On Thu, 18 Sep 2014 11:27:05 +0200, Guillaume GARDET wrote:

> diff --git a/package/c-icap/Config.in b/package/c-icap/Config.in
> new file mode 100644
> index 0000000..827bfbf
> --- /dev/null
> +++ b/package/c-icap/Config.in
> @@ -0,0 +1,10 @@
> +config BR2_PACKAGE_C_ICAP
> +        bool "c-icap"
> +        help
> +          c-icap is an implementation of an ICAP server. It can be used with 
> +          HTTP proxies that support the ICAP protocol to implement content 
> +          adaptation and filtering services.
> +          Most of the commercial HTTP proxies must support the ICAP protocol.
> +          The open source Squid 3.x proxy server supports it.
> +
> +          http://c-icap.sourceforge.net/
> \ No newline at end of file

Missing new line at the end of the file.

Also, did you check that this package indeed builds with a minimal
toolchain such as
http://autobuild.buildroot.org/toolchains/configs/br-arm-basic.config ?
It's also good to check if it builds with
http://autobuild.buildroot.org/toolchains/configs/bfin-linux-uclibc.config
and then
http://autobuild.buildroot.org/toolchains/configs/bfin-uclinux.config.



> diff --git a/package/c-icap/c-icap.mk b/package/c-icap/c-icap.mk
> new file mode 100644
> index 0000000..d35d958
> --- /dev/null
> +++ b/package/c-icap/c-icap.mk
> @@ -0,0 +1,24 @@
> +################################################################################
> +#
> +# c-icap
> +#
> +################################################################################
> +
> +C_ICAP_VERSION = 0.3.4
> +C_ICAP_SOURCE = c_icap-$(C_ICAP_VERSION).tar.gz
> +C_ICAP_SITE = http://downloads.sourceforge.net/c-icap/
> +C_ICAP_INSTALL_STAGING = YES

Is this package really installing some libraries used by other packages?

> +C_ICAP_INSTALL_TARGET = YES

Not needed, this is the default.

The C_ICAP_LICENSE and C_ICAP_LICENSE_FILES variables are missing.

> +C_ICAP_CONF_OPT = --sysconfdir=/etc --with-process-mutexes=posix --enable-large-files

--enable-large-files seems to indicate you at least need largefile
support. Or you should make it optional with something like

ifeq ($(BR2_LARGEFILE),y)
ICAP_CONF_OPTS += --enable-large-files
else
ICAP_CONF_OPTS += --disable-large-files
endif

> +C_ICAP_DEPENDENCIES = 

Not needed, it defaults to empty.

> +define C_ICAP_INSTALL_TARGET_CONFIGS
> +	mkdir -p $(TARGET_DIR)/var/run/c-icap/
> +	mkdir -p $(TARGET_DIR)/tmp/c-icap/
> +	mkdir -p $(STAGING_DIR)/var/run/c-icap/
> +	mkdir -p $(STAGING_DIR)/tmp/c-icap/

Creating these directories in $(STAGING_DIR) is not needed. Just
curious, why is /tmp/c-icap/ necessary?

> +endef
> +
> +C_ICAP_POST_INSTALL_TARGET_HOOKS = C_ICAP_INSTALL_TARGET_CONFIGS
> +
> +$(eval $(autotools-package))
> \ No newline at end of file

Missing newline at end of file.

Could you take those comments into account and send an updated version?

Thanks a lot!

Thomas Petazzoni
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH 3/3] New package: c-icap-modules Additionals modules for c-icap server.

[Thomas Petazzoni]

Dear Guillaume GARDET,

Title should be:

	c-icap-modules: new package

On Thu, 18 Sep 2014 11:27:06 +0200, Guillaume GARDET wrote:

> diff --git a/package/Config.in b/package/Config.in
> index 24a0a8e..86a50ae 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -951,6 +951,7 @@ menu "Networking applications"
>  	source "package/bridge-utils/Config.in"
>  	source "package/bwm-ng/Config.in"
>  	source "package/c-icap/Config.in"
> +	source "package/c-icap-modules/Config.in"
>  	source "package/can-utils/Config.in"
>  	source "package/chrony/Config.in"
>  	source "package/civetweb/Config.in"
> diff --git a/package/c-icap-modules/Config.in b/package/c-icap-modules/Config.in
> new file mode 100644
> index 0000000..7946c17
> --- /dev/null
> +++ b/package/c-icap-modules/Config.in
> @@ -0,0 +1,7 @@
> +config BR2_PACKAGE_C_ICAP_MODULES
> +        bool "c-icap-modules"

Indentation should be one tab.

> +	depends on BR2_PACKAGE_C_ICAP
> +	help
> +          Additionals modules for c-icap server.
> +
> +          http://c-icap.sourceforge.net/
> \ No newline at end of file

Missing newline at end of file.

> diff --git a/package/c-icap-modules/c-icap-modules.mk b/package/c-icap-modules/c-icap-modules.mk
> new file mode 100644
> index 0000000..683dd4f
> --- /dev/null
> +++ b/package/c-icap-modules/c-icap-modules.mk
> @@ -0,0 +1,27 @@
> +################################################################################
> +#
> +# c-icap-modules
> +#
> +################################################################################
> +
> +C_ICAP_MODULES_VERSION = 0.3.2
> +C_ICAP_MODULES_SOURCE = c_icap_modules-$(C_ICAP_MODULES_VERSION).tar.gz
> +C_ICAP_MODULES_SITE = http://downloads.sourceforge.net/c-icap/
> +C_ICAP_MODULES_INSTALL_STAGING = NO
> +C_ICAP_MODULES_INSTALL_TARGET = YES

Those two lines are not needed, since it's the default.

> +C_ICAP_MODULES_CONF_OPT = --with-c-icap=$(STAGING_DIR)/usr/ --includedir=$(STAGING_DIR)/usr/include/  \
> +		CPPFLAGS="-I $(STAGING_DIR)/usr/include/ -I $(STAGING_DIR)/usr/include/c_icap/" \
> +		CFLAGS="$(CFLAGS) -I $(STAGING_DIR)/usr/include/ -I $(STAGING_DIR)/usr/include/c_icap/"

--with-c-icap and --includedir can indeed go in
C_ICAP_MODULES_CONF_OPTS. However, the other two should go in
C_ICAP_MODULES_CONF_ENV.

Are you sure --includedir=$(STAGING_DIR)/usr/include is needed? It
seems wrong, because --includedir indicates the include directory on
the target... which clearly isn't $(STAGING_DIR)/usr/include.

Also, the two -I$(STAGING_DIR)/usr/include are not needed, the compiler
already looks there by default for header files. The
-I$(STAGING_DIR)/usr/include/c_icap/ might be needed, though.

> +C_ICAP_MODULES_DEPENDENCIES = c-icap
> +
> +define C_ICAP_MODULES_REMOVE_BROKEN_INCLUDE_PATH
> +	$(SED) "s#-I/usr/include -I/usr/include/c_icap###" $(C_ICAP_MODULES_DIR)/Makefile
> +	$(SED) "s#-I/usr/include -I/usr/include/c_icap###" $(C_ICAP_MODULES_DIR)/services/Makefile
> +	$(SED) "s#-I/usr/include -I/usr/include/c_icap###" $(C_ICAP_MODULES_DIR)/services/url_check/Makefile
> +	$(SED) "s#-I/usr/include -I/usr/include/c_icap###" $(C_ICAP_MODULES_DIR)/services/virus_scan/Makefile	
> +endef
> +
> +
> +C_ICAP_MODULES_POST_CONFIGURE_HOOKS = C_ICAP_MODULES_REMOVE_BROKEN_INCLUDE_PATH

I'm tempted to say that those fixes should be done by using a patch
rather than using some SED expression. And of course, the patch should
be submitted to the upstream c-icap-modules project, so that one day, a
new upstream version can be used in Buildroot and we can remove the
fixup patch.

> +
> +$(eval $(autotools-package))
> \ No newline at end of file

Missing newline at end of file.

Same as the other patches: could you take into account the comments,
and submit an updated version?

Thanks a lot!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH 1/3] package/squid: enable ICAP server support in squid and define logdir, pidfile and swapdir

[Guillaume GARDET - Oliséo]

Hi Thomas,


Le 05/10/2014 23:52, Thomas Petazzoni a ?crit :
> Dear Guillaume GARDET,
>
> On Thu, 18 Sep 2014 11:27:04 +0200, Guillaume GARDET wrote:
>> This patch adds ICAP server support to squid and also defines logdir,
>> pidfile and swapdir.
>>
>> Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>
> Thanks for this contribution. A few comments below.
>
>> diff --git a/package/squid/squid.mk b/package/squid/squid.mk
>> index 6dffae8..f68a402 100644
>> --- a/package/squid/squid.mk
>> +++ b/package/squid/squid.mk
>> @@ -26,7 +26,11 @@ SQUID_CONF_OPT =	--enable-async-io=8 --enable-linux-netfilter \
>>   			--enable-auth-negotiate="wrapper" \
>>   			--enable-auth-ntlm="fake" \
>>   			--disable-strict-error-checking \
>> -			--enable-external-acl-helpers="file_userip"
>> +			--enable-external-acl-helpers="file_userip" \
>> +			--with-logdir=/var/log/squid/ \
>> +			--with-pidfile=/var/run/squid.pid \
>> +			--with-swapdir=/var/cache/squid/ \
>> +			--enable-icap-client
>>   
>>   # On uClibc librt needs libpthread
>>   ifeq ($(BR2_TOOLCHAIN_HAS_THREADS)$(BR2_TOOLCHAIN_USES_UCLIBC),yy)
>> @@ -45,6 +49,10 @@ define SQUID_CLEANUP_TARGET
>>   		cachemgr.conf mime.conf.default squid.conf.default)
>>   endef
>>   
>> -SQUID_POST_INSTALL_TARGET_HOOKS += SQUID_CLEANUP_TARGET
>> +define SQUID_CREATE_MISSING_FOLDER
>> +	mkdir -p $(TARGET_DIR)/var/log/squid/
>> +endef
>> +
>> +SQUID_POST_INSTALL_TARGET_HOOKS += SQUID_CLEANUP_TARGET SQUID_CREATE_MISSING_FOLDER
>>   
>>   $(eval $(autotools-package))
> I believe the changes of adding the log directory, pidfile and swapdir
> on one side, and adding the ICAP client option on the other side should
> be two separate patches.

Ok, will submit two patches then.

>
> Also, how much does the ICAP client support adds to Squid size? I'm
> trying to see if this should be optional instead of always enabled like
> you're proposing.

Without: 2.7 MB,and with c-icap: 3.0 MB

Should I make it optionnal?


Guillaume

[Buildroot] [PATCH 1/3] package/squid: enable ICAP server support in squid and define logdir, pidfile and swapdir

[Thomas Petazzoni]

Hello,

On Wed, 08 Oct 2014 22:03:27 +0200, Guillaume GARDET - Olis?o wrote:

> > I believe the changes of adding the log directory, pidfile and swapdir
> > on one side, and adding the ICAP client option on the other side should
> > be two separate patches.
> 
> Ok, will submit two patches then.

Great thanks.

> > Also, how much does the ICAP client support adds to Squid size? I'm
> > trying to see if this should be optional instead of always enabled like
> > you're proposing.
> 
> Without: 2.7 MB,and with c-icap: 3.0 MB
> 
> Should I make it optionnal?

I would tend to say yes. Making it optional not only saves space, but
it also makes it explicit that our Squid build can support the ICAP
client functionality.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH 1/3] package/squid: enable ICAP server support in squid and define logdir, pidfile and swapdir

[Gustavo Zacarias]

On 10/08/2014 05:18 PM, Thomas Petazzoni wrote:

>> Without: 2.7 MB,and with c-icap: 3.0 MB
>>
>> Should I make it optionnal?
> 
> I would tend to say yes. Making it optional not only saves space, but
> it also makes it explicit that our Squid build can support the ICAP
> client functionality.

I wouldn't mind keeping it as is, squid is a big RAM and storage
sinkhole so it's a no-go for memory-constrained targets in general,
after all it's a caching (in disk) proxy with the RAM usage for keeping
indexes at hand.
Regards.

[Buildroot] [PATCH 1/3] package/squid: enable ICAP server support in squid and define logdir, pidfile and swapdir

[Guillaume GARDET - Oliséo]

Le 08/10/2014 22:24, Gustavo Zacarias a ?crit :
> On 10/08/2014 05:18 PM, Thomas Petazzoni wrote:
>
>>> Without: 2.7 MB,and with c-icap: 3.0 MB
>>>
>>> Should I make it optionnal?
>> I would tend to say yes. Making it optional not only saves space, but
>> it also makes it explicit that our Squid build can support the ICAP
>> client functionality.
> I wouldn't mind keeping it as is, squid is a big RAM and storage
> sinkhole so it's a no-go for memory-constrained targets in general,
> after all it's a caching (in disk) proxy with the RAM usage for keeping
> indexes at hand.
> Regards.
>
>

Then, should I make it optionnal or not? I think it should be ok to keep it included as it will never be used on underpowered SoC.

Thomas, would you be ok with that?

Guillaume

[Buildroot] [PATCH 1/3] package/squid: enable ICAP server support in squid and define logdir, pidfile and swapdir

[Thomas Petazzoni]

Dear Guillaume GARDET - Olis?o,

On Thu, 16 Oct 2014 11:52:19 +0200, Guillaume GARDET - Olis?o wrote:

> Then, should I make it optionnal or not? I think it should be ok to
> keep it included as it will never be used on underpowered SoC.
> 
> Thomas, would you be ok with that?

Yes, fair enough, keeping it always enabled is fine with me.

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH V2 0/4] Add ICAP server support

[Guillaume GARDET]

Hi,

This patch series enable ICAP support in squid and add c-icap and c-icap-modules packages.

Comments done on V1 have been integrated.


Regards,

Guillaume

---

Guillaume GARDET (4):
  package/squid: define the log directory, pidfile and swapdir
  package/squid: enable ICAP server support in squid
  c-icap: new package
  c-icap-modules: new package

 package/Config.in                        |  2 ++
 package/c-icap-modules/Config.in         |  7 +++++++
 package/c-icap-modules/c-icap-modules.mk | 27 +++++++++++++++++++++++++++
 package/c-icap/Config.in                 | 10 ++++++++++
 package/c-icap/c-icap.mk                 | 27 +++++++++++++++++++++++++++
 package/squid/squid.mk                   | 12 ++++++++++--
 6 files changed, 83 insertions(+), 2 deletions(-)
 create mode 100644 package/c-icap-modules/Config.in
 create mode 100644 package/c-icap-modules/c-icap-modules.mk
 create mode 100644 package/c-icap/Config.in
 create mode 100644 package/c-icap/c-icap.mk

-- 
1.8.4.5

[Buildroot] [PATCH V2 1/4] package/squid: define the log directory, pidfile and swapdir

[Guillaume GARDET]

Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>

---
 package/squid/squid.mk | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/package/squid/squid.mk b/package/squid/squid.mk
index e8a2ccc..ed51881 100644
--- a/package/squid/squid.mk
+++ b/package/squid/squid.mk
@@ -26,7 +26,10 @@ SQUID_CONF_OPTS =	--enable-async-io=8 --enable-linux-netfilter \
 			--enable-auth-negotiate="wrapper" \
 			--enable-auth-ntlm="fake" \
 			--disable-strict-error-checking \
-			--enable-external-acl-helpers="file_userip"
+			--enable-external-acl-helpers="file_userip" \
+			--with-logdir=/var/log/squid/ \
+			--with-pidfile=/var/run/squid.pid \
+			--with-swapdir=/var/cache/squid/
 
 # On uClibc librt needs libpthread
 ifeq ($(BR2_TOOLCHAIN_HAS_THREADS)$(BR2_TOOLCHAIN_USES_UCLIBC),yy)
-- 
1.8.4.5

[Buildroot] [PATCH V2 2/4] package/squid: enable ICAP server support in squid

[Guillaume GARDET]

Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>

---
 package/squid/squid.mk | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/package/squid/squid.mk b/package/squid/squid.mk
index ed51881..652fdb8 100644
--- a/package/squid/squid.mk
+++ b/package/squid/squid.mk
@@ -29,7 +29,8 @@ SQUID_CONF_OPTS =	--enable-async-io=8 --enable-linux-netfilter \
 			--enable-external-acl-helpers="file_userip" \
 			--with-logdir=/var/log/squid/ \
 			--with-pidfile=/var/run/squid.pid \
-			--with-swapdir=/var/cache/squid/
+			--with-swapdir=/var/cache/squid/ \
+			--enable-icap-client
 
 # On uClibc librt needs libpthread
 ifeq ($(BR2_TOOLCHAIN_HAS_THREADS)$(BR2_TOOLCHAIN_USES_UCLIBC),yy)
@@ -48,6 +49,10 @@ define SQUID_CLEANUP_TARGET
 		cachemgr.conf mime.conf.default squid.conf.default)
 endef
 
-SQUID_POST_INSTALL_TARGET_HOOKS += SQUID_CLEANUP_TARGET
+define SQUID_CREATE_MISSING_FOLDER
+	mkdir -p $(TARGET_DIR)/var/log/squid/
+endef
+
+SQUID_POST_INSTALL_TARGET_HOOKS += SQUID_CLEANUP_TARGET SQUID_CREATE_MISSING_FOLDER
 
 $(eval $(autotools-package))
-- 
1.8.4.5

[Buildroot] [PATCH V2 3/4] c-icap: new package

[Guillaume GARDET]

Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>

---
 package/Config.in        |  1 +
 package/c-icap/Config.in | 10 ++++++++++
 package/c-icap/c-icap.mk | 27 +++++++++++++++++++++++++++
 3 files changed, 38 insertions(+)
 create mode 100644 package/c-icap/Config.in
 create mode 100644 package/c-icap/c-icap.mk

diff --git a/package/Config.in b/package/Config.in
index 28cf703..fe0af07 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -983,6 +983,7 @@ menu "Networking applications"
 	source "package/boa/Config.in"
 	source "package/bridge-utils/Config.in"
 	source "package/bwm-ng/Config.in"
+	source "package/c-icap/Config.in"
 	source "package/can-utils/Config.in"
 	source "package/chrony/Config.in"
 	source "package/civetweb/Config.in"
diff --git a/package/c-icap/Config.in b/package/c-icap/Config.in
new file mode 100644
index 0000000..e8ac241
--- /dev/null
+++ b/package/c-icap/Config.in
@@ -0,0 +1,10 @@
+config BR2_PACKAGE_C_ICAP
+	bool "c-icap"
+	help
+	  c-icap is an implementation of an ICAP server. It can be used with 
+	  HTTP proxies that support the ICAP protocol to implement content 
+	  adaptation and filtering services.
+	  Most of the commercial HTTP proxies must support the ICAP protocol.
+	  The open source Squid 3.x proxy server supports it.
+
+	  http://c-icap.sourceforge.net/
diff --git a/package/c-icap/c-icap.mk b/package/c-icap/c-icap.mk
new file mode 100644
index 0000000..8f93638
--- /dev/null
+++ b/package/c-icap/c-icap.mk
@@ -0,0 +1,27 @@
+################################################################################
+#
+# c-icap
+#
+################################################################################
+
+C_ICAP_VERSION = 0.3.5
+C_ICAP_SOURCE = c_icap-$(C_ICAP_VERSION).tar.gz
+C_ICAP_SITE = http://downloads.sourceforge.net/c-icap/
+C_ICAP_LICENSE = LGPLv2.1+
+C_ICAP_LICENSE_FILES = COPYING
+C_ICAP_INSTALL_STAGING = YES
+C_ICAP_CONF_OPTS = --sysconfdir=/etc --with-process-mutexes=posix
+ifeq ($(BR2_LARGEFILE),y)
+C_ICAP_CONF_OPTS += --enable-large-files
+else
+C_ICAP_CONF_OPTS += --disable-large-files
+endif
+
+define C_ICAP_INSTALL_TARGET_CONFIGS
+	mkdir -p $(TARGET_DIR)/var/run/c-icap/
+	mkdir -p $(TARGET_DIR)/tmp/c-icap/
+endef
+
+C_ICAP_POST_INSTALL_TARGET_HOOKS = C_ICAP_INSTALL_TARGET_CONFIGS
+
+$(eval $(autotools-package))
-- 
1.8.4.5

[Buildroot] [PATCH V2 4/4] c-icap-modules: new package

[Guillaume GARDET]

Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>

---
 package/Config.in                        |  1 +
 package/c-icap-modules/Config.in         |  7 +++++++
 package/c-icap-modules/c-icap-modules.mk | 27 +++++++++++++++++++++++++++
 3 files changed, 35 insertions(+)
 create mode 100644 package/c-icap-modules/Config.in
 create mode 100644 package/c-icap-modules/c-icap-modules.mk

diff --git a/package/Config.in b/package/Config.in
index fe0af07..8993814 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -984,6 +984,7 @@ menu "Networking applications"
 	source "package/bridge-utils/Config.in"
 	source "package/bwm-ng/Config.in"
 	source "package/c-icap/Config.in"
+	source "package/c-icap-modules/Config.in"
 	source "package/can-utils/Config.in"
 	source "package/chrony/Config.in"
 	source "package/civetweb/Config.in"
diff --git a/package/c-icap-modules/Config.in b/package/c-icap-modules/Config.in
new file mode 100644
index 0000000..616456b
--- /dev/null
+++ b/package/c-icap-modules/Config.in
@@ -0,0 +1,7 @@
+config BR2_PACKAGE_C_ICAP_MODULES
+	bool "c-icap-modules"
+	depends on BR2_PACKAGE_C_ICAP
+	help
+	  Additionals modules for c-icap server.
+
+	  http://c-icap.sourceforge.net/
diff --git a/package/c-icap-modules/c-icap-modules.mk b/package/c-icap-modules/c-icap-modules.mk
new file mode 100644
index 0000000..8c3be67
--- /dev/null
+++ b/package/c-icap-modules/c-icap-modules.mk
@@ -0,0 +1,27 @@
+################################################################################
+#
+# c-icap-modules
+#
+################################################################################
+
+C_ICAP_MODULES_VERSION = 0.3.2
+C_ICAP_MODULES_SOURCE = c_icap_modules-$(C_ICAP_MODULES_VERSION).tar.gz
+C_ICAP_MODULES_SITE = http://downloads.sourceforge.net/c-icap/
+C_ICAP_MODULES_LICENCE = GPLv2+
+C_ICAP_MODULES_LICENCE_FILE = COPYING
+C_ICAP_MODULES_CONF_OPTS = --with-c-icap=$(STAGING_DIR)/usr/
+C_ICAP_MODULES_CONF_ENV = CFLAGS="$(CFLAGS) -I$(STAGING_DIR)/usr/include/c_icap/"
+C_ICAP_MODULES_DEPENDENCIES = c-icap
+
+# This sed hack is needed since values are obtained using c-icap-config script
+define C_ICAP_MODULES_REMOVE_BROKEN_INCLUDE_PATH
+	$(SED) "s#-I/usr/include -I/usr/include/c_icap###" $(C_ICAP_MODULES_DIR)/Makefile
+	$(SED) "s#-I/usr/include -I/usr/include/c_icap###" $(C_ICAP_MODULES_DIR)/services/Makefile
+	$(SED) "s#-I/usr/include -I/usr/include/c_icap###" $(C_ICAP_MODULES_DIR)/services/url_check/Makefile
+	$(SED) "s#-I/usr/include -I/usr/include/c_icap###" $(C_ICAP_MODULES_DIR)/services/virus_scan/Makefile	
+endef
+
+
+C_ICAP_MODULES_POST_CONFIGURE_HOOKS = C_ICAP_MODULES_REMOVE_BROKEN_INCLUDE_PATH
+
+$(eval $(autotools-package))
-- 
1.8.4.5

[Buildroot] [PATCH V2 2/4] package/squid: enable ICAP server support in squid

[Thomas Petazzoni]

Dear Guillaume GARDET,

On Wed, 12 Nov 2014 17:16:31 +0100, Guillaume GARDET wrote:
> Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>
> 
> ---
>  package/squid/squid.mk | 9 +++++++--
>  1 file changed, 7 insertions(+), 2 deletions(-)
> 
> diff --git a/package/squid/squid.mk b/package/squid/squid.mk
> index ed51881..652fdb8 100644
> --- a/package/squid/squid.mk
> +++ b/package/squid/squid.mk
> @@ -29,7 +29,8 @@ SQUID_CONF_OPTS =	--enable-async-io=8 --enable-linux-netfilter \
>  			--enable-external-acl-helpers="file_userip" \
>  			--with-logdir=/var/log/squid/ \
>  			--with-pidfile=/var/run/squid.pid \
> -			--with-swapdir=/var/cache/squid/
> +			--with-swapdir=/var/cache/squid/ \
> +			--enable-icap-client

The commit title says you enable the "ICAP server", but in fact you are
enabling the ICAP client.

>  
>  # On uClibc librt needs libpthread
>  ifeq ($(BR2_TOOLCHAIN_HAS_THREADS)$(BR2_TOOLCHAIN_USES_UCLIBC),yy)
> @@ -48,6 +49,10 @@ define SQUID_CLEANUP_TARGET
>  		cachemgr.conf mime.conf.default squid.conf.default)
>  endef
>  
> -SQUID_POST_INSTALL_TARGET_HOOKS += SQUID_CLEANUP_TARGET
> +define SQUID_CREATE_MISSING_FOLDER
> +	mkdir -p $(TARGET_DIR)/var/log/squid/
> +endef

This is wrong, for two reasons:

 1/ It's related to your PATCH 1/4, which adds
    --with-logdir=/var/log/squid, so it should go with it.

 2/ Creating a directory in /var/log at build time doesn't make sense,
    because /var/log in Buildroot is a symbolic link to /tmp, and /tmp
    is mounted as a tmpfs so that the root filesystem can remain
    read-only. Therefore, the proper solution is to create this
    directory at boot time. We normally do this in the startup script
    provided by the package, but squid doesn't provide one.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH V2 0/4] Add ICAP server support

[Thomas Petazzoni]

Dear Guillaume GARDET,

On Wed, 12 Nov 2014 17:16:29 +0100, Guillaume GARDET wrote:

> This patch series enable ICAP support in squid and add c-icap and c-icap-modules packages.
> 
> Comments done on V1 have been integrated.

Please mention which comment have been taken into account when you
write a changelog.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH V2 3/4] c-icap: new package

[Thomas Petazzoni]

Dear Guillaume GARDET,

On Wed, 12 Nov 2014 17:16:32 +0100, Guillaume GARDET wrote:
> Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>
> 
> ---
>  package/Config.in        |  1 +
>  package/c-icap/Config.in | 10 ++++++++++
>  package/c-icap/c-icap.mk | 27 +++++++++++++++++++++++++++
>  3 files changed, 38 insertions(+)
>  create mode 100644 package/c-icap/Config.in
>  create mode 100644 package/c-icap/c-icap.mk
> 
> diff --git a/package/Config.in b/package/Config.in
> index 28cf703..fe0af07 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -983,6 +983,7 @@ menu "Networking applications"
>  	source "package/boa/Config.in"
>  	source "package/bridge-utils/Config.in"
>  	source "package/bwm-ng/Config.in"
> +	source "package/c-icap/Config.in"
>  	source "package/can-utils/Config.in"
>  	source "package/chrony/Config.in"
>  	source "package/civetweb/Config.in"
> diff --git a/package/c-icap/Config.in b/package/c-icap/Config.in
> new file mode 100644
> index 0000000..e8ac241
> --- /dev/null
> +++ b/package/c-icap/Config.in
> @@ -0,0 +1,10 @@
> +config BR2_PACKAGE_C_ICAP
> +	bool "c-icap"

A quick look at the source seems to indicate that this package uses
thread support, so you should depend on BR2_TOOLCHAIN_HAS_THREADS, and
add the corresponding comment (see other packages for examples).

> +	help
> +	  c-icap is an implementation of an ICAP server. It can be used with 
> +	  HTTP proxies that support the ICAP protocol to implement content 
> +	  adaptation and filtering services.
> +	  Most of the commercial HTTP proxies must support the ICAP protocol.
> +	  The open source Squid 3.x proxy server supports it.
> +
> +	  http://c-icap.sourceforge.net/
> diff --git a/package/c-icap/c-icap.mk b/package/c-icap/c-icap.mk
> new file mode 100644
> index 0000000..8f93638
> --- /dev/null
> +++ b/package/c-icap/c-icap.mk
> @@ -0,0 +1,27 @@
> +################################################################################
> +#
> +# c-icap
> +#
> +################################################################################
> +
> +C_ICAP_VERSION = 0.3.5
> +C_ICAP_SOURCE = c_icap-$(C_ICAP_VERSION).tar.gz
> +C_ICAP_SITE = http://downloads.sourceforge.net/c-icap/
> +C_ICAP_LICENSE = LGPLv2.1+
> +C_ICAP_LICENSE_FILES = COPYING
> +C_ICAP_INSTALL_STAGING = YES
> +C_ICAP_CONF_OPTS = --sysconfdir=/etc --with-process-mutexes=posix

--sysconfdir=/etc is not needed, it is already passed by the
autotools-package infrastructure to all target packages.

> +ifeq ($(BR2_LARGEFILE),y)
> +C_ICAP_CONF_OPTS += --enable-large-files
> +else
> +C_ICAP_CONF_OPTS += --disable-large-files
> +endif
> +
> +define C_ICAP_INSTALL_TARGET_CONFIGS
> +	mkdir -p $(TARGET_DIR)/var/run/c-icap/
> +	mkdir -p $(TARGET_DIR)/tmp/c-icap/
> +endef

Like your change for squid, I don't think this makes sense because /tmp
is mounted as a tmpfs, and /var/run is a symlink to /tmp.

You also probably want:

ifeq ($(BR2_INET_IPV6),y)
C_ICAP_CONF_OPTS += --enable-ipv6
else
C_ICAP_CONF_OPTS += --disable-ipv6
endif

Probably also --without-perl, unless perl is a mandatory dependency.

Also, looking at configure.in, it needs the libdl library, so you'll
need to depend on !BR2_PREFER_STATIC_LIB in Config.in.

It also has some optional dependencies on zlib, bzip2, berkeleydb and
ldap.

For the first three, you can to handle the optional dependencies, by
doing something like:

ifeq ($(BR2_PACKAGE_ZLIB),y)
C_ICAP_CONF_OPTS += --with-zlib
C_ICAP_DEPENDENCIES += zlib
else
C_ICAP_CONF_OPTS += --without-zlib
endif

For the last one, since we don't have a LDAP package in Buildroot,
simply pass --without-ldap in C_ICAP_CONF_OPTS.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH V2 2/4] package/squid: enable ICAP server support in squid

[Guillaume GARDET - Oliséo]

Le 16/11/2014 22:38, Thomas Petazzoni a ?crit :
> Dear Guillaume GARDET,
>
> On Wed, 12 Nov 2014 17:16:31 +0100, Guillaume GARDET wrote:
>> Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>
>>
>> ---
>>   package/squid/squid.mk | 9 +++++++--
>>   1 file changed, 7 insertions(+), 2 deletions(-)
>>
>> diff --git a/package/squid/squid.mk b/package/squid/squid.mk
>> index ed51881..652fdb8 100644
>> --- a/package/squid/squid.mk
>> +++ b/package/squid/squid.mk
>> @@ -29,7 +29,8 @@ SQUID_CONF_OPTS =	--enable-async-io=8 --enable-linux-netfilter \
>>   			--enable-external-acl-helpers="file_userip" \
>>   			--with-logdir=/var/log/squid/ \
>>   			--with-pidfile=/var/run/squid.pid \
>> -			--with-swapdir=/var/cache/squid/
>> +			--with-swapdir=/var/cache/squid/ \
>> +			--enable-icap-client
> The commit title says you enable the "ICAP server", but in fact you are
> enabling the ICAP client.

ok.

>
>>   
>>   # On uClibc librt needs libpthread
>>   ifeq ($(BR2_TOOLCHAIN_HAS_THREADS)$(BR2_TOOLCHAIN_USES_UCLIBC),yy)
>> @@ -48,6 +49,10 @@ define SQUID_CLEANUP_TARGET
>>   		cachemgr.conf mime.conf.default squid.conf.default)
>>   endef
>>   
>> -SQUID_POST_INSTALL_TARGET_HOOKS += SQUID_CLEANUP_TARGET
>> +define SQUID_CREATE_MISSING_FOLDER
>> +	mkdir -p $(TARGET_DIR)/var/log/squid/
>> +endef
> This is wrong, for two reasons:
>
>   1/ It's related to your PATCH 1/4, which adds
>      --with-logdir=/var/log/squid, so it should go with it.

Right.

>
>   2/ Creating a directory in /var/log at build time doesn't make sense,
>      because /var/log in Buildroot is a symbolic link to /tmp, and /tmp
>      is mounted as a tmpfs so that the root filesystem can remain
>      read-only. Therefore, the proper solution is to create this
>      directory at boot time. We normally do this in the startup script
>      provided by the package, but squid doesn't provide one.

What would be the solution then?


Guillaume

[Buildroot] [PATCH V2 2/4] package/squid: enable ICAP server support in squid

[Thomas Petazzoni]

Hello,

On Tue, 18 Nov 2014 13:47:01 +0100, Guillaume GARDET - Olis?o wrote:

> >   2/ Creating a directory in /var/log at build time doesn't make sense,
> >      because /var/log in Buildroot is a symbolic link to /tmp, and /tmp
> >      is mounted as a tmpfs so that the root filesystem can remain
> >      read-only. Therefore, the proper solution is to create this
> >      directory at boot time. We normally do this in the startup script
> >      provided by the package, but squid doesn't provide one.
> 
> What would be the solution then?

I guess the best solution is probably to add an init script to start
Squid at boot time.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH V3 0/5] Add ICAP server support

[Guillaume GARDET]

* Changes in V3:
  - squid: fix patch split, fix patch title (ICAP server => ICAP client),
  add init script, remove dir creation in *.mk files.
  - c-icap: depends on BR2_TOOLCHAIN_HAS_THREADS and !BR2_PREFER_STATIC_LIB.
  Use zlib, bzip2 and berkeleydb, if available. Enable IPv6, if available.
  Disable explicitly LDAP. Add init script. Remove dir creation in *.mk files.
  - c-icap-modules: no changes

* Changes in V2:
  - squid: split patch
  - c-icap: Fix commit message, fix new lines at end of file, remove 
  unneeded lines/options, enable large files support if available.
  - c-icap-modules: Fix commit message, fix indentation, new lines at end of 
  file, remove unneeded lines/options

---
Guillaume GARDET (5):
  package/squid: define the log directory, pidfile and swapdir
  package/squid: add init script
  package/squid: enable ICAP client support in squid
  c-icap: new package
  c-icap-modules: new package

 package/Config.in                        |  2 +
 package/c-icap-modules/Config.in         |  7 ++++
 package/c-icap-modules/c-icap-modules.mk | 27 ++++++++++++++
 package/c-icap/Config.in                 | 15 ++++++++
 package/c-icap/S96cicap                  | 35 +++++++++++++++++
 package/c-icap/c-icap.mk                 | 64 ++++++++++++++++++++++++++++++++
 package/squid/S97squid                   | 36 ++++++++++++++++++
 package/squid/squid.mk                   | 13 ++++++-
 8 files changed, 198 insertions(+), 1 deletion(-)
 create mode 100644 package/c-icap-modules/Config.in
 create mode 100644 package/c-icap-modules/c-icap-modules.mk
 create mode 100644 package/c-icap/Config.in
 create mode 100644 package/c-icap/S96cicap
 create mode 100644 package/c-icap/c-icap.mk
 create mode 100644 package/squid/S97squid

-- 
1.8.4.5

[Buildroot] [PATCH V3 1/5] package/squid: define the log directory, pidfile and swapdir

[Guillaume GARDET]

Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>

---
 package/squid/squid.mk | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/package/squid/squid.mk b/package/squid/squid.mk
index e8a2ccc..ed51881 100644
--- a/package/squid/squid.mk
+++ b/package/squid/squid.mk
@@ -26,7 +26,10 @@ SQUID_CONF_OPTS =	--enable-async-io=8 --enable-linux-netfilter \
 			--enable-auth-negotiate="wrapper" \
 			--enable-auth-ntlm="fake" \
 			--disable-strict-error-checking \
-			--enable-external-acl-helpers="file_userip"
+			--enable-external-acl-helpers="file_userip" \
+			--with-logdir=/var/log/squid/ \
+			--with-pidfile=/var/run/squid.pid \
+			--with-swapdir=/var/cache/squid/
 
 # On uClibc librt needs libpthread
 ifeq ($(BR2_TOOLCHAIN_HAS_THREADS)$(BR2_TOOLCHAIN_USES_UCLIBC),yy)
-- 
1.8.4.5

[Buildroot] [PATCH V3 2/5] package/squid: add init script

[Guillaume GARDET]

Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>

---
 package/squid/S97squid | 36 ++++++++++++++++++++++++++++++++++++
 package/squid/squid.mk |  7 +++++++
 2 files changed, 43 insertions(+)
 create mode 100644 package/squid/S97squid

diff --git a/package/squid/S97squid b/package/squid/S97squid
new file mode 100644
index 0000000..0908784
--- /dev/null
+++ b/package/squid/S97squid
@@ -0,0 +1,36 @@
+#!/bin/sh
+
+SQUID_BIN="/usr/sbin/squid"
+SQUID_LOG_DIR="/var/log/squid"
+SQUID_PID=/var/run/squid.pid
+SQUID_CONF=/etc/squid.conf
+SQUID_OPTS=""
+
+# (Re)create log directory and give access to squid user (nobody)
+if [ ! -d $SQUID_LOG_DIR ]; then
+	mkdir -p $SQUID_LOG_DIR 
+	chmod 777 $SQUID_LOG_DIR
+fi
+
+case "$1" in
+    start)
+	echo -n "Starting WWW-proxy squid "
+	start-stop-daemon -S -x $SQUID_BIN -- -f $SQUID_CONF $SQUID_OPTS
+	[ $? == 0 ] && echo "OK" || echo "FAIL"
+	;;
+    stop)
+	echo -n "Shutting down WWW-proxy squid "
+	start-stop-daemon -K -x $SQUID_BIN
+	;;
+    restart | force-reload)
+	$0 stop
+	sleep 2
+	$0 start
+	if [ "$?" != "0" ]; then
+		exit 1
+	fi
+	;;
+     *)
+	echo "Usage: /etc/init.d/S97squid {start|stop|restart|force-reload}"
+	exit 1
+esac
diff --git a/package/squid/squid.mk b/package/squid/squid.mk
index ed51881..b0fa69f 100644
--- a/package/squid/squid.mk
+++ b/package/squid/squid.mk
@@ -50,4 +50,11 @@ endef
 
 SQUID_POST_INSTALL_TARGET_HOOKS += SQUID_CLEANUP_TARGET
 
+define SQUID_INSTALL_INIT_SYSV
+        # install start/stop script
+        @if [ ! -f $(TARGET_DIR)/etc/init.d/S97squid ]; then \
+                $(INSTALL) -m 0755 -D package/squid/S97squid $(TARGET_DIR)/etc/init.d/S97squid; \
+        fi
+endef
+
 $(eval $(autotools-package))
-- 
1.8.4.5

[Buildroot] [PATCH V3 3/5] package/squid: enable ICAP client support in squid

[Guillaume GARDET]

Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>

---
 package/squid/squid.mk | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/package/squid/squid.mk b/package/squid/squid.mk
index b0fa69f..cd4090e 100644
--- a/package/squid/squid.mk
+++ b/package/squid/squid.mk
@@ -29,7 +29,8 @@ SQUID_CONF_OPTS =	--enable-async-io=8 --enable-linux-netfilter \
 			--enable-external-acl-helpers="file_userip" \
 			--with-logdir=/var/log/squid/ \
 			--with-pidfile=/var/run/squid.pid \
-			--with-swapdir=/var/cache/squid/
+			--with-swapdir=/var/cache/squid/ \
+			--enable-icap-client
 
 # On uClibc librt needs libpthread
 ifeq ($(BR2_TOOLCHAIN_HAS_THREADS)$(BR2_TOOLCHAIN_USES_UCLIBC),yy)
-- 
1.8.4.5

[Buildroot] [PATCH V3 4/5] c-icap: new package

[Guillaume GARDET]

Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>

---
 package/Config.in        |  1 +
 package/c-icap/Config.in | 15 ++++++++++++
 package/c-icap/S96cicap  | 35 ++++++++++++++++++++++++++
 package/c-icap/c-icap.mk | 64 ++++++++++++++++++++++++++++++++++++++++++++++++
 4 files changed, 115 insertions(+)
 create mode 100644 package/c-icap/Config.in
 create mode 100644 package/c-icap/S96cicap
 create mode 100644 package/c-icap/c-icap.mk

diff --git a/package/Config.in b/package/Config.in
index 28cf703..fe0af07 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -983,6 +983,7 @@ menu "Networking applications"
 	source "package/boa/Config.in"
 	source "package/bridge-utils/Config.in"
 	source "package/bwm-ng/Config.in"
+	source "package/c-icap/Config.in"
 	source "package/can-utils/Config.in"
 	source "package/chrony/Config.in"
 	source "package/civetweb/Config.in"
diff --git a/package/c-icap/Config.in b/package/c-icap/Config.in
new file mode 100644
index 0000000..71ab470
--- /dev/null
+++ b/package/c-icap/Config.in
@@ -0,0 +1,15 @@
+config BR2_PACKAGE_C_ICAP
+	bool "c-icap"
+	depends on !BR2_PREFER_STATIC_LIB 
+	depends on BR2_TOOLCHAIN_HAS_THREADS
+	help
+	  c-icap is an implementation of an ICAP server. It can be used with 
+	  HTTP proxies that support the ICAP protocol to implement content 
+	  adaptation and filtering services.
+	  Most of the commercial HTTP proxies must support the ICAP protocol.
+	  The open source Squid 3.x proxy server supports it.
+
+	  http://c-icap.sourceforge.net/
+
+comment "c-icap needs a toolchain w/ threads and libdl library"
+	depends on BR2_PREFER_STATIC_LIB  || !BR2_TOOLCHAIN_HAS_THREADS
diff --git a/package/c-icap/S96cicap b/package/c-icap/S96cicap
new file mode 100644
index 0000000..f176d9e
--- /dev/null
+++ b/package/c-icap/S96cicap
@@ -0,0 +1,35 @@
+#!/bin/sh
+
+CICAP_BIN="/usr/bin/c-icap"
+CICAP_DIR="/var/run/c-icap"
+CICAP_LOG_DIR="/var/log/c-icap/"
+# (Re)create directories
+if [ ! -d $CICAP_DIR ]; then
+	mkdir -p $CICAP_DIR 
+fi
+if [ ! -d $CICAP_LOG_DIR ]; then
+	mkdir -p $CICAP_LOG_DIR 
+fi
+
+case "$1" in
+    start)
+	echo -n "Starting c-icap server "
+	start-stop-daemon -S -x $CICAP_BIN
+	[ $? == 0 ] && echo "OK" || echo "FAIL"
+	;;
+    stop)
+	echo -n "Shutting down c-icap server "
+	start-stop-daemon -K -x $CICAP_BIN
+	;;
+    restart | force-reload)
+	$0 stop
+	sleep 2
+	$0 start
+	if [ "$?" != "0" ]; then
+		exit 1
+	fi
+	;;
+     *)
+	echo "Usage: /etc/init.d/S96cicap {start|stop|restart|force-reload}"
+	exit 1
+esac
diff --git a/package/c-icap/c-icap.mk b/package/c-icap/c-icap.mk
new file mode 100644
index 0000000..05e2f6a
--- /dev/null
+++ b/package/c-icap/c-icap.mk
@@ -0,0 +1,64 @@
+################################################################################
+#
+# c-icap
+#
+################################################################################
+
+C_ICAP_VERSION = 0.3.5
+C_ICAP_SOURCE = c_icap-$(C_ICAP_VERSION).tar.gz
+C_ICAP_SITE = http://downloads.sourceforge.net/c-icap/
+C_ICAP_LICENSE = LGPLv2.1+
+C_ICAP_LICENSE_FILES = COPYING
+C_ICAP_INSTALL_STAGING = YES
+C_ICAP_CONF_OPTS = --with-process-mutexes=posix --without-ldap
+
+ifeq ($(BR2_LARGEFILE),y)
+C_ICAP_CONF_OPTS += --enable-large-files
+else
+C_ICAP_CONF_OPTS += --disable-large-files
+endif
+
+ifeq ($(BR2_INET_IPV6),y)
+C_ICAP_CONF_OPTS += --enable-ipv6
+else
+C_ICAP_CONF_OPTS += --disable-ipv6
+endif
+
+ifeq ($(BR2_PACKAGE_BERKELEYDB),y)
+C_ICAP_CONF_OPTS += --with-berkeleydb
+C_ICAP_DEPENDENCIES += berkeleydb
+else
+C_ICAP_CONF_OPTS += --without-berkeleydb
+endif
+
+ifeq ($(BR2_PACKAGE_BZIP2),y)
+C_ICAP_CONF_OPTS += --with-bzip2
+C_ICAP_DEPENDENCIES += bzip2
+else
+C_ICAP_CONF_OPTS += --without-bzip2
+endif
+
+ifeq ($(BR2_PACKAGE_PERL),y)
+C_ICAP_CONF_OPTS += --with-perl
+C_ICAP_DEPENDENCIES += perl
+else
+C_ICAP_CONF_OPTS += --without-perl
+endif
+
+ifeq ($(BR2_PACKAGE_ZLIB),y)
+C_ICAP_CONF_OPTS += --with-zlib
+C_ICAP_DEPENDENCIES += zlib
+else
+C_ICAP_CONF_OPTS += --without-zlib
+endif
+
+C_ICAP_POST_INSTALL_TARGET_HOOKS = C_ICAP_INSTALL_TARGET_CONFIGS
+
+define C_ICAP_INSTALL_INIT_SYSV
+        # install start/stop script
+        @if [ ! -f $(TARGET_DIR)/etc/init.d/S96cicap ]; then \
+                $(INSTALL) -m 0755 -D package/c-icap/S96cicap $(TARGET_DIR)/etc/init.d/S96cicap; \
+        fi
+endef
+
+$(eval $(autotools-package))
-- 
1.8.4.5

[Buildroot] [PATCH V3 5/5] c-icap-modules: new package

[Guillaume GARDET]

Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>

---
 package/Config.in                        |  1 +
 package/c-icap-modules/Config.in         |  7 +++++++
 package/c-icap-modules/c-icap-modules.mk | 27 +++++++++++++++++++++++++++
 3 files changed, 35 insertions(+)
 create mode 100644 package/c-icap-modules/Config.in
 create mode 100644 package/c-icap-modules/c-icap-modules.mk

diff --git a/package/Config.in b/package/Config.in
index fe0af07..8993814 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -984,6 +984,7 @@ menu "Networking applications"
 	source "package/bridge-utils/Config.in"
 	source "package/bwm-ng/Config.in"
 	source "package/c-icap/Config.in"
+	source "package/c-icap-modules/Config.in"
 	source "package/can-utils/Config.in"
 	source "package/chrony/Config.in"
 	source "package/civetweb/Config.in"
diff --git a/package/c-icap-modules/Config.in b/package/c-icap-modules/Config.in
new file mode 100644
index 0000000..616456b
--- /dev/null
+++ b/package/c-icap-modules/Config.in
@@ -0,0 +1,7 @@
+config BR2_PACKAGE_C_ICAP_MODULES
+	bool "c-icap-modules"
+	depends on BR2_PACKAGE_C_ICAP
+	help
+	  Additionals modules for c-icap server.
+
+	  http://c-icap.sourceforge.net/
diff --git a/package/c-icap-modules/c-icap-modules.mk b/package/c-icap-modules/c-icap-modules.mk
new file mode 100644
index 0000000..9029b1c
--- /dev/null
+++ b/package/c-icap-modules/c-icap-modules.mk
@@ -0,0 +1,27 @@
+################################################################################
+#
+# c-icap-modules
+#
+################################################################################
+
+C_ICAP_MODULES_VERSION = 0.3.2
+C_ICAP_MODULES_SOURCE = c_icap_modules-$(C_ICAP_MODULES_VERSION).tar.gz
+C_ICAP_MODULES_SITE = http://downloads.sourceforge.net/c-icap/
+C_ICAP_MODULES_LICENCE = GPLv2+
+C_ICAP_MODULES_LICENCE_FILE = COPYING
+C_ICAP_MODULES_CONF_OPTS = --with-c-icap=$(STAGING_DIR)/usr/
+C_ICAP_MODULES_CONF_ENV = CFLAGS="$(CFLAGS) -I$(STAGING_DIR)/usr/include/c_icap/"
+C_ICAP_MODULES_DEPENDENCIES = c-icap
+
+# This sed hack is needed since values are obtained using c-icap-config script
+define C_ICAP_MODULES_REMOVE_BROKEN_INCLUDE_PATH
+	$(SED) "s#-I/usr/include -I/usr/include/c_icap###" $(C_ICAP_MODULES_DIR)/Makefile
+	$(SED) "s#-I/usr/include -I/usr/include/c_icap###" $(C_ICAP_MODULES_DIR)/services/Makefile
+	$(SED) "s#-I/usr/include -I/usr/include/c_icap###" $(C_ICAP_MODULES_DIR)/services/url_check/Makefile
+	$(SED) "s#-I/usr/include -I/usr/include/c_icap###" $(C_ICAP_MODULES_DIR)/services/virus_scan/Makefile	
+endef
+
+
+C_ICAP_MODULES_POST_CONFIGURE_HOOKS = C_ICAP_MODULES_REMOVE_BROKEN_INCLUDE_PATH
+
+$(eval $(autotools-package))
-- 
1.8.4.5

[Buildroot] [PATCH V3 1/5] package/squid: define the log directory, pidfile and swapdir

[Thomas Petazzoni]

Dear Guillaume GARDET,

On Tue, 18 Nov 2014 15:54:43 +0100, Guillaume GARDET wrote:
> Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>

Patch applied to next, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH V3 2/5] package/squid: add init script

[Thomas Petazzoni]

Dear Guillaume GARDET,

On Tue, 18 Nov 2014 15:54:44 +0100, Guillaume GARDET wrote:

> +# (Re)create log directory and give access to squid user (nobody)
> +if [ ! -d $SQUID_LOG_DIR ]; then
> +	mkdir -p $SQUID_LOG_DIR 
> +	chmod 777 $SQUID_LOG_DIR

Clearly, this doesn't look good. Don't we have a better option than 777
the log directory?

> +    restart | force-reload)
> +	$0 stop
> +	sleep 2

sleep 2 really needed here?

> +	$0 start
> +	if [ "$?" != "0" ]; then
> +		exit 1
> +	fi
> +	;;
> +     *)
> +	echo "Usage: /etc/init.d/S97squid {start|stop|restart|force-reload}"

Replace /etc/init.d/S97squid by $0 maybe?

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH V3 3/5] package/squid: enable ICAP client support in squid

[Thomas Petazzoni]

Dear Guillaume GARDET,

On Tue, 18 Nov 2014 15:54:45 +0100, Guillaume GARDET wrote:
> Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>
> 
> ---
>  package/squid/squid.mk | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

Applied to next, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH V3 2/5] package/squid: add init script

[Guillaume GARDET - Oliséo]

Le 23/11/2014 22:18, Thomas Petazzoni a ?crit :
> Dear Guillaume GARDET,
>
> On Tue, 18 Nov 2014 15:54:44 +0100, Guillaume GARDET wrote:
>
>> +# (Re)create log directory and give access to squid user (nobody)
>> +if [ ! -d $SQUID_LOG_DIR ]; then
>> +	mkdir -p $SQUID_LOG_DIR
>> +	chmod 777 $SQUID_LOG_DIR
> Clearly, this doesn't look good. Don't we have a better option than 777
> the log directory?

Maybe create a squid user? Not sure how to handle this in buildroot. What do you think?

>
>> +    restart | force-reload)
>> +	$0 stop
>> +	sleep 2
> sleep 2 really needed here?

Not sure.

>
>> +	$0 start
>> +	if [ "$?" != "0" ]; then
>> +		exit 1
>> +	fi
>> +	;;
>> +     *)
>> +	echo "Usage: /etc/init.d/S97squid {start|stop|restart|force-reload}"
> Replace /etc/init.d/S97squid by $0 maybe?

If you want.


Guillaume

[Buildroot] [PATCH V3 2/5] package/squid: add init script

[Thomas Petazzoni]

Hello,

On Sat, 29 Nov 2014 20:41:52 +0100, Guillaume GARDET - Olis?o wrote:

> >> +# (Re)create log directory and give access to squid user (nobody)
> >> +if [ ! -d $SQUID_LOG_DIR ]; then
> >> +	mkdir -p $SQUID_LOG_DIR
> >> +	chmod 777 $SQUID_LOG_DIR
> > Clearly, this doesn't look good. Don't we have a better option than 777
> > the log directory?
> 
> Maybe create a squid user? Not sure how to handle this in buildroot. What do you think?

Well, if squid currently runs as root, why do you need to "chmod 777"
the log directory?

But indeed, it would be nicer to run squid as a user. Look at
<pkg>_USERS and <pkg>_PERMISSIONS in the Buildroot manual, or grep
through the packages for such definitions, you'll see some examples on
how to achieve that.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH V3 4/5] c-icap: new package

[Thomas Petazzoni]

Dear Guillaume GARDET,

On Tue, 18 Nov 2014 15:54:46 +0100, Guillaume GARDET wrote:
> Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>
> 
> ---
>  package/Config.in        |  1 +
>  package/c-icap/Config.in | 15 ++++++++++++
>  package/c-icap/S96cicap  | 35 ++++++++++++++++++++++++++
>  package/c-icap/c-icap.mk | 64 ++++++++++++++++++++++++++++++++++++++++++++++++
>  4 files changed, 115 insertions(+)
>  create mode 100644 package/c-icap/Config.in
>  create mode 100644 package/c-icap/S96cicap
>  create mode 100644 package/c-icap/c-icap.mk

I've applied, but a lot of changes were needed to make the patch
correct. In fact, with the patch you posted, the c-icap daemon was not
even starting properly at boot time because the c-icap.conf file
contained invalid paths.

Here are the changes I made before committing:

    [Thomas:
     - Remove trailing whitespace everywhere
     - Rewrap the Config.in help text
     - Fix Config.in comment, the proper text for BR2_PREFER_STATIC_LIB is
       "dynamic library", not "libdl library"
     - Use a PID file in the init script
     - Show OK / FAIL when stopping the service
     - Remove the largefile/IPv6 handling in c-icap.mk, since they are now
       mandatory. We therefore pass --enable-large-files and --enable-ipv6
       unconditionally.
     - Remove the reference to a non-existing
       C_ICAP_INSTALL_TARGET_CONFIGS install target hook.
     - Install the init script unconditionally.
     - Remove unneeded example configuration files (*.default) from /etc,
       and fixup the c-icap.conf so that the default values work with
       Buildroot
     - Properly handle the *-config scripts installed by c-icap.
     - Add hash file.]

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH V3 5/5] c-icap-modules: new package

[Thomas Petazzoni]

Dear Guillaume GARDET,

On Tue, 18 Nov 2014 15:54:47 +0100, Guillaume GARDET wrote:
> Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>
> 
> ---
>  package/Config.in                        |  1 +
>  package/c-icap-modules/Config.in         |  7 +++++++
>  package/c-icap-modules/c-icap-modules.mk | 27 +++++++++++++++++++++++++++
>  3 files changed, 35 insertions(+)
>  create mode 100644 package/c-icap-modules/Config.in
>  create mode 100644 package/c-icap-modules/c-icap-modules.mk

I applied your patch, but did quite a few changes before doing so:

    [Thomas:
      - Do not pass custom CFLAGS and remove the hacks on the Makefile:
        since the c-icap-config script is now fixed, it returns correct
        values, and the hacks are no longer needed.
      - Add optional dependency on zlib and berkeleydb.
      - Explicitly disable clamav support.]

Thanks for your contribution!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH V3 4/5] c-icap: new package

[Guillaume GARDET - Oliséo]

Le 10/07/2015 19:09, Thomas Petazzoni a ?crit :
> Dear Guillaume GARDET,
>
> On Tue, 18 Nov 2014 15:54:46 +0100, Guillaume GARDET wrote:
>> Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>
>>
>> ---
>>   package/Config.in        |  1 +
>>   package/c-icap/Config.in | 15 ++++++++++++
>>   package/c-icap/S96cicap  | 35 ++++++++++++++++++++++++++
>>   package/c-icap/c-icap.mk | 64 ++++++++++++++++++++++++++++++++++++++++++++++++
>>   4 files changed, 115 insertions(+)
>>   create mode 100644 package/c-icap/Config.in
>>   create mode 100644 package/c-icap/S96cicap
>>   create mode 100644 package/c-icap/c-icap.mk
> I've applied, but a lot of changes were needed to make the patch
> correct. In fact, with the patch you posted, the c-icap daemon was not
> even starting properly at boot time because the c-icap.conf file
> contained invalid paths.

Thanks.


Guillaume

>
> Here are the changes I made before committing:
>
>      [Thomas:
>       - Remove trailing whitespace everywhere
>       - Rewrap the Config.in help text
>       - Fix Config.in comment, the proper text for BR2_PREFER_STATIC_LIB is
>         "dynamic library", not "libdl library"
>       - Use a PID file in the init script
>       - Show OK / FAIL when stopping the service
>       - Remove the largefile/IPv6 handling in c-icap.mk, since they are now
>         mandatory. We therefore pass --enable-large-files and --enable-ipv6
>         unconditionally.
>       - Remove the reference to a non-existing
>         C_ICAP_INSTALL_TARGET_CONFIGS install target hook.
>       - Install the init script unconditionally.
>       - Remove unneeded example configuration files (*.default) from /etc,
>         and fixup the c-icap.conf so that the default values work with
>         Buildroot
>       - Properly handle the *-config scripts installed by c-icap.
>       - Add hash file.]
>
> Thanks!
>
> Thomas

[Buildroot] [PATCH V3 5/5] c-icap-modules: new package

[Guillaume GARDET - Oliséo]

Le 13/07/2015 00:26, Thomas Petazzoni a ?crit :
> Dear Guillaume GARDET,
>
> On Tue, 18 Nov 2014 15:54:47 +0100, Guillaume GARDET wrote:
>> Signed-off-by: Guillaume GARDET <guillaume.gardet@oliseo.fr>
>>
>> ---
>>   package/Config.in                        |  1 +
>>   package/c-icap-modules/Config.in         |  7 +++++++
>>   package/c-icap-modules/c-icap-modules.mk | 27 +++++++++++++++++++++++++++
>>   3 files changed, 35 insertions(+)
>>   create mode 100644 package/c-icap-modules/Config.in
>>   create mode 100644 package/c-icap-modules/c-icap-modules.mk
> I applied your patch, but did quite a few changes before doing so:
>
>      [Thomas:
>        - Do not pass custom CFLAGS and remove the hacks on the Makefile:
>          since the c-icap-config script is now fixed, it returns correct
>          values, and the hacks are no longer needed.
>        - Add optional dependency on zlib and berkeleydb.
>        - Explicitly disable clamav support.]
>
> Thanks for your contribution!

Thanks.


Guillaume

>
> Thomas