[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,
aws-vault-and-deps.patch
Description: Text Data
--
Collin J. Doering
http://rekahsoft.ca
http://blog.rekahsoft.ca
http://git.rekahsoft.ca