[Buildroot] [PATCH v2 3/6] package/gstreamer1/gst1-plugins-good: convert package type to meson
Arnout Vandecappelle
arnout at mind.be
Fri Jul 12 09:42:57 UTC 2019
On 11/07/2019 21:30, Peter Seiderer wrote:
> Hello Arnout,
>
> On Wed, 10 Jul 2019 23:53:24 +0200, Arnout Vandecappelle <arnout at mind.be> wrote:
>
>> On 10/07/2019 22:52, Peter Seiderer wrote:
>>> Hello Adam,
>>>
>>> On Wed, 10 Jul 2019 11:36:31 -0400, aduskett at gmail.com wrote:
>>
>> [snip]
>>>> - --disable-osx_audio \
>>>> - --disable-osx_video \
>>>> - --disable-aalib \
>>>> - --disable-aalibtest \
>>>> - --disable-libcaca
>>>> + -Dexamples=disabled \
>>>> + -Dtests=disabled \
>>>
>>> While your are at it , the following options should move to the
>>> 'Options which require currently unpackaged libraries' section
>>> some lines below:
>>
>> No, that should be done as a separate patch.
>>
>> It would also be nice BTW if all the options would be ordered alphabetically.
>> Now they're a bit of a hodgepodge.
>
> O.k. stuff for an follow up patch, I would prefer option ordering according
> to meson_options.txt (to enable easy check for added/removed/changed options)...
OK. then indeed it can be done at the same time as the conversion to meson.
The ordering choice should be mentioned in the commit message then.
>> [snip]
>>>> ifeq ($(BR2_PACKAGE_GST1_PLUGINS_GOOD_ZLIB),y)
>>>> -GST1_PLUGINS_GOOD_CONF_OPTS += --enable-zlib
>>>> GST1_PLUGINS_GOOD_DEPENDENCIES += zlib
>>>> -else
>>>> -GST1_PLUGINS_GOOD_CONF_OPTS += --disable-zlib
>>>> endif
>>>
>>> Should be:
>>>
>>> # checked for by meson.build, gst/isomp4/meson.build, gst/matroska/meson.build
>>> # and gst/icydemux/meson.build
>>
>> We don't add such a comment for any other package, so not needed.
>
> Would like to keep it, because:
>
> - as prove of correctness of the removed BR2_PACKAGE_GST1_PLUGINS_GOOD_ZLIB Config.in entry
> - as a reminder/blueprint for identifying zlib-like minor feature dependencies not expressed
> in the root meson_options.txt file
Good point, but IMHO it fits more as a commit message paragaph than as a
comment in the source file.
Regards,
Arnout
[snip]
More information about the buildroot
mailing list