Hi Ludovic,

Thank you for getting back to me.

> Hi Collin,
>
> "Collin J. Doering" <collin@rekahsoft.ca> skribis:
>
>> aws-vault is a tool to securely store and use aws credentials. This patch set packages aws-vault as well as missing dependencies.
>>
>> Note: perhaps gnu/packages/golang.scm is not the best fit for these changes. Any suggestions for a better spot appreciated.
>
> Hmm not sure; most of the dependencies are at home in golang.scm IMO.
> I’d leave it as is.
>
>> Collin J. Doering (12):
>>   gnu: Add go-keyring
>>   gnu: Add go-androidnsfix
>>   gnu: Add go-percent
>>   gnu: Add go-aws-sdk-go
>>   gnu: Add go-jose2go
>>   gnu: Add go-libsecret
>>   gnu: Add go-ini
>>   gnu: Add go-open-golang
>>   gnu: Add go-alecthomas-template
>>   gnu: Add go-alecthomas-units
>>   gnu: Add go-kingpin
>>   gnu: Add aws-vault
>
> I tried to apply those (overall it LGTM), but the order is incorrect:
> for instance ‘go-keyring’ depends on ‘go-percent’ and ‘go-libsecret’.
>
> So I have the following queries:
>
>   1. Do you think you could send them in the “right order”?
>

Done (see new patch set attached to this email).

>   2. As you go, could you pass them to:
>
>       ./pre-inst-env guix style PKG
>
>      ?
>

I actually had issues doing this. The above command would segfault. After running the `clean` and `clean-go` make targets, it would no longer segfault, but would then complain about ABI compatibility. I didn't debug further instead ran the `etc/indent-code.el` source file, commiting as necessary. However, I did find that after rebasing my changes onto the tip of master, the `guix style` command started to work as expected.

Another note: I use emacs and did load the `.dir-locals.el` file when making these changes, so I would have expected formatting to be fine already. Must be something on my end that is affecting auto-formatting in emacs.

>   3. Last, it would be perfect if you could provide the conventional
>      commit logs.
>

Done. Please let me know if the new logs provided match the expected conventions.

> Thanks in advance, and sorry for the delay!
>
> Ludo’.

Thank you again for your consideration and time reviewing this patchset.

Kind regards,