[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 at 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