unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / Atom feed
* broken "make dist" on core-updates-frozen
@ 2021-11-15  0:31 Vagrant Cascadian
  2021-11-15  1:12 ` Vagrant Cascadian
  0 siblings, 1 reply; 3+ messages in thread
From: Vagrant Cascadian @ 2021-11-15  0:31 UTC (permalink / raw)
  To: guix-devel

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

Heads up for anyone working on merging core-updates-frozen into master!

There is a very long patch in core-updates-frozen for the gd package
which breaks "make dist" ... and about 10k packages are transitively
dependent on gd... so this seems (at least to me) really important to
fix before merging into master...

I've pushed a fix (simply using a shorter patch filename) to
core-updates-frozen-batched-changes:

  https://git.savannah.gnu.org/cgit/guix.git/commit/?h=core-updates-frozen-batched-changes&id=22477c20604d88f2867c64405f86e7306a333f32

I hope that was a correct place to push it; I'm a little unclear as to
the correct branches with all the core-updates* variants right
now... clarity on that would be helpful.


I'll also make a plea to create a "make dist" job on ci.guix.gnu.org
soon, to catch these things earlier, as issues like this could block
making a release!


live well,
  vagrant

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

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: broken "make dist" on core-updates-frozen
  2021-11-15  0:31 broken "make dist" on core-updates-frozen Vagrant Cascadian
@ 2021-11-15  1:12 ` Vagrant Cascadian
  2021-11-17 11:14   ` Ludovic Courtès
  0 siblings, 1 reply; 3+ messages in thread
From: Vagrant Cascadian @ 2021-11-15  1:12 UTC (permalink / raw)
  To: guix-devel

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

On 2021-11-14, Vagrant Cascadian wrote:
> There is a very long patch in core-updates-frozen for the gd package
> which breaks "make dist" ... and about 10k packages are transitively
> dependent on gd... so this seems (at least to me) really important to
> fix before merging into master...
>
> I've pushed a fix (simply using a shorter patch filename) to
> core-updates-frozen-batched-changes:
>
>   https://git.savannah.gnu.org/cgit/guix.git/commit/?h=core-updates-frozen-batched-changes&id=22477c20604d88f2867c64405f86e7306a333f32

And apparently created core-updates-frozen-batched-changes in the
process? ugh. sorry.

I've now pushed to core-updates-frozen:

  https://git.savannah.gnu.org/cgit/guix.git/commit/?h=core-updates-frozen&id=6cdf4e5bf230fdbe17e592c2ec74fb34dba70eb5


> I'll also make a plea to create a "make dist" job on ci.guix.gnu.org
> soon, to catch these things earlier, as issues like this could block
> making a release!

Still think this would be a great idea! :)


live well,
  vagrant

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

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: broken "make dist" on core-updates-frozen
  2021-11-15  1:12 ` Vagrant Cascadian
@ 2021-11-17 11:14   ` Ludovic Courtès
  0 siblings, 0 replies; 3+ messages in thread
From: Ludovic Courtès @ 2021-11-17 11:14 UTC (permalink / raw)
  To: Vagrant Cascadian; +Cc: guix-devel

Hi!

Vagrant Cascadian <vagrant@debian.org> skribis:

> I've now pushed to core-updates-frozen:
>
>   https://git.savannah.gnu.org/cgit/guix.git/commit/?h=core-updates-frozen&id=6cdf4e5bf230fdbe17e592c2ec74fb34dba70eb5

Thanks!

>> I'll also make a plea to create a "make dist" job on ci.guix.gnu.org
>> soon, to catch these things earlier, as issues like this could block
>> making a release!
>
> Still think this would be a great idea! :)

Yes.  Note also that ‘guix lint -c patch-file-names’ also reports these:

--8<---------------cut here---------------start------------->8---
$ LANGUAGE= guix lint -c patch-file-names |grep "too long"
gnu/packages/admin.scm:2672:2: debops@1.1.0: debops-constants-for-external-program-names.patch: file name is too long
gnu/packages/astronomy.scm:768:2: xplanet@1.3.1: xplanet-1.3.1-libdisplay_DisplayOutput.cpp.patch: file name is too long
gnu/packages/astronomy.scm:768:2: xplanet@1.3.1: xplanet-1.3.1-xpUtil-Add2017LeapSecond.cpp.patch: file name is too long
gnu/packages/geo.scm:311:2: libgeotiff@1.5.1: libgeotiff-adapt-test-script-for-proj-6.2.patch: file name is too long
gnu/packages/glib.scm:478:2: gobject-introspection@1.62.0: gobject-introspection-absolute-shlib-path.patch: file name is too long
checking go-github-com-google-gmail-oauth2-tools-go-sendgmail@0.0.0-0.e322915 [patch-file-names]..gnu/packages/java.scm:13120:2: java-apache-ivy@2.4.0: java-apache-ivy-port-to-latest-bouncycastle.patch: file name is too long
gnu/packages/java.scm:8926:2: java-tunnelvisionlabs-antlr4-runtime-annotations@4.7.4: java-tunnelvisionlabs-antlr-code-too-large.patch: file name is too long
gnu/packages/java.scm:9040:2: java-tunnelvisionlabs-antlr4@4.7.4: java-tunnelvisionlabs-antlr-code-too-large.patch: file name is too long
gnu/packages/java.scm:9032:2: java-tunnelvisionlabs-antlr4-runtime@4.7.4: java-tunnelvisionlabs-antlr-code-too-large.patch: file name is too long
gnu/packages/kde-frameworks.scm:3419:2: plasma-framework@5.70.1: plasma-framework-fix-KF5PlasmaMacros.cmake.patch: file name is too long
gnu/packages/llvm.scm:98:2: clang-runtime@3.9.1: clang-runtime-3.9-libsanitizer-mode-field.patch: file name is too long
gnu/packages/llvm.scm:98:2: clang-runtime@3.7.1: clang-runtime-3.8-libsanitizer-mode-field.patch: file name is too long
gnu/packages/llvm.scm:98:2: clang-runtime@3.8.1: clang-runtime-3.8-libsanitizer-mode-field.patch: file name is too long
gnu/packages/llvm.scm:851:4: clang-runtime@3.5.2: clang-runtime-3.5-libsanitizer-mode-field.patch: file name is too long
$ guix describe
Generacio 194   Nov 07 2021 23:40:30    (nuna)
  guix bd41e59
    repository URL: https://git.savannah.gnu.org/git/guix.git
    branch: master
    commit: bd41e590dd24e54797fb8b6854c244efd4d12df5
--8<---------------cut here---------------end--------------->8---

It’s conservative so maybe not all these are problematic for “make
dist” (yet), but they’re worth fixing.

Ludo’.


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-11-17 11:14 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-15  0:31 broken "make dist" on core-updates-frozen Vagrant Cascadian
2021-11-15  1:12 ` Vagrant Cascadian
2021-11-17 11:14   ` Ludovic Courtès

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).