[systemd-devel] [PATCH] ldconfig: dont run it if ldconfig is not installed
Umut Tezduyar Lindskog
umut at tezduyar.com
Mon Aug 4 02:11:28 PDT 2014
On Mon, Aug 4, 2014 at 2:19 AM, Lennart Poettering
<lennart at poettering.net> wrote:
> On Wed, 30.07.14 09:02, Umut Tezduyar Lindskog (umut.tezduyar at axis.com) wrote:
>
>> ---
>> units/ldconfig.service | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/units/ldconfig.service b/units/ldconfig.service
>> index 43c145b..09a2b74 100644
>> --- a/units/ldconfig.service
>> +++ b/units/ldconfig.service
>> @@ -13,6 +13,7 @@ Conflicts=shutdown.target
>> After=systemd-readahead-collect.service systemd-readahead-replay.service systemd-remount-fs.service
>> Before=sysinit.target shutdown.target systemd-update-done.service
>> ConditionNeedsUpdate=/etc
>> +ConditionFileIsExecutable=/sbin/ldconfig
>>
>> [Service]
>> Type=oneshot
>
> Ummmmmhh... Do we really want this? I presume you are not using glibc,
> are you?
We are actually. glibc 2-15 but not x86 ISA.
>
> I mean, whether ldconfig is around or not is not really an admin
> decision, it's an OS vendor decision. Because of that we should probably
> make shiping the ldconfig service file optional, rather then always ship
> it but then conditionalize it...
>
> I'd prefer to keep conditions rather minimal in general...
I could also mask the service but it is good to deviate as little as
possible. Maybe a configure check to install the service or not?
Umut
>
> Lennart
>
> --
> Lennart Poettering, Red Hat
> _______________________________________________
> systemd-devel mailing list
> systemd-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/systemd-devel
More information about the systemd-devel
mailing list