all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Pier-Hugues Pellerin <ph@heykimo.com>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: guix-devel@gnu.org
Subject: Re: Updating from Go 1.17 to 1.18
Date: Sun, 10 Apr 2022 17:16:53 -0400	[thread overview]
Message-ID: <CA+RyfLkMQ=JgDrX-e0emQpqayP7GPVEOHdsUeAkUHXVYXRsE7g@mail.gmail.com> (raw)
In-Reply-To: <87k0bw4qh8.fsf@gnu.org>

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

Hello,

I think it makes sense, looking at the number of impacted packages and from
my experience working
in go, even if the contract is the same, sometimes it does break on minor.

One more thing, should I do it in two patches like this?

1. Add go 1.18 inheriting from 1.17, allowing people to use 1.18 sooner?
2. Move to the default Go to 1.18 with (define-public go go-1.17) and
create the testing branch from that?

Thanks for the `refresh` tips.


On Sun, Apr 10, 2022 at 4:45 PM Ludovic Courtès <ludo@gnu.org> wrote:

> Hi,
>
> Pier-Hugues Pellerin <ph@heykimo.com> skribis:
>
> > I am trying to update Go to 1.18, I do have a *working* patch that
> defines
> > a package that inherits from 1.17 and that adjusts the inputs.
>
> Nice!
>
> > However, I have a few questions:
> >
> > 1. Is inheriting the way to do that? The package's build system appears
> to
> > be the same from 1.17.
> > 2. How can I rebuild dependents packages to ensure they are still
> building
> > correctly?
> > 3. Anything I should look into or test more for these kinds of packages?
>
> You can define Go 1.18 inheriting from 1.17; that’ll allow us to have
> both versions, and eventually we’ll remove the older one.
>
> Note that, to actually migrate Go packages to 1.18, you’ll need to
> additionally change this line:
>
>   (define-public go go-1.17)
>
> ‘guix refresh -l go@1.17’ shows the “contour” of the set of packages
> that currently depend on Go 1.17.  These are all the packages that need
> to be rebuilt if you change the line above.  Since that’s a lot of them,
> we could set up a dedicated branch and have it built by ci.guix.  We’d
> merge it once problems have been resolved.
>
> Does that make sense?
>
> Thanks,
> Ludo’.
>


-- 
ph,
http://heykimo.com

[-- Attachment #2: Type: text/html, Size: 2555 bytes --]

  reply	other threads:[~2022-04-10 21:17 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-08 19:45 Updating from Go 1.17 to 1.18 Pier-Hugues Pellerin
2022-04-10 20:45 ` Ludovic Courtès
2022-04-10 21:16   ` Pier-Hugues Pellerin [this message]
2022-04-20 23:36     ` Katherine Cox-Buday
2022-04-20 23:43       ` Pier-Hugues Pellerin
2022-04-21  0:24         ` Pier-Hugues Pellerin
2022-04-21 14:58           ` Katherine Cox-Buday
2022-05-01 19:07             ` Pier-Hugues Pellerin

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

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CA+RyfLkMQ=JgDrX-e0emQpqayP7GPVEOHdsUeAkUHXVYXRsE7g@mail.gmail.com' \
    --to=ph@heykimo.com \
    --cc=guix-devel@gnu.org \
    --cc=ludo@gnu.org \
    /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 external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.