unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
From: Vagrant Cascadian <vagrant@debian.org>
To: Alan & Kim Zimmerman <alan.zimm@gmail.com>
Cc: 53423@debbugs.gnu.org,
	Katherine Cox-Buday <cox.katherine.e+guix@gmail.com>,
	Sharlatan Hellseher <sharlatanus@gmail.com>,
	Ivan Vilata i Balaguer <ivan@selidor.net>
Subject: bug#53423: [fix] nncp: Fails to build (renamed file not found)
Date: Wed, 07 Feb 2024 19:20:54 -0800	[thread overview]
Message-ID: <878r3vprop.fsf@wireframe> (raw)
In-Reply-To: <CANma=H8K+5x2y7+z4d72rueZdCapNQ+kUpiXOHbjtooVB5c5kg@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 2594 bytes --]

On 2023-06-24, Alan & Kim Zimmerman wrote:
> I took a look at this, and the problem seems to be that the cwd ends up
> different from before, so all the file operations fail.
>
> It needs (chdir "../nncp-7.5.0") in the 'go-unpack section.
>
> Attached is a patch that does this, if it works via gmail.

Thanks for the patch! Miraculously, it still applies after all this
time, and it does allow the build to proceed further, but still fails in
tests:

  starting phase `check'
  do  test
  # _/tmp/guix-build-nncp-7.5.0.drv-0/nncp-7.5.0/src/cmd/nncp-cfgdir
  cmd/nncp-cfgdir/main.go:91:4: unknown field 'AllowMinusZero' in struct literal of type hjson.EncoderOptions
  ok      _/tmp/guix-build-nncp-7.5.0.drv-0/nncp-7.5.0/src        37.407s                                                             ?       _/tmp/guix-build-nncp-7.5.0.drv-0/nncp-7.5.0/src/cmd/nncp-bundle        [no test files]
  ?       _/tmp/guix-build-nncp-7.5.0.drv-0/nncp-7.5.0/src/cmd/nncp-call  [no test files]
  ?       _/tmp/guix-build-nncp-7.5.0.drv-0/nncp-7.5.0/src/cmd/nncp-caller        [no test files]
  do: test: got exit code 2
  error: in phase 'check': uncaught exception:
  %exception #<&invoke-error program: "contrib/do" arguments: ("-c" "test") exit-status: 1 term-signal: #f stop-signal: #f>
  phase `check' failed after 44.5 seconds
  command "contrib/do" "-c" "test" failed with status 1

CCed the members of the go team who may have a better idea of, well,
packaging go programs. :)

live well,
  vagrant

> From f2cc08e9cd657717049936938077a210773ab193 Mon Sep 17 00:00:00 2001
> Message-Id: <f2cc08e9cd657717049936938077a210773ab193.1687601540.git.alan.zimm@gmail.com>
> From: Alan Zimmerman <alan.zimm@gmail.com>
> Date: Fri, 23 Jun 2023 23:57:48 +0100
> Subject: [PATCH] nncp: set directory so build succeeds
>
> ---
>  gnu/packages/uucp.scm | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/gnu/packages/uucp.scm b/gnu/packages/uucp.scm
> index e10de59aa2..65e71c1b1a 100644
> --- a/gnu/packages/uucp.scm
> +++ b/gnu/packages/uucp.scm
> @@ -98,6 +98,7 @@ (define-public nncp
>             (assoc-ref go:%standard-phases 'setup-go-environment))
>           (add-after 'unpack 'go-unpack
>             (lambda* (#:key source #:allow-other-keys)
> +             (chdir "../nncp-7.5.0")
>               ;; Copy source to GOPATH.
>               (copy-recursively "src" "../src/go.cypherpunks.ru/nncp/v7")
>               ;; Move bundled dependencies to GOPATH.
>
> base-commit: f25529b08e356f89ca7cecc44295085531a8faba
> -- 
> 2.40.1

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

  reply	other threads:[~2024-02-08  3:22 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-21 20:58 bug#53423: nncp: Fails to build (renamed file not found) Ivan Vilata i Balaguer
2023-06-24 11:04 ` bug#53423: [fix] " Alan & Kim Zimmerman
2024-02-08  3:20   ` Vagrant Cascadian [this message]
2024-02-08  3:36     ` Vagrant Cascadian
2024-02-08 11:37 ` bug#53423: " Sharlatan Hellseher

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=878r3vprop.fsf@wireframe \
    --to=vagrant@debian.org \
    --cc=53423@debbugs.gnu.org \
    --cc=alan.zimm@gmail.com \
    --cc=cox.katherine.e+guix@gmail.com \
    --cc=ivan@selidor.net \
    --cc=sharlatanus@gmail.com \
    /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).