Re: [Pki-devel] [PATCH] 0122 Modify ExternalProcessKeyRetriever to read JSON
On 6/4/2016 6:00 AM, Fraser Tweedale wrote:
Hi Endi et al,
Attached patch changes ExternalProcessKeyRetriever to read JSON data
(https://fedorahosted.org/pki/ticket/2351). Would be nice to get
this into 10.2.2 because it will simplify IPA custodia retrieval
helper.
I am using Jackson for JSON parsing. This is already an implicit
dependency, but should I also add it spec file as explicit
dependency?
Cheers,
Fraser
I think in general we should declare direct dependency on something that
we use explicitly because there's no guarantee the indirect dependency
won't change. This can be added separately later.
ACK. Pushed to master.
--
Endi S. Dewata