[Pki-devel] [PATCH] 0122 Modify ExternalProcessKeyRetriever to read JSON

Endi Sukma Dewata edewata at redhat.com
Sun Jun 5 18:12:09 UTC 2016


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




More information about the Pki-devel mailing list