GNU bug report logs - #53048
28.0.90; [PATCH] GNU ELPA local build fails silently on macOS

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Stefan Kangas <stefan <at> marxist.se>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#53048: closed (28.0.90; [PATCH] GNU ELPA local build fails
 silently on macOS)
Date: Sun, 09 Jan 2022 13:26:02 +0000

[Message part 1 (text/plain, inline)]

Your message dated Sun, 9 Jan 2022 07:25:22 -0600
with message-id <CADwFkmm2AqThQQnrfL7fYUoyVHzArMpm1ezfA=ojELjN-HOmCg <at> mail.gmail.com>
and subject line Re: bug#53048: 28.0.90; [PATCH] GNU ELPA local build fails silently on macOS
has caused the debbugs.gnu.org bug report #53048,
regarding 28.0.90; [PATCH] GNU ELPA local build fails silently on macOS
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
53048: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=53048
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Y. E. <yet <at> ego.team>
To: bug-gnu-emacs <at> gnu.org
Subject: 28.0.90; [PATCH] GNU ELPA local build fails silently on macOS
Date: Thu, 06 Jan 2022 12:35:41 +0200

[Message part 3 (text/plain, inline)]

On macOS, with the default 'bsdtar', 'make build/<pkg>' fails to create
a tarball, but does not warn about it, outputting:

> ======== Building tarball archive-devel/<pkg-vers>.tar...
> ######## Built new package archive-devel/<pkg-vers>.tar!
> ======== Building tarball archive/<pkg-ver>.tar...
> ######## Built new package archive/<pkg-ver>.tar!

['bsdtar' does not expose '--transform' option. Though it provides a
similar '-s' option with a bit different syntax.]

This patch suggests asserting a tarball was generated. This would
result in a more helpful output on a failed generation:

> ======== Building tarball archive-devel/<pkg-vers>.tar...
> Build error for archive-devel/<pkg-vers>.tar: (cl-assertion-failed (file-readable-p tarball))
> ######## Build of package archive-devel/<pkg-vers>.tar FAILED!!
> ======== Building tarball archive/<pkg-ver>.tar...
> Build error for archive/<pkg-ver>.tar: (cl-assertion-failed (file-readable-p tarball))
> ######## Build of package archive/<pkg-ver>.tar FAILED!!



In GNU Emacs 28.0.90 (build 1, x86_64-apple-darwin21.1.0, NS appkit-2113.00 Version 12.0.1 (Build 21A559))

[0001-admin-elpa-admin.el-elpaa-make-one-tarball-1-Assert-.patch (text/x-patch, attachment)]

[Message part 5 (message/rfc822, inline)]

From: Stefan Kangas <stefan <at> marxist.se>
To: "Y. E." <yet <at> ego.team>
Cc: 53048-done <at> debbugs.gnu.org
Subject: Re: bug#53048: 28.0.90; [PATCH] GNU ELPA local build fails silently
 on macOS
Date: Sun, 9 Jan 2022 07:25:22 -0600

Y. E. <yet <at> ego.team> writes:

> From b8ce44d9f3da636a9236d7fc6a29dffb73d819ec Mon Sep 17 00:00:00 2001
> From: YugaEgo <yet <at> ego.team>
> Date: Thu, 6 Jan 2022 11:55:27 +0200
> Subject: [PATCH] * admin/elpa-admin.el (elpaa--make-one-tarball-1): Assert
>  tarball exists
>
> ---
>  elpa-admin.el | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/elpa-admin.el b/elpa-admin.el
> index b4ad318ef6..d94cc7fd73 100644
> --- a/elpa-admin.el
> +++ b/elpa-admin.el
> @@ -612,6 +612,7 @@ auxillary files unless TARBALL-ONLY is non-nil ."
>                ,(format "s|^packages/%s|%s-%s|" pkgname pkgname vers)
>                "-chf" ,tarball
>                ,(concat "packages/" pkgname)))
> +     (cl-assert (file-readable-p tarball))
>       (unless tarball-only
>         (let* ((pkgdesc
>                 ;; FIXME: `elpaa--write-pkg-file' wrote the metadata to

Makes sense, pushed as commit 102694997e.

Thanks for the patch!

Previous Next

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