[Buildroot] [PATCH v2] gst1-plugins-imx: add package
Luca Ceresoli
luca at lucaceresoli.net
Sun Nov 2 22:13:21 UTC 2014
Dear Gary,
thanks for you resubmission.
Gary Bisson wrote:
> Open-source GStreamer 1.0 plugins for i.MX platforms.
> More info at https://github.com/Freescale/gstreamer-imx
>
> Signed-off-by: Gary Bisson <bisson.gary at gmail.com>
> ---
> Modifications v1->v2:
> - Use of github helper macro
> - Modify package comment with Peter S. original patch
> - Add comment for eglibc dependency (due to GPU libs)
> - Remove comments on package dependencies
>
> One thing I wasn't sure about, should all new package contain a .hash file? If
> so I can submit a v3 right away.
It is not mandatory. AFAIK the hash is recommended for security-related
packages only.
[...]
> diff --git a/package/gstreamer1/gst1-plugins-imx/Config.in b/package/gstreamer1/gst1-plugins-imx/Config.in
> new file mode 100644
> index 0000000..d499874
> --- /dev/null
> +++ b/package/gstreamer1/gst1-plugins-imx/Config.in
> @@ -0,0 +1,28 @@
> +comment "gst1-plugins-imx needs an imx-specific Linux kernel to be built"
> + depends on BR2_arm && !BR2_LINUX_KERNEL
> +
> +# Required by gpu-viv-bin-mx6q
> +comment "gst1-plugins-imx needs an (e)glibc toolchain"
> + depends on BR2_arm
> + depends on !BR2_TOOLCHAIN_USES_GLIBC
These messages are much cleaner than in version 1 of your patch.
However, for an ARM configuration with a musl or uClibc toolchain and
without a kernel build activated, two lines are displayed. This is not
coherent with the general Buildroot behavior.
It should be:
comment "gst1-plugins-imx needs an (e)glibc toolchain and an
imx-specific Linux kernel to be built"
depends on BR2_arm && (!BR2_LINUX_KERNEL || !BR2_TOOLCHAIN_USES_GLIBC)
Although that's a bit lengthy...
With that fixed (which can be done by the committer):
Reviewed-by: Luca Ceresoli <luca at lucaceresoli.net>
I'll try to test it during the coming week and give my Tested-by tag, or
report any issues.
--
Luca
More information about the buildroot
mailing list