GNU bug report logs - #59829
[PATCH core-updates] gnu: wlroots: Add propagated input xcb-util-renderutil

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 59829 in the body.
You can then email your comments to 59829 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Andy Tai <lichengtai <at> gmail.com>
To: guix-patches <at> gnu.org
Subject: [PATCH core-updates] gnu: wlroots: Add propagated input
 xcb-util-renderutil
Date: Sun, 4 Dec 2022 22:44:45 -0800

[Message part 1 (text/plain, inline)]

[0001-gnu-wlroots-Add-propagated-input-xcb-util-renderutil.patch (text/x-patch, attachment)]

Message #8 received at 59829 <at> debbugs.gnu.org (full text, mbox):

From: zimoun <zimon.toutoune <at> gmail.com>
To: Andy Tai <lichengtai <at> gmail.com>
Cc: 59829 <at> debbugs.gnu.org
Subject: Re: bug#59829: [PATCH core-updates] gnu: wlroots: Add propagated
 input xcb-util-renderutil
Date: Fri, 06 Jan 2023 12:53:34 +0100

Hi,

On Sun, 04 Dec 2022 at 22:44, Andy Tai <lichengtai <at> gmail.com> wrote:

> gnu/packages/wm.scm (wlroots): Add propagated input xcb-util-renderutil

Could you explain why this propagation is required?  Because propagation
should be avoid at first and it should be used when no other option are
possible, IMHO.


Cheers,
simon

Message #11 received at 59829 <at> debbugs.gnu.org (full text, mbox):

From: Andy Tai <lichengtai <at> gmail.com>
To: zimoun <zimon.toutoune <at> gmail.com>
Cc: 59829 <at> debbugs.gnu.org
Subject: Re: bug#59829: [PATCH core-updates] gnu: wlroots: Add propagated
 input xcb-util-renderutil
Date: Fri, 6 Jan 2023 23:13:19 -0800

[Message part 1 (text/plain, inline)]

