* selinux version to link against libsepol, too
@ 2008-12-16 9:54 Harald Hoyer
2008-12-16 14:00 ` Kay Sievers
` (4 more replies)
0 siblings, 5 replies; 6+ messages in thread
From: Harald Hoyer @ 2008-12-16 9:54 UTC (permalink / raw)
To: linux-hotplug
[-- Attachment #1: Type: text/plain, Size: 0 bytes --]
[-- Attachment #2: udev-135-sepol.patch --]
[-- Type: text/plain, Size: 767 bytes --]
commit 9aa4548cb8720f772ca5e3648d689fbd931dffe1
Author: Harald Hoyer <harald@redhat.com>
Date: Tue Dec 16 10:48:21 2008 +0100
LDFLAGS="-all-static" and configure --withselinux needs to link against libsepol, too
diff --git a/configure.ac b/configure.ac
index 4439166..cd4ccc5 100644
--- a/configure.ac
+++ b/configure.ac
@@ -55,6 +55,13 @@ if test "x$with_selinux" = xyes; then
AC_DEFINE(USE_SELINUX, [1] ,[compile with SELinux support])
SELINUX_LIBS="-lselinux"
fi
+if test "x$with_selinux" = xyes; then
+ LIBS_save=$LIBS
+ AC_CHECK_LIB(sepol, sepol_genbools,
+ [SELINUX_LIBS="$SELINUX_LIBS -lsepol"],
+ [])
+ LIBS=$LIBS_save
+fi
AC_SUBST([SELINUX_LIBS])
AM_CONDITIONAL(USE_SELINUX, [test "x$with_selinux" = xyes], [compile with SELinux support])
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: selinux version to link against libsepol, too
2008-12-16 9:54 selinux version to link against libsepol, too Harald Hoyer
@ 2008-12-16 14:00 ` Kay Sievers
2008-12-16 14:13 ` Harald Hoyer
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Kay Sievers @ 2008-12-16 14:00 UTC (permalink / raw)
To: linux-hotplug
On Tue, Dec 16, 2008 at 10:54, Harald Hoyer <harald@redhat.com> wrote:
>
> commit 9aa4548cb8720f772ca5e3648d689fbd931dffe1
> Author: Harald Hoyer <harald@redhat.com>
> Date: Tue Dec 16 10:48:21 2008 +0100
>
> LDFLAGS="-all-static" and configure --withselinux needs to link against libsepol, too
Why do we need to check for sepol to be available, we can just add it
unconditionally, right?
What are you doing with -all-static? You still put static crap in your
initrd? :)
Thanks,
Kay
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: selinux version to link against libsepol, too
2008-12-16 9:54 selinux version to link against libsepol, too Harald Hoyer
2008-12-16 14:00 ` Kay Sievers
@ 2008-12-16 14:13 ` Harald Hoyer
2008-12-16 15:04 ` Kay Sievers
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Harald Hoyer @ 2008-12-16 14:13 UTC (permalink / raw)
To: linux-hotplug
Kay Sievers wrote:
> On Tue, Dec 16, 2008 at 10:54, Harald Hoyer <harald@redhat.com> wrote:
>> commit 9aa4548cb8720f772ca5e3648d689fbd931dffe1
>> Author: Harald Hoyer <harald@redhat.com>
>> Date: Tue Dec 16 10:48:21 2008 +0100
>>
>> LDFLAGS="-all-static" and configure --withselinux needs to link against libsepol, too
>
> Why do we need to check for sepol to be available, we can just add it
> unconditionally, right?
hmm, maybe :)
>
> What are you doing with -all-static? You still put static crap in your
> initrd? :)
We never used static crap in our initrd (from udev), but I provide udev-static
as an rpm for those who might need it. :-)
>
> Thanks,
> Kay
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: selinux version to link against libsepol, too
2008-12-16 9:54 selinux version to link against libsepol, too Harald Hoyer
2008-12-16 14:00 ` Kay Sievers
2008-12-16 14:13 ` Harald Hoyer
@ 2008-12-16 15:04 ` Kay Sievers
2008-12-16 15:23 ` Kay Sievers
2008-12-18 10:30 ` Harald Hoyer
4 siblings, 0 replies; 6+ messages in thread
From: Kay Sievers @ 2008-12-16 15:04 UTC (permalink / raw)
To: linux-hotplug
On Tue, Dec 16, 2008 at 15:13, Harald Hoyer <harald@redhat.com> wrote:
> Kay Sievers wrote:
>>
>> On Tue, Dec 16, 2008 at 10:54, Harald Hoyer <harald@redhat.com> wrote:
>>>
>>> commit 9aa4548cb8720f772ca5e3648d689fbd931dffe1
>>> Author: Harald Hoyer <harald@redhat.com>
>>> Date: Tue Dec 16 10:48:21 2008 +0100
>>>
>>> LDFLAGS="-all-static" and configure --withselinux needs to link against
>>> libsepol, too
>>
>> Why do we need to check for sepol to be available, we can just add it
>> unconditionally, right?
>
> hmm, maybe :)
That's how it was before we switched to autotools. The -as-needed
should take care of stripping it again.
>> What are you doing with -all-static? You still put static crap in your
>> initrd? :)
>
> We never used static crap in our initrd (from udev), but I provide
> udev-static as an rpm for those who might need it. :-)
Yeah, it's christmas time, and we want to make the hopelessly clueless
happy. I understand. :)
Kay
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: selinux version to link against libsepol, too
2008-12-16 9:54 selinux version to link against libsepol, too Harald Hoyer
` (2 preceding siblings ...)
2008-12-16 15:04 ` Kay Sievers
@ 2008-12-16 15:23 ` Kay Sievers
2008-12-18 10:30 ` Harald Hoyer
4 siblings, 0 replies; 6+ messages in thread
From: Kay Sievers @ 2008-12-16 15:23 UTC (permalink / raw)
To: linux-hotplug
On Tue, Dec 16, 2008 at 16:04, Kay Sievers <kay.sievers@vrfy.org> wrote:
> On Tue, Dec 16, 2008 at 15:13, Harald Hoyer <harald@redhat.com> wrote:
>> Kay Sievers wrote:
>>>
>>> On Tue, Dec 16, 2008 at 10:54, Harald Hoyer <harald@redhat.com> wrote:
>>>>
>>>> commit 9aa4548cb8720f772ca5e3648d689fbd931dffe1
>>>> Author: Harald Hoyer <harald@redhat.com>
>>>> Date: Tue Dec 16 10:48:21 2008 +0100
>>>>
>>>> LDFLAGS="-all-static" and configure --withselinux needs to link against
>>>> libsepol, too
>>>
>>> Why do we need to check for sepol to be available, we can just add it
>>> unconditionally, right?
>>
>> hmm, maybe :)
>
> That's how it was before we switched to autotools. The -as-needed
> should take care of stripping it again.
Added it.
Btw, why does libselinux not ship with a pkgconfig file? Do you have
that on your system?
That would make all that work without these ugly workarounds.
Thanks,
Kay
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: selinux version to link against libsepol, too
2008-12-16 9:54 selinux version to link against libsepol, too Harald Hoyer
` (3 preceding siblings ...)
2008-12-16 15:23 ` Kay Sievers
@ 2008-12-18 10:30 ` Harald Hoyer
4 siblings, 0 replies; 6+ messages in thread
From: Harald Hoyer @ 2008-12-18 10:30 UTC (permalink / raw)
To: linux-hotplug
Kay Sievers wrote:
> On Tue, Dec 16, 2008 at 16:04, Kay Sievers <kay.sievers@vrfy.org> wrote:
>> On Tue, Dec 16, 2008 at 15:13, Harald Hoyer <harald@redhat.com> wrote:
>>> Kay Sievers wrote:
>>>> On Tue, Dec 16, 2008 at 10:54, Harald Hoyer <harald@redhat.com> wrote:
>>>>> commit 9aa4548cb8720f772ca5e3648d689fbd931dffe1
>>>>> Author: Harald Hoyer <harald@redhat.com>
>>>>> Date: Tue Dec 16 10:48:21 2008 +0100
>>>>>
>>>>> LDFLAGS="-all-static" and configure --withselinux needs to link against
>>>>> libsepol, too
>>>> Why do we need to check for sepol to be available, we can just add it
>>>> unconditionally, right?
>>> hmm, maybe :)
>> That's how it was before we switched to autotools. The -as-needed
>> should take care of stripping it again.
>
> Added it.
>
> Btw, why does libselinux not ship with a pkgconfig file? Do you have
> that on your system?
Don't know. No, no pkgconfig file.
>
> That would make all that work without these ugly workarounds.
Well, well.. ;-)
>
> Thanks,
> Kay
> --
> To unsubscribe from this list: send the line "unsubscribe linux-hotplug" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2008-12-18 10:30 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-12-16 9:54 selinux version to link against libsepol, too Harald Hoyer
2008-12-16 14:00 ` Kay Sievers
2008-12-16 14:13 ` Harald Hoyer
2008-12-16 15:04 ` Kay Sievers
2008-12-16 15:23 ` Kay Sievers
2008-12-18 10:30 ` Harald Hoyer
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.