all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Troy Figiel <troy@troyfigiel.com>
To: Sharlatan Hellseher <sharlatanus@gmail.com>, 69244@debbugs.gnu.org
Subject: [bug#69244] [PATCH] gnu: go-golang-org-x-net-html: Add missing input.
Date: Sun, 18 Feb 2024 23:47:14 +0100	[thread overview]
Message-ID: <f4f7d694-0926-4505-9cfc-7a9ed1078220@troyfigiel.com> (raw)
In-Reply-To: <87o7cdjuid.fsf@gmail.com>


[-- Attachment #1.1.1: Type: text/plain, Size: 1336 bytes --]

Hi Oleg,

On 2024-02-18 23:08, Sharlatan Hellseher wrote> Did you check the go.mod
for the commit of go-golang-org-x-net package?
> It looks like it need 2 more modules
> 

The entire module (golang.org/x/net) does need those two Go packages,
but the golang.org/x/net/html Go package does not, i.e. the inputs as I
sent them are correct.

This is a result of how Guix has packaged Go in the past, with a Guix
package per Go package, not per Go module. This means we cannot reliably
use the go.mod file to determine dependencies, as there might be a bunch
that we do not need.

For future Guix packages, I think it will be easier to follow the Go
module structure instead, but that is a topic for the mailing list :-)

> Due to large amount of impact for any packages in golang-build, I would
> place it in go-team branch (not existing yet).
> 

With "guix refresh -l go-golang-org-x-net-html" I only see 2 dependents
(emacs-ghq rebuilds both). I am not sure I understand what you mean.

In general though, feel free to apply these missing inputs, fixes, etc.
whenever is convenient. I am locally working with a separate branch,
which already uses the "/..." fix (for both the build and check phases)
and I regularly rebase this branch on master. Sending updated patches is
no issue.

Best wishes,

Troy

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 6367 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  reply	other threads:[~2024-02-18 22:48 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-02-17 17:38 [bug#69244] [PATCH] gnu: go-golang-org-x-net-html: Add missing input Troy Figiel
2024-02-18 22:08 ` Sharlatan Hellseher
2024-02-18 22:47   ` Troy Figiel [this message]
2024-02-20  0:04 ` bug#69244: " 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

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

  git send-email \
    --in-reply-to=f4f7d694-0926-4505-9cfc-7a9ed1078220@troyfigiel.com \
    --to=troy@troyfigiel.com \
    --cc=69244@debbugs.gnu.org \
    --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 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.