GNU bug report logs - #37444
[PATCH] added aerc, a simple email client and its dependcies

Previous Next

Package: guix-patches;

Reported by: Martin Becze <mjbecze <at> riseup.net>

Date: Wed, 18 Sep 2019 08:25:01 UTC

Severity: normal

Tags: patch

Done: "(" <paren <at> disroot.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Martin Becze <mjbecze <at> riseup.net>
To: Ricardo Wurmus <rekado <at> elephly.net>
Cc: 37444 <at> debbugs.gnu.org
Subject: [bug#37444] [PATCH] added aerc, a simple email client and its dependcies
Date: Wed, 18 Sep 2019 02:20:26 -0700
Certainly! I'll get that up later today. On the propagated-inputs issue.
So I might be wrong here. But since all the go libraries are source only
it doesn't matter to the end result if the are propagated or not if the
libraries are used as `native-inputs`.. they shouldn't be left lying
around (I think?). Now the reason why the need to be propagated is that
when the go compiler go to compile the top level app (aerc) it needs to
have in it's GOPATH all of the dependencies and the dependencies of the
dependencies. The other way to do it is to put all of dependencies and
the dependencies of the dependencies in the native-input. But that
didn't seem right to me. Let me know what you think!

On 2019-09-18 09:01, Ricardo Wurmus wrote:
> Hi Martin,
> 
>> This adds aerc, which is a terminal based email client written in go.
> 
> Thank you for your patch!
> 
> Could you please separate the independent changes into separate commits
> and add commit summaries?  For example, the update of
> go-github-com-mitchellh-go-homedir should be done separately.
> 
> Please also explain why “inputs” had to be changed to
> “propagated-inputs” in some packages, and mention these kind of changes
> in the commit summary.
> 
> Thanks!




This bug report was last modified 2 years and 347 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.