[SSSD] [PATCHES] Turn the ConfDB into its own package
Stephen Gallagher
sgallagh at redhat.com
Fri Feb 12 13:08:56 UTC 2010
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 02/12/2010 08:04 AM, Sumit Bose wrote:
> On Tue, Feb 09, 2010 at 03:46:10PM -0500, Stephen Gallagher wrote:
> On 02/08/2010 03:34 AM, Sumit Bose wrote:
>>>> On Fri, Feb 05, 2010 at 12:25:05PM -0500, Stephen Gallagher wrote:
>>>> These three patches are the first steps towards an sssd-devel package.
>>>>
>>>> Patch 0001: Build the confdb as a shared library. This is so that
>>>> third-party backend implementers can access our configuration if they
>>>> choose.
>>>>
>>>> Patch 0002: Add comprehensive doxygen docs for the confdb
>>>>
>>>> Patch 0003: Add a -devel subpackage to the sssd.spec for the confdb and
>>>> its docs.
>>>>
>>>>> I think patch 2 and 3 are still useful and should be include although I
>>>>> would NACK them because of a missing 'BuildRequires: doxygen'.
>
> I think only patch 2 makes sense at all, since if the confdb is not
> going to be a public interface, we probably don't need to package its
> documentation.
>
> I've attached an updated version of patch 0002 that will not build the
> documentation automatically; only when 'make docs' is run.
>
>
>> I have two comments:
>
>> +docs: all
>> + $(MAKE) -C server $@
>
>> why do I have 'make all' before I can create the docs?
>
>> It would be nice to get a message about missing doxygen instead of
>> 'true doxy.config' if doxygen is not installed.
>
Ah, these are leftovers from when I was building the docs as part of the
installable data (for the RPM).
We don't need the dependency on 'all', and I can change the configure
test for doxygen. Replacing $(DOXYGEN) with 'true' was there to ensure
that 'make install' didn't fail trying to build the docs.
I will prepare a new patch.
- --
Stephen Gallagher
RHCE 804006346421761
Delivering value year after year.
Red Hat ranks #1 in value among software vendors.
http://www.redhat.com/promo/vendor/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
iEYEARECAAYFAkt1UugACgkQeiVVYja6o6NJ1gCgk7Ez2Ebb39aogBFEv0dSpFsN
l3wAn2GyQVdiVV9oAZxi+o8ucuHY4gGQ
=wSpx
-----END PGP SIGNATURE-----
More information about the sssd-devel
mailing list