GNU bug report logs -
#37444
[PATCH] added aerc, a simple email client and its dependcies
Previous Next
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
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.