all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Rutger Helling <rhelling@mykolab.com>
To: Marius Bakke <mbakke@fastmail.com>
Cc: 34824@debbugs.gnu.org
Subject: [bug#34824] [PATCH staging] libdrm/Mesa Meson patch series
Date: Mon, 18 Mar 2019 10:44:41 +0100	[thread overview]
Message-ID: <20190318104441.2dacc080@mykolab.com> (raw)
In-Reply-To: <87y35d8jmw.fsf@fastmail.com>

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

Hi Marius,

thanks for the review.

So when I rebuilt my entire system a while back with these patches it
was only those few SDL packages that were affected. Unfortunately I have
no way of knowing for sure if it won't happen anywhere else, but it
does seem to be something contained to just SDL(1).

The packages fail with the following error if "mesa" isn't an explicit
input:

ld: cannot find -lGL
collect2: error: ld returned 1 exit status

WDYT? Should I go ahead and push these patches or do you have other
ideas?

On Sun, 17 Mar 2019 19:01:59 +0100
Marius Bakke <mbakke@fastmail.com> wrote:

> Rutger Helling <rhelling@mykolab.com> writes:
> 
> > Hey Guix,
> >
> > this patch series changes libdrm and Mesa to use Meson.
> >
> > A few notes to go along with the patches:
> >
> > Patch #1: Libdrm on Github mentions that Autotools is the legacy
> > build system for libdrm, so it seems better to change this to use
> > Meson. Patch #2: Mesa is gonna drop support for Autotools entirely
> > at some point in 2019, so it seems good to make the switch sooner
> > rather than later. I think I converted all the configure flags to
> > the new format. Patches #3-6: For some reason these packages now
> > need an explicit mesa input. I haven't run into other packages that
> > have this problem yet.  
> 
> The first two patches LGTM, but we should figure out why some packages
> need special attention and squash the fixes with the Mesa (or libdrm)
> change.
> 
> I see the other packages are SDL-related, maybe the libSDL pkg-config
> files are different after this change?  Or "sdl-config" prints
> something weird?
> 
> Thanks for looking after these packages :-)


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

  reply	other threads:[~2019-03-18  9:45 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-03-12 13:16 [bug#34824] [PATCH staging] libdrm/Mesa Meson patch series Rutger Helling
2019-03-17 18:01 ` Marius Bakke
2019-03-18  9:44   ` Rutger Helling [this message]
2019-03-18 14:17     ` Marius Bakke
2019-03-18 15:25       ` Rutger Helling
2019-03-18 16:50         ` Marius Bakke
2019-03-19 10:10           ` bug#34824: " Rutger Helling

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=20190318104441.2dacc080@mykolab.com \
    --to=rhelling@mykolab.com \
    --cc=34824@debbugs.gnu.org \
    --cc=mbakke@fastmail.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.