[SSSD] [INI] Patches for ding-libs: Merging config sections, handling metadata, remaining Coverity issues...

Stephen Gallagher sgallagh at redhat.com
Thu Jan 6 11:49:32 UTC 2011


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 01/06/2011 03:02 AM, Sumit Bose wrote:
> On Wed, Jan 05, 2011 at 04:03:38PM -0500, Dmitri Pal wrote:
>> Stephen Gallagher wrote:
>>> On 01/05/2011 03:40 PM, Dmitri Pal wrote:
>>>>>>>> 0003--BUILD-Allow-trace-per-component.patch <- This patch allows
>>>>> tracing
>>>>>>>> per component
>>>>>>> Nack. Requiring a file doesn't make sense. Just add
>>> --enable-trace-ini,
>>>>>>> --enable-trace-collection etc. flags.
>>>>>> This is exactly what I did not like to do and tried to avoid. It
>>> is more
>>>>>> code on the configure script, more checks and less convenient to
>>> turn on
>>>>>> and off becuase you can run the same configure command with one switch
>>>>>> enabled from history and do "touch trace" & "rm trace" as needed
>>> in the
>>>>>> directory you need.
>>>>>> Saves a lot of typing for me :-)
>>>>>> If you do not like the patch I will have to keep it as private
>>> patch as
>>>>>> it is more convenient for me during development than the flags.
>>>>>
>>>>> I still vote nack, as I don't like the approach.
>>>> So this is the dead lock becuase I like the approach and do not like
>>>> yours. How we are going to resolve it?
>>>
>>>
>>> Ultimately, it's your project, not mine. You have commit privilege and
>>> full discretion on when to use it. You asked for my review comments. You
>>> don't have to listen to them :)
>>
>> Nack. I think we need to come to some terms and find consensus. I will
>> definitely not push it over your head using my privilege.
>> I agree that what you suggest is logical and consistent with how things
>> are done. The problem is that it does not help me much.
>> I will keep patch in my tree for now.
> 
> To enlarge the quorum I throw in my opinion here. In general I agree
> with Stephen because the file approach is quite unusual and needs to be
> well documented so that other developers can use it. You might also want
> to use $builddir instead of $srcdir to allow a separate build directory.
> 
> But I would like to suggest another approach which might be even easier
> to use. You can introduce a make target like 'trace' or 'devel' which
> will build everything with tracing enabled. This way you do not even
> have to rerun configure to enable tracing.

That's a good idea. The trace level could then be set by environment
variable as well.

e.g.
./configure
COL_TRACE=3 make devel

And this would tell it to pass -DTRACE_LEVEL=3 to the COLLECTION_CFLAGS

- -- 
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.11 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/

iEYEARECAAYFAk0lrEgACgkQeiVVYja6o6NR6QCgrFxeWYXwrD2+rSrTyatiGnBu
tUsAoIbyturzf1tsuyZtnphZwrhANv2Z
=unKr
-----END PGP SIGNATURE-----



More information about the sssd-devel mailing list