From: ludo@gnu.org (Ludovic Courtès)
To: Leo Famulari <leo@famulari.name>
Cc: 28586@debbugs.gnu.org
Subject: [bug#28586] [PATCH 2/2] gnu: Add Syncthing.
Date: Tue, 26 Sep 2017 09:41:36 +0200 [thread overview]
Message-ID: <87d16d3o0f.fsf@gnu.org> (raw)
In-Reply-To: <20170925192852.GA12757@jasmine.lan> (Leo Famulari's message of "Mon, 25 Sep 2017 15:28:52 -0400")
Heya!
Leo Famulari <leo@famulari.name> skribis:
> On Mon, Sep 25, 2017 at 02:21:44PM -0400, Leo Famulari wrote:
>> This suggests to me that the Syncthing build process is not using the
>> compiled objects of these packages but is instead trying to rebuild
>> them.
>>
>> So, perhaps something is not quite right with the go-build-system — I'm
>> not sure.
>
> While debugging, I tried changing the go-build-system's build procedure
> to this:
>
> ------
> (define* (build #:key import-path #:allow-other-keys)
> "Build the package named by IMPORT-PATH."
> (unless (zero? (system* "go" "install" "-v" import-path))
> (zero? (system* "go" "env"))))
> ------
Note that it should always return a Boolean:
(or (zero? (system* "go" "install" …))
(begin ;we failed but we want to print debugging info
(system* "go" "env")
#f))
> However, when it encounters a failure, it does not seem to run `go env`:
>
> ------
> [...]
> /gnu/store/40m4imp31qkhl2yqvxm4dfaw0j6hgfr2-golang-github-com-golang-groupcache-lru-0.0.0-0.72d04f9/src/github.com/pkg/errors
> /gnu/store/dn3hbj11bzk6ys3yj5k36k7a9fyg1zp8-golang-github-com-edsrzf-mmap-go-0.0.0-0.0bce6a6/src/github.com/pkg/errors
> /gnu/store/h36806f660r6p5xzwsc89zrbfnvi6nwi-golang-github-com-d4l3k-messagediff-1.1.0-0.29f32d8/src/github.com/pkg/errors
> exit status 1
> exit status 1
> phase `build' failed after 1.9 seconds
I think it just means that “go env” returned with exit code 1, no?
Ludo’.
PS: Great to see ‘go-build-system’ coming!
next prev parent reply other threads:[~2017-09-26 7:42 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-24 20:36 [bug#28586] go-build-system and Syncthing Leo Famulari
2017-09-24 20:40 ` [bug#28586] [PATCH 1/2] build: Add the Go build system Leo Famulari
2017-09-24 20:40 ` [bug#28586] [PATCH 2/2] gnu: Add Syncthing Leo Famulari
2017-09-25 18:21 ` Leo Famulari
2017-09-25 19:28 ` Leo Famulari
2017-09-26 7:41 ` Ludovic Courtès [this message]
2017-09-29 19:24 ` Leo Famulari
2017-10-02 20:23 ` [bug#28586] go-build-system and Syncthing Leo Famulari
2017-10-02 21:01 ` ng0
2017-10-02 21:42 ` Leo Famulari
2017-10-13 1:47 ` bug#28586: " Leo Famulari
2017-10-13 8:39 ` [bug#28586] " Ludovic Courtès
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87d16d3o0f.fsf@gnu.org \
--to=ludo@gnu.org \
--cc=28586@debbugs.gnu.org \
--cc=leo@famulari.name \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).