[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