all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Nicolas Graves via Guix-patches via <guix-patches@gnu.org>
To: Maxim Cournoyer <maxim.cournoyer@gmail.com>
Cc: 74517@debbugs.gnu.org
Subject: [bug#74517] [PATCH 0/8] Remove gtk@4 input dependency on qtbase@5.
Date: Mon, 25 Nov 2024 08:47:25 +0100	[thread overview]
Message-ID: <87v7wbsqk2.fsf@ngraves.fr> (raw)
In-Reply-To: <87ttbwknxb.fsf@gmail.com>

On 2024-11-25 12:11, Maxim Cournoyer wrote:

> Hi,
>
> Nicolas Graves <ngraves@ngraves.fr> writes:
>
>> I found that curious that gtk@4 inherently requires qtbase@5
>> in the store's inputs.  Turns out it's based on a few libraries that
>> have optional graphical helping tools.  I've disabled them using
>> minimal variants.  After this patch series there are no input-only
>> path from gtk@4 to qtbase@5.
>>
>> Nicolas Graves (8):
>>   gnu: Add v4l-utils-minimal.
>>   gnu: zbar: Split outputs.
>>   gnu: Add zbar-minimal.
>>   gnu: libde265: Remove unused qtbase-5 input.
>
> I left some comments; I think I'd prefer a simpler rebuilt variant
> solution compared to the fancier but more frail (because of more
> complex) runpath hacking/trivial-build-system copying business involved
> in this version.

I get it, I chose that because of its lack of need to rebuild, while the
simpler rebuild variant would indeed require a rebuild (here a copy).
It's not that big of a subject here, these libraries are not super heavy
themselves so I'll change that.

I still like this solution for this reason, and I think this could be
applied for some other packages I've worked on (e.g. libreoffice is very
tricky to separate the build into steps, because some of its build
elements require the whole package, but it can be split at the end).
Agree to keep this as an exception when the alternative is really more
complex though, or when the build is very heavy (and thus making a copy
despite additional complexity makes sense).

>>   gnu: libde265: Update to 1.0.14.
>>   gnu: libdc1394: Replace v4l-utils by v4l-utils-minimal.
>>   gnu: gst-plugins-bad: Replace inputs with -minimal variants.
>>   gnu: gst-plugins-good: Replace v4l-utils by v4l-utils-minimal.
>
> These LGTM.
>
> Thanks a lot for tackling this problem!

