Re: [PATCH] Allow disabling the build of all of systemd, leaving just udev

[William Hubbs <w.d.hubbs@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1919 bytes --]

On Wed, Jun 06, 2012 at 01:51:05PM -0500, Bruce Dubbs wrote:
> William Hubbs wrote:
> 
> With this type of setup, ENABLE_SYSTEMD must imply ENABLE_TOOLS and 
> ENABLE_UDEV.

What I want is for all of the enable_* options to be true by default so
that we don't affect the result of the build unless a packager requests
it by disabling portions.

My goal for the tools option is to find the things in the distribution
that can be useful whether or not your distro is using systemd and set
up so you can build those separately from systemd and udev if you aren't
quite ready to make the jump to systemd.

> The first thing that will have to be done is to break up 
> libsystemd_shared_la_SOURCES in Makefile.am.  That definition combines 
> files incompatible with udev-only with files required by udev.  udev needs:
> 
> log
> label
> mkdir
> cgroup-util
> strv
> path-util
> conf-files
> hashmap
> set
> exit-status
> util
> dev-setup
> 
> but does not need:
> 
> cgroup-show
> conf-parser
> hwclock
> pager
> ratelimit
> socket-util
> spawn-ask-password-agent
> spawn-polkit-agent
> specifier
> utf8
> virt
> watchdog
> 
> Basically all of the above are combined into libsystemd-core.la and that 
> needs to be split into two.  I'd just split the first group into 
> libudev-core.la.

I'm confused here because you first refer to
libsystemd_shared_la_SOURCES then libsystemd_core.la. What about
libsystemd_shared.la? Also, there is already a libudev-core.la. I will
look at it and see what it includes.

> 
> To link, udev also needs the file in libsystemd_daemon_la_SOURCES 
> (sd-daumon) for udevd, but that probably could be removed with some 
> appropriate ifdefs in udevd.c.

I'''m not seeing where udev needs this off the top of my head. Actually
I just ran "git grep -i sd.daumon" on the tree and that turns up
nothing.


 Thanks,

 William


[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]