On Tue, May 2, 2023 at 5:24 AM Iñaki Ucar <iu...@fedoraproject.org> wrote:
>
> On Tue, 4 Apr 2023 at 16:23, Stephen Gallagher <sgall...@redhat.com> wrote:
> >
> > On Tue, Apr 4, 2023 at 9:45 AM Sérgio Basto <ser...@serjux.com> wrote:
> > >
> > > On Thu, 2023-03-16 at 11:19 -0400, Stephen Gallagher wrote:
> > > > On Wed, Mar 15, 2023 at 1:16 PM Jerry James <loganje...@gmail.com>
> > ...
> > > I found a problem related with yarnpkg rpm, the macro % __find_requires
> > > finds that yarn scripts uses and needs /usr/bin/node , which is added
> > > to the requires of rpm [1] and this makes yarnpkg pull nodejs (18) even
> > > when nodejs20 is installed .
> > > To avoid this rpm automatic requires, we may add to yarnpkg.spec [2]
> > >
> > > [2]
> > > %global __script_requires %{nil}
> > >
> > >  [1]
> > > dnf repoquery yarnpkg --available --requires -q
> > > /usr/bin/bash
> > > /usr/bin/node
> > > /usr/bin/sh
> > >
> >
> > I'm not sure what you think is a bug here? Do you think `yarnpkg`
> > should use *any* nodejs version that's installed? The whole point of
> > the way this is broken down is that we have a default version (18, in
> > this case) with the option to install 16 and 20 in parallel, but you
> > have to do extra work if you want to *use* those non-default versions
> > (such as patching shebang lines).
>
> This is broken again in rawhide:
>
> $ dnf -qy install yarnpkg
> $ ll /usr/bin/node*
> lrwxrwxrwx. 1 root root     7 Apr 28 00:00 /usr/bin/node -> node-20
> -rwxr-xr-x. 1 root root 28272 Apr 28 00:00 /usr/bin/node-20
> lrwxrwxrwx. 1 root root    39 Mar 21 00:00 /usr/bin/nodejs-yarn ->
> ../lib/node_modules_19/yarn/bin/yarn.js

This is a bug in nodejs-packaging; it should always have been
installing to node_modules_20, but the %nodejs_sitearch variable is
wrong. I'll get that fixed.

>
> If yarn should be pinned to a node version, please rebuild yarn when
> there's a major version change. And it would be nice if there's a
> mechanism to detect such a breakage. I.e. a nodejs(abi) version or
> something like that.

We have exactly that. The version of nodejs(abi) matches the output of
`/usr/bin/node -p process.versions.modules` (which is also the version
suffix of libnode)

That being said, you are correct that it should be rebuilt when we do
a major version change (i.e. 18->20). If yarnpkg had been using
`Requires:

>
> --
> Iñaki Úcar
>
_______________________________________________
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue

Reply via email to