pkgsrc-Bugs archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: pkg/55912: gstreamer1 incomplete package installation / failed build



The following reply was made to PR pkg/55912; it has been noted by GNATS.

From: Aleksey Arens <aza.sea.agenda%gmail.com@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: 
Subject: Re: pkg/55912: gstreamer1 incomplete package installation / failed build
Date: Wed, 6 Jan 2021 03:05:59 -0800

 --00000000000090ce5605b8394fb0
 Content-Type: text/plain; charset="UTF-8"
 
 Hi Nia,
 
 The build of gstreamer1 was failing with
 PKG_DEFAULT_OPTIONS+=-introspection.  Evidently, one of the dependencies
 lacked introspection support, and thus the files related to introspection
 in gstreamer1 were not being built in gstreamer1.
 
 I wonder, if the introspection was actually meant to be enabled throughout
 all of pkgsrc?  If that is the case, then the naming of option itself
 `introspection' with a vanilla description is confusing, as it appears to
 be only an option.  If the option was introduced as a means to workaround
 issues on certain platforms, then perhaps it should have been really a part
 of "platform-workarounds" options (e.g. win32-workarounds,
 xnu-10.4-workarounds), and/or have been handled in the .if/.else clauses in
 the packages' Makefiles.  I would in general advocate for the workaround
 options appearing in the PKG_DEFAULT_OPTIONS, and being automatically
 activated in the Makefiles by platform check clauses.
 
 To me, as a system administrator, the presence of a *vanilla*
 `introspection' signifies that there's yet more bloat to be removed.  If
 those features turn out to be essential for some uses in important
 dependencies, then it makes sense for them to be treated as not really
 options, but rather as platform workarounds.  This is especially so, if
 they do not add heavy dependencies (they really don't), or questionable
 dependencies (like e.g. dbus -- and they don't), and don't affect the
 performance adversely (that I'm not absolutely 146% sure about, but I would
 assume since they're being merely type descriptions, and since they must be
 describing generic datastructures, the performance should not be affected).
 
 Finally, there's a question, as to how gstreamer1 managed to be built on
 platforms with introspection disabled in its dependencies?  In those cases,
 the build must have failed, especially with
 PKG_DEFAULT_OPTIONS+=-introspection.
 
 --
 Aleksey
 
 
 On Wed, Jan 6, 2021 at 2:15 AM nia <nia%netbsd.org@localhost> wrote:
 
 > The following reply was made to PR pkg/55912; it has been noted by GNATS.
 >
 > From: nia <nia%NetBSD.org@localhost>
 > To: gnats-bugs%netbsd.org@localhost
 > Cc:
 > Subject: Re: pkg/55912: gstreamer1 incomplete package installation / failed
 >  build
 > Date: Wed, 6 Jan 2021 10:11:12 +0000
 >
 >  Hi,
 >
 >  why's introspection disabled? IIRC these options were added as a result
 >  of problems with introspection on some platforms, which are now fixed?
 >
 >
 
 --00000000000090ce5605b8394fb0
 Content-Type: text/html; charset="UTF-8"
 Content-Transfer-Encoding: quoted-printable
 
 <div dir=3D"ltr">Hi Nia,<br><br>The build of gstreamer1 was failing with PK=
 G_DEFAULT_OPTIONS+=3D-introspection.=C2=A0 Evidently, one of the dependenci=
 es lacked introspection support, and thus the files related to introspectio=
 n in gstreamer1 were not being built in gstreamer1.<div><br></div><div>I wo=
 nder, if the introspection was actually meant to be enabled throughout all =
 of pkgsrc?=C2=A0 If that is the case, then the naming of option itself `int=
 rospection&#39; with a vanilla description is confusing, as it appears to b=
 e only an option.=C2=A0 If the option was introduced as a means to workarou=
 nd issues on certain platforms, then perhaps it should have been really a p=
 art of &quot;platform-workarounds&quot; options (e.g. win32-workarounds, xn=
 u-10.4-workarounds), and/or have been handled in the .if/.else clauses in t=
 he packages&#39; Makefiles.=C2=A0 I would in general advocate for the worka=
 round options=C2=A0appearing in the PKG_DEFAULT_OPTIONS, and being automati=
 cally activated in the Makefiles by platform check clauses.</div><div><br><=
 /div><div>To me, as a system administrator, the presence of a *vanilla* `in=
 trospection&#39; signifies that there&#39;s yet more bloat to be removed.=
 =C2=A0 If those features turn out to be essential for some uses in importan=
 t dependencies, then it makes sense for them to be treated as not really op=
 tions, but rather as platform workarounds.=C2=A0 This is especially so, if =
 they do not add heavy dependencies (they really don&#39;t), or questionable=
  dependencies (like e.g. dbus -- and they don&#39;t), and don&#39;t affect =
 the performance adversely (that I&#39;m not absolutely 146% sure about, but=
  I would assume since they&#39;re being merely type descriptions, and since=
  they must be describing generic datastructures, the performance should not=
  be affected).</div><div><br></div><div>Finally, there&#39;s a question, as=
  to how gstreamer1 managed to be built on platforms with introspection disa=
 bled in its dependencies?=C2=A0 In those cases, the build must have failed,=
 =C2=A0especially with PKG_DEFAULT_OPTIONS+=3D-introspection.</div><div><br>=
 </div><div>--</div><div>Aleksey</div></div><br><br><div class=3D"gmail_quot=
 e"><div dir=3D"ltr" class=3D"gmail_attr">On Wed, Jan 6, 2021 at 2:15 AM nia=
  &lt;<a href=3D"mailto:nia%netbsd.org@localhost";>nia%netbsd.org@localhost</a>&gt; wrote:<br></d=
 iv><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;bord=
 er-left:1px solid rgb(204,204,204);padding-left:1ex">The following reply wa=
 s made to PR pkg/55912; it has been noted by GNATS.<br>
 <br>
 From: nia &lt;nia%NetBSD.org@localhost&gt;<br>
 To: <a href=3D"mailto:gnats-bugs%netbsd.org@localhost"; target=3D"_blank">gnats-bugs@n=
 etbsd.org</a><br>
 Cc: <br>
 Subject: Re: pkg/55912: gstreamer1 incomplete package installation / failed=
 <br>
 =C2=A0build<br>
 Date: Wed, 6 Jan 2021 10:11:12 +0000<br>
 <br>
 =C2=A0Hi,<br>
 <br>
 =C2=A0why&#39;s introspection disabled? IIRC these options were added as a =
 result<br>
 =C2=A0of problems with introspection on some platforms, which are now fixed=
 ?<br>
 <br>
 </blockquote></div>
 
 --00000000000090ce5605b8394fb0--
 


Home | Main Index | Thread Index | Old Index