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

["Guillaume GARDET - Oliséo" <guillaume.gardet@oliseo.fr>]

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