On Sun, Jun 03, 2012 at 09:55:55AM -0700, Bryan Kadzban wrote: > William Hubbs wrote: > > this looks good to me, but there are still pieces being installed > > which should not be installed for a udev-only build afaik. > > > > You might want to look at removing those also. > > Which pieces? > > The only things that get installed in my case are the rules, udev.conf, > systemd-udevd, udevadm, the helper binaries, the manpages, libudev, the > udev and libudev pkg-config files, and stuff in /usr/share/doc. Though > I do use a lot of the existing ./configure knobs to turn other systemd > things off; maybe that's related. Maybe it is, because I get the following things that would also be installed: etc/bash_completion.d etc/systemd usr/include/systemd usr/lib/sysctl.d usr/lib/systemd/system-generators usr/lib/systemd/user usr/share/man/man1 usr/share/man/man3 usr/share/man/man5 usr/share/polkit-1 usr/share/polkit-1/actions usr/share/systemd > Unless you mean the directories that are getting created, as in the > previous message -- but I don't see a way to stop automake from doing > that. Well, other than setting them to nothing in the Makefile.am if > !ENABLE_SYSTEMD; duh, I should have done that. But that still seems a > bit cosmetic to me. Yes, I am referring to empty directories and I can see how you might see it as cosmetic, but would you consider not creating the directories? > (One thing I need to do today is fix this so that "make distdir" works > even with --disable-systemd. Should be able to pull the EXTRA_DIST > stuff out of the conditionals to do that. So I'll probably look into > setting these directories to nothing in that case as well.) Ok, that may do it. One more thing though, thinking about it, is that we will need the systemd-tmpfiles tool because udev is not going to support the /lib/udev/devices directory any longer for creating custom devices, so you might want to still build and install all of the parts of that for a standalone udev build if doing so doesn't bring in more dependencies. Thanks, William
Attachment:
pgpFEEuIXKlZw.pgp
Description: PGP signature