-- 
Best regards,
Nicolas Graves




  reply	other threads:[~2024-11-25  7:48 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-11-24 22:09 [bug#74517] [PATCH 0/8] Remove gtk@4 input dependency on qtbase@5 Nicolas Graves via Guix-patches via
2024-11-24 22:29 ` [bug#74517] [PATCH 1/8] gnu: Add v4l-utils-minimal Nicolas Graves via Guix-patches via
2024-11-24 22:29   ` [bug#74517] [PATCH 2/8] gnu: zbar: Split outputs Nicolas Graves via Guix-patches via
2024-11-25  3:04     ` Maxim Cournoyer
2024-11-25  7:42       ` Nicolas Graves via Guix-patches via
2024-11-24 22:29   ` [bug#74517] [PATCH 3/8] gnu: Add zbar-minimal Nicolas Graves via Guix-patches via
2024-11-25  3:08     ` Maxim Cournoyer
2024-11-24 22:29   ` [bug#74517] [PATCH 4/8] gnu: libde265: Remove unused qtbase-5 input Nicolas Graves via Guix-patches via
2024-11-25  3:08     ` Maxim Cournoyer
2024-11-24 22:29   ` [bug#74517] [PATCH 5/8] gnu: libde265: Update to 1.0.14 Nicolas Graves via Guix-patches via
2024-11-25  3:09     ` Maxim Cournoyer
2024-11-24 22:29   ` [bug#74517] [PATCH 6/8] gnu: libdc1394: Replace v4l-utils by v4l-utils-minimal Nicolas Graves via Guix-patches via
2024-11-24 22:29   ` [bug#74517] [PATCH 7/8] gnu: gst-plugins-bad: Replace inputs with -minimal variants Nicolas Graves via Guix-patches via
2024-11-24 22:29   ` [bug#74517] [PATCH 8/8] gnu: gst-plugins-good: Replace v4l-utils by v4l-utils-minimal Nicolas Graves via Guix-patches via
2024-11-25  2:54   ` [bug#74517] [PATCH 1/8] gnu: Add v4l-utils-minimal Maxim Cournoyer
2024-11-25  2:56   ` Maxim Cournoyer
2024-11-25  2:38 ` [bug#74517] [PATCH 0/8] Remove gtk@4 input dependency on qtbase@5 Maxim Cournoyer
2024-11-25  3:11 ` Maxim Cournoyer
2024-11-25  7:47   ` Nicolas Graves via Guix-patches via [this message]
2024-11-25  9:43 ` [bug#74517] [PATCH v2 01/10] gnu: Add v4l-utils-minimal Nicolas Graves via Guix-patches via
2024-11-25  9:43   ` [bug#74517] [PATCH v2 02/10] gnu: zbar: Improve style Nicolas Graves via Guix-patches via
2024-11-25  9:43   ` [bug#74517] [PATCH v2 03/10] gnu: zbar: Disable static and qt build Nicolas Graves via Guix-patches via
2024-11-25  9:43   ` [bug#74517] [PATCH v2 04/10] gnu: zbar: Update to 0.23.92 Nicolas Graves via Guix-patches via
2024-11-25  9:43   ` [bug#74517] [PATCH v2 05/10] gnu: Add zbar-minimal Nicolas Graves via Guix-patches via
2024-11-25  9:43   ` [bug#74517] [PATCH v2 06/10] gnu: libde265: Remove unused qtbase-5 input Nicolas Graves via Guix-patches via
2024-11-25  9:43   ` [bug#74517] [PATCH v2 07/10] gnu: libde265: Update to 1.0.14 Nicolas Graves via Guix-patches via
2024-11-25  9:43   ` [bug#74517] [PATCH v2 08/10] gnu: libdc1394: Replace v4l-utils by v4l-utils-minimal Nicolas Graves via Guix-patches via
2024-11-25  9:43   ` [bug#74517] [PATCH v2 09/10] gnu: gst-plugins-bad: Replace inputs with -minimal variants Nicolas Graves via Guix-patches via
2024-11-25  9:43   ` [bug#74517] [PATCH v2 10/10] gnu: gst-plugins-good: Replace v4l-utils by v4l-utils-minimal Nicolas Graves via Guix-patches via
2024-11-25  9:52 ` [bug#74517] [PATCH v3 01/10] gnu: Add v4l-utils-minimal Nicolas Graves via Guix-patches via
2024-11-25  9:52   ` [bug#74517] [PATCH v3 02/10] gnu: zbar: Improve style Nicolas Graves via Guix-patches via
2024-11-25  9:52   ` [bug#74517] [PATCH v3 03/10] gnu: zbar: Disable static and qt build Nicolas Graves via Guix-patches via
2024-11-25  9:52   ` [bug#74517] [PATCH v3 04/10] gnu: zbar: Update to 0.23.92 Nicolas Graves via Guix-patches via
2024-11-25  9:52   ` [bug#74517] [PATCH v3 05/10] gnu: Add zbar-minimal Nicolas Graves via Guix-patches via
2024-11-25  9:52   ` [bug#74517] [PATCH v3 06/10] gnu: libde265: Remove unused qtbase-5 input Nicolas Graves via Guix-patches via
2024-11-25  9:52   ` [bug#74517] [PATCH v3 07/10] gnu: libde265: Update to 1.0.14 Nicolas Graves via Guix-patches via
2024-11-25  9:52   ` [bug#74517] [PATCH v3 08/10] gnu: libdc1394: Replace v4l-utils by v4l-utils-minimal Nicolas Graves via Guix-patches via
2024-11-25  9:52   ` [bug#74517] [PATCH v3 09/10] gnu: gst-plugins-bad: Replace inputs with -minimal variants Nicolas Graves via Guix-patches via
2024-11-25  9:52   ` [bug#74517] [PATCH v3 10/10] gnu: gst-plugins-good: Replace v4l-utils by v4l-utils-minimal Nicolas Graves via Guix-patches via

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=87v7wbsqk2.fsf@ngraves.fr \
    --to=guix-patches@gnu.org \
    --cc=74517@debbugs.gnu.org \
    --cc=maxim.cournoyer@gmail.com \
    --cc=ngraves@ngraves.fr \
    /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.