Hi, I thought propagation inputs are for letting dependent packages to be
able to automatically add this input to its dependency list, like, say,
glib as a propagation input for gtk+ as all gtk+ [programs has glib as an
input or dependency.  xcb-util-renderutil seems to have a similar role for
wlroots.

If this is not correct, please file a patch to change it, and merge it.

On Fri, Jan 6, 2023 at 4:07 AM zimoun <zimon.toutoune <at> gmail.com> wrote:

> Hi,
>
> On Sun, 04 Dec 2022 at 22:44, Andy Tai <lichengtai <at> gmail.com> wrote:
>
> > gnu/packages/wm.scm (wlroots): Add propagated input xcb-util-renderutil
>
> Could you explain why this propagation is required?  Because propagation
> should be avoid at first and it should be used when no other option are
> possible, IMHO.
>
>
> Cheers,
> simon
>


-- 
Andy Tai, atai <at> atai.org  Skype: licheng.tai, Line.me: andy_tai, WeChat:
andytai1010
Year 2023 民國112年
自動的精神力是信仰與覺悟
自動的行為力是勞動與技能

[Message part 2 (text/html, inline)]

Message #14 received at 59829 <at> debbugs.gnu.org (full text, mbox):

From: Simon Tournier <zimon.toutoune <at> gmail.com>
To: Andy Tai <lichengtai <at> gmail.com>
Cc: 59829 <at> debbugs.gnu.org
Subject: Re: [bug#59829] [PATCH core-updates] gnu: wlroots: Add propagated
 input xcb-util-renderutil
Date: Mon, 09 Jan 2023 11:20:54 +0100

Hi,

On ven., 06 janv. 2023 at 23:13, Andy Tai <lichengtai <at> gmail.com> wrote:

> Hi, I thought propagation inputs are for letting dependent packages to be
> able to automatically add this input to its dependency list, like, say,
> glib as a propagation input for gtk+ as all gtk+ [programs has glib as an
> input or dependency.  xcb-util-renderutil seems to have a similar role for
> wlroots.

The manual [1] reads for propagated-inputs:

        Lastly, propagated-inputs is similar to inputs, but the
        specified packages will be automatically installed to profiles
        (see the role of profiles in Guix) alongside the package they
        belong to (see guix package, for information on how guix package
        deals with propagated inputs).

        For example this is necessary when packaging a C/C++ library
        that needs headers of another library to compile, or when a
        pkg-config file refers to another one via its Requires field.

        Another example where propagated-inputs is useful is for
        languages that lack a facility to record the run-time search
        path akin to the RUNPATH of ELF files; this includes Guile,
        Python, Perl, and more. When packaging libraries written in
        those languages, ensure they can find library code they depend
        on at run time by listing run-time dependencies in
        propagated-inputs rather than inputs.


Other said, a dependency is added to the propagated-inputs list when
Guix is not able to automatically determine if the package references a
dependency from the inputs list.

My naive question is: is xcb-util-renderutil required by wlroots to
correctly run?

1: <https://guix.gnu.org/manual/devel/en/guix.html#package_002dpropagated_002dinputs>


Cheers,
simon

Message #17 received at 59829 <at> debbugs.gnu.org (full text, mbox):

From: Andy Tai <lichengtai <at> gmail.com>
To: Simon Tournier <zimon.toutoune <at> gmail.com>
Cc: 59829 <at> debbugs.gnu.org
Subject: Re: [bug#59829] [PATCH core-updates] gnu: wlroots: Add propagated
 input xcb-util-renderutil
Date: Wed, 11 Jan 2023 00:18:25 -0800

I added it for XWayland, which requires this if XWayland support is
included in wlroots, which I believe shall be there.

On Mon, Jan 9, 2023 at 4:20 AM Simon Tournier <zimon.toutoune <at> gmail.com> wrote:
>
>
> My naive question is: is xcb-util-renderutil required by wlroots to
> correctly run?
>
>

Message #20 received at 59829 <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Andy Tai <lichengtai <at> gmail.com>
Cc: 59829 <at> debbugs.gnu.org, Simon Tournier <zimon.toutoune <at> gmail.com>
Subject: Re: bug#59829: [PATCH core-updates] gnu: wlroots: Add propagated
 input xcb-util-renderutil
Date: Mon, 16 Jan 2023 10:01:14 -0500

Hi Andy,

Andy Tai <lichengtai <at> gmail.com> writes:

> I added it for XWayland, which requires this if XWayland support is
> included in wlroots, which I believe shall be there.

It's good to know the reason behind the change, but as Simon said for
propagated inputs, we need a clear, exact understanding of why it is so,
as it's otherwise preferred to simple use simple inputs, and sometimes
wrapping.

Valid reasons currently include: references in public headers or
mentioned in Requires* in pkg-config files (those ending in '.pc'), or
for languages such as Python that lack a way to register dependencies in
binaries like C or C++ can (via RUNPATH in elf).

Can you spot which of the above it is?

-- 
Thanks,
Maxim

Message #23 received at 59829 <at> debbugs.gnu.org (full text, mbox):

From: Andy Tai <lichengtai <at> gmail.com>
To: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
Cc: 59829 <at> debbugs.gnu.org, Simon Tournier <zimon.toutoune <at> gmail.com>
Subject: Re: bug#59829: [PATCH core-updates] gnu: wlroots: Add propagated
 input xcb-util-renderutil
Date: Mon, 16 Jan 2023 13:34:21 -0800

hi, I will update the patch moving xcb-util-renderutil to inputs....
that will probably be safer.

On Mon, Jan 16, 2023 at 7:01 AM Maxim Cournoyer
<maxim.cournoyer <at> gmail.com> wrote:
>
> Hi Andy,
>
> Andy Tai <lichengtai <at> gmail.com> writes:
>
> > I added it for XWayland, which requires this if XWayland support is
> > included in wlroots, which I believe shall be there.
>
> It's good to know the reason behind the change, but as Simon said for
> propagated inputs, we need a clear, exact understanding of why it is so,
> as it's otherwise preferred to simple use simple inputs, and sometimes
> wrapping.
>
> Valid reasons currently include: references in public headers or
> mentioned in Requires* in pkg-config files (those ending in '.pc'), or
> for languages such as Python that lack a way to register dependencies in
> binaries like C or C++ can (via RUNPATH in elf).
>
> Can you spot which of the above it is?
>
> --
> Thanks,
> Maxim



-- 
Andy Tai, atai <at> atai.org  Skype: licheng.tai, Line.me: andy_tai,
WeChat: andytai1010
Year 2023 民國112年
自動的精神力是信仰與覺悟
自動的行為力是勞動與技能

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.