guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#51989] [PATCH 00/12] Add aws-vault and missing dependent packages


From: Collin J. Doering
Subject: [bug#51989] [PATCH 00/12] Add aws-vault and missing dependent packages
Date: Mon, 27 Dec 2021 11:19:37 -0500
User-agent: mu4e 1.6.10; emacs 27.2

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,

Attachment: aws-vault-and-deps.patch
Description: Text Data

-- 
Collin J. Doering

http://rekahsoft.ca
http://blog.rekahsoft.ca
http://git.rekahsoft.ca

reply via email to

[Prev in Thread] Current Thread [Next in Thread]