Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 16 Mar 2021 23:19:12 +0100
From:      Mathieu Arnold <mat@freebsd.org>
To:        Juraj Lutter <otis@freebsd.org>
Cc:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   Re: svn commit: r568508 - head/devel/node-gyp
Message-ID:  <20210316221912.smaeylfqtx7yfm6i@aching.in.mat.cc>
In-Reply-To: <2239997E-E3B6-403F-B36D-4326CDF16EFE@FreeBSD.org>
References:  <202103152102.12FL2tRP068067@repo.freebsd.org> <20210316211715.pa4w3lrffzxr3ftx@aching.in.mat.cc> <2239997E-E3B6-403F-B36D-4326CDF16EFE@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help

--m24qsdxsnuqurlda
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Tue, Mar 16, 2021 at 11:10:16PM +0100, Juraj Lutter wrote:
>=20
> > On 16 Mar 2021, at 22:17, Mathieu Arnold <mat@FreeBSD.org> wrote:
> >=20
> > On Mon, Mar 15, 2021 at 09:02:55PM +0000, Juraj Lutter wrote:
> >> Author: otis
> >> Date: Mon Mar 15 21:02:55 2021
> >> New Revision: 568508
> >> URL: https://svnweb.freebsd.org/changeset/ports/568508
> >>=20
> >> Log:
> >>  Add port: devel/node-gyp: a build frmework for building Node.js appli=
cations.
> >=20
> > A few problems with this, and the next 2 ports added, so putting my
> > portmgr's hat on.
> >=20
> > We do not add node modules to the ports tree.
> > We really do not add node modules to the ports tree.
> >=20
> > If an application needs nodes modules to work, then the port for that
> > application needs to handle the modules installing, and put those in a
> > private place.
> >=20
> > lib/node_modules is a really aweful place to put this.  As a side note,
> > if we were to add node modules to the ports tree, which we are not, it
> > would probably be libdata/node_modules or share/node_modules.
> >=20
> > Please remove those ports.
>=20
>=20
> The rationale behind this is:
> - Upcoming textproc/kibana7 has =E2=80=9Cre2=E2=80=9D node module bundled=
, which also contains
> a re2.node binary module, that (from distribution tarball) can be for Lin=
ux or Darwin.
>=20
> - In order to build re2.node binary component for FreeBSD in poudriere wi=
thout need for
> network access that would otherwise be needed, separate port seemed the m=
ost ellegant way.
>=20
> I value your comment and any hints/advises are welcome.

The rationale being that given X ports that depend on a node module, you
will probably need more than X versions of that module because they
will all depend on incompatible versions of said module.

A port that requires node modules must provide its own.  It can
absolutely be by having an extra step for the maintainer to generate a
tarball with all the required modules that can simply be extracted as
needed.  I have not checked, but there are probably ports already doing
this.

--=20
Mathieu Arnold

--m24qsdxsnuqurlda
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQKTBAABCgB9FiEEVhwchfRfuV0unqO5KesJApEdfgIFAmBRLttfFIAAAAAALgAo
aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldDU2
MUMxQzg1RjQ1RkI5NUQyRTlFQTNCOTI5RUIwOTAyOTExRDdFMDIACgkQKesJApEd
fgI8uQ/9EP4GeVzCv3wWp9eRRgNIGcPxzYdGPbnF/mE+8gWBV+C81WnV0jdlEORk
FRKY+JOEVi5MzPTfIu3B93maxQft4tLOlXA0S76Tn4fznEIPvPVFZhl/8uisLAL0
pKGj7PQnVp+yy56BfVygPWJKb6Ws82TX2YxqOyj0W0I2i7QizaMuhHQ1ftVpkp85
bOSDA/gH42Kkn+zMYu3FtYP79I9N+kTd29PKUeuAq/6M7LbFt+w0TL8lko5ah+d/
CCXNQKVU4MrQfmRz2wUQlBOEJk5S0TbwcxV5T8kJwNjMO7HUeAfdX+HgIVatB9E/
dwxleFo+ppDQLWNVH1MB/+kSBYUwP9Qs9mW659VwSkEbyTGQWFQBpNOq/T1Ye2Kt
RG6KFK1vFI9/jW6XSFUsYLohw+X0nnZx7IuqZSwMGf9wItD2yXR1JKqhk/dKHhEg
w+TBTJvAMKiNCQ6MN+KwzYDmVKjtaAzocrjEUocOOUKDHjBkyDuEHTET7jz2uz6Q
COwXvIP9OqB9ljAawLsiOFSsvOHlSk4rZhoj8uwXfIH/5kb/GDqn7AwKIM+knNw2
K2zak0F1FBcBrKRP8Kg6ueJqS588GVWC3RH3DlTAdnQD7YOV6Kh0KfctR0DNic/e
aPtZS6UctJPUWnoSthELAuNNBS/+wXy8vgamuoV/H7Nwtf1mBbM=
=3ada
-----END PGP SIGNATURE-----

--m24qsdxsnuqurlda--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20210316221912.smaeylfqtx7yfm6i>