unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: Maxime Devos <maximedevos@telenet.be>
To: John Kehayias <john.kehayias@protonmail.com>
Cc: "49339@debbugs.gnu.org" <49339@debbugs.gnu.org>
Subject: [bug#49339] [PATCH core-updates] gnu: mesa: Update to 21.1.4.
Date: Fri, 09 Jul 2021 14:48:08 +0200	[thread overview]
Message-ID: <ff9db3620a35942a1d0aba98bcd85bbd3a85cf7e.camel@telenet.be> (raw)
In-Reply-To: <_GQD4UBisH4wjKiOm3O6fzv8b1vdI890mQCX4gX6FQB7pXuDXpAU-mLiKMPRDMKu9vp_UFBUcOs6mlIKuKAbZ9W1lPQ5QnwhPBc2fra-Yow=@protonmail.com>

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

John Kehayias schreef op vr 09-07-2021 om 02:41 [+0000]:
> 
[...]
> > If with this update of mesa, (almost) every package using mesa also needs libglvnd,
> > 
> > then why not add 'libglvnd' to the propagated-inputs mesa, for about the same reasons
> > 
> > that 'atk' has 'glib' in 'propagated-inputs'?
> > 
> > Not sure if the comparison applies though.
> > [...]

> That sounds like a good idea, but I think there may be some kinks to work out. Adding libglvnd to propagated-inputs of Mesa does lead to the successful building of dependents (tested on libepoxy and virtualgl, for example).
> However, libepoxy fails on a test because it doesn't find libGL.
> That is no longer in Mesa's lib but from libglvnd if I'm understanding correctly. May just be a problem with the test since building works (which checks for GL).

Warning: I've no idea how building mesa and libglvnd works,
how linking against mesa and libglvnd works, and how mesa and libglvnd
work, besides ‘you can use the GL_... functions to do GL stuff’.

That said, it appears some package definitions expect "libGL.so" to be in mesa.
(Search for "/lib/libGL" and "lib/libEGL" with 'git grep -F"').
I've found about twenty such occurences, including libepoxy.

So it appears that adding libglvnd to the propagated-inputs and fixing
these twenty package definitions should be doable.

Looking at libepoxy in particular:

             (let ((python (assoc-ref inputs "python"))
                   (mesa (assoc-ref inputs "mesa")))
               (substitute* "src/gen_dispatch.py"
                 (("/usr/bin/env python") python))
               (substitute* (find-files "." "\\.[ch]$")
                 (("libGL.so.1") (string-append mesa "/lib/libGL.so.1"))
                 (("libEGL.so.1") (string-append mesa "/lib/libEGL.so.1")))
               #t))))))

it seems like the test failure isn't a false positive, as libGL.so.1 is searched
for in the wrong location.

> 
> Anyway, perhaps we want to tackle libglvnd separately? I don't think it is specific to the Mesa version change, but more of how we want to handle gl across packages. Still, it will involve changes to how we build Mesa as well as possibly other packages. I'm not sure that the Mesa version change will require other changes in dependent packages (I can only test a few on my own).
> 
> How do you think we should proceed?

I'd suggest adding libglvnd to propagated-inputs
and adjusting the twenty package definitions to refer
to libglvnd, and testing some graphical applications.

"mesa-utils" has "glxdemo" and "glxheads" and has few dependencies,
maybe start with that?

Greetings,
Maxime.

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 260 bytes --]

  reply	other threads:[~2021-07-09 12:50 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-07-02 15:10 [bug#49339] [PATCH core-updates] gnu: mesa: Update to 21.1.4 Irfan S
2021-07-02 19:29 ` John Kehayias via Guix-patches via
2021-07-05 15:35 ` John Kehayias via Guix-patches via
2021-07-08  1:35   ` John Kehayias via Guix-patches via
2021-07-08  2:24     ` John Kehayias via Guix-patches via
2021-07-08 15:40       ` John Kehayias via Guix-patches via
2021-07-08 20:55     ` Maxime Devos
2021-07-09  2:41       ` John Kehayias via Guix-patches via
2021-07-09 12:48         ` Maxime Devos [this message]
2021-07-09 15:34           ` John Kehayias via Guix-patches via
2021-07-13 15:42             ` John Kehayias via Guix-patches via
2021-07-13 16:26               ` John Kehayias via Guix-patches via
2021-07-27 21:54                 ` John Kehayias via Guix-patches via
2021-07-13  4:45 ` [bug#49339] Irfan S
2021-07-28 21:35 ` [bug#49339] [PATCH core-updates] gnu: mesa: Update to 21.1.4 Kaelyn via Guix-patches via
2021-07-29  4:52 ` [bug#49339] [PATCH core-updates] gnu: mesa: Update to 21.1.6 John Kehayias via Guix-patches via
2021-07-31 10:33   ` bug#49339: [PATCH core-updates] gnu: mesa: Update to 21.1.4 Ludovic Courtès
2021-07-31 14:01     ` [bug#49339] " John Kehayias via Guix-patches via
2021-08-02 13:27       ` Ludovic Courtès

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

  List information: https://guix.gnu.org/

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

  git send-email \
    --in-reply-to=ff9db3620a35942a1d0aba98bcd85bbd3a85cf7e.camel@telenet.be \
    --to=maximedevos@telenet.be \
    --cc=49339@debbugs.gnu.org \
    --cc=john.kehayias@protonmail.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 public inbox

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

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