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

William Hubbs <[email protected]>
Newsgroups gmane.linux.hotplug.devel
Message-ID <20120606225342.GA12809@linux1>
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
signature.asc (application/pgp-signature, 198 B)
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.17 (GNU/Linux)

iEYEARECAAYFAk/P33YACgkQblQW9DDEZTiS+QCeIhsp+GDGg+G4jwktRcoc6qoo
xN0AmgKZfvmvDH6jt7OVSG942nVJ3Dip
=3hSI
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.