all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Pier-Hugues Pellerin <ph@heykimo.com>
To: Katherine Cox-Buday <cox.katherine.e@gmail.com>
Cc: Guix Devel <guix-devel@gnu.org>
Subject: Re: Updating from Go 1.17 to 1.18
Date: Wed, 20 Apr 2022 20:24:44 -0400	[thread overview]
Message-ID: <CA+RyfLm_=Vh32g9-isnHNkX6iezUG6r8EzcLTJibN6BLmi7emw@mail.gmail.com> (raw)
In-Reply-To: <CA+RyfL=3pApFuP+0xxCFaiw08mHvkhTqvACiLeA5agreQTycDQ@mail.gmail.com>

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

Hello,

I've looked a bit more in detail on the Go package definition, Maybe we
should rethink how we are building the Go package in guix?
If I understand correctly, the current bootstrap chain is the following.

1.17 -> 1.16-> 1.4 which is the last gcc version of go.

Looking at the current proposal in Go
https://mail.google.com/mail/u/0/#inbox/KtbxLthKLdQNrjXvKllfQHQBQcKqVFdmPg
I don't think we will be able to remove 1.17 soon and also 1.4.  Because
they want to move to use 1.17 as the minimum
 version in the next minor.

We should probably also try to mimic how their bootstrap and have the
following:

   - Bootstrap 1.17 with 1.4
   - Bootstrap 1.18 with 1.4
   - Have 1.19 bootstrap with 1.17.

So maybe we should do something like this.

- Add go-1.18 build from 1.4.
- Add go-next that point to go-1.18
- Refactor 1.17 package definition that is shared between go-1.18. and
go-1.17.
- Move  go to go-1.18  in another branch
- Remove 1.16?

WDYT? I am not super familiar with the scheme but this seems like a good
exercise.

Thanks


On Wed, Apr 20, 2022 at 7:43 PM Pier-Hugues Pellerin <ph@heykimo.com> wrote:

> That sound great and great timing I was working on that as you send the
> mail
>
> I think also go-next is a good idea, I will split them in the following
> commits:
>
> 1. Add go-1.17 inherits from 1.18 (actually reversing the patch in the
> previous email)
> 2. Add go-next pointing to 1.18.
> 3. Make go point to 1.18.
>
> This will allow merging things right away to add support for go 1.18 and
> also allow Ludovic to create the branch and rebuild.
>
> Thanks
>
> On Wed, Apr 20, 2022 at 7:36 PM Katherine Cox-Buday <
> cox.katherine.e@gmail.com> wrote:
>
>> >>> Pier-Hugues Pellerin <ph@heykimo.com> writes:
>> >> Ludovic Courtès writes:
>>
>> >>> 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!
>>
>> Yes, thank you! I just found out I need this and came to see if anyone had
>> started on it.
>>
>> >> 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.
>>
>> I suggest inverting this: copy/paste go-1.17 to go-1.18, and then make
>> go-1.17
>> inherit from go-1.18. This means that when it's time to sunset a version,
>> it's
>> a simple delete and not something that cascades through all recent
>> versions.
>>
>> > 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.
>>
>> I was wondering if we don't want to start publishing a go-next package
>> like we do with emacs-next? That would allow us to publish the latest
>> version of Go without needing to immediately address building all the
>> packages that depend on it.
>>
>> Kindest regards,
>> --
>> Katherine
>>
>
>
> --
> ph,
> http://heykimo.com
>


-- 
ph,
http://heykimo.com

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

  reply	other threads:[~2022-04-21  0:25 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
2022-04-20 23:36     ` Katherine Cox-Buday
2022-04-20 23:43       ` Pier-Hugues Pellerin
2022-04-21  0:24         ` Pier-Hugues Pellerin [this message]
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+RyfLm_=Vh32g9-isnHNkX6iezUG6r8EzcLTJibN6BLmi7emw@mail.gmail.com' \
    --to=ph@heykimo.com \
    --cc=cox.katherine.e@gmail.com \
    --cc=guix-devel@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.