On Wed, 19 Nov 2014, Simo Sorce wrote:
----- Original Message -----
From: "Alexander Bokovoy" <aboko...@redhat.com>
[...]
Regarding the patchset itself:
Patch 0001: fix 'wuld' in the commit message. The rest is fine.
Fixed.
Patch 0002:
- ticket number is missing in the commit message
Added.
- perhaps, an instruction how to regenerate asn1 code can be made a
Makefile target? We don't need to call it ourselves but this would
simplify things in future
Added make regenerate target to asn1c makefile
- I'm little uncomfortable how ASN_DEBUG() output goes explicitly to
stderr but I guess this is something we currently cannot override
with DS-specific log printing, so no big deal right now
ASN_DEBUG() is currently disabled as EMIT_ASN_DEBUG is undefined, we can
later provide a replacement ASN_DEBUG function to hook debugging, but
given the same code is used in both DS plugins and ipa-getkeytab binary
I did not want to assume anything, and how to wire it up (if we even want
to) should probably be discussed at a later time.
- any specific need to get asn1/compile committed? We don't commit it
in the client code (ipa-client/compile).
Added 'compile' to .gitignore in second patch
Patch 0003: OK
Nothing changed here.
I also remembered the patch naming policy :-) so new patch names/numbers
are 514,515,516, third revision.
Thanks. The only complaint I have left is number of whitespace errors that git
says are in the 515th patch.
Otherwise, ACK. I've tested it again and everything works except getting
stronger than asked TGT enctype but this is not an issue with getkeytab
controls.
--
/ Alexander Bokovoy
_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
