[Buildroot] [PATCH 1/1] package/lrzip: new package
Thomas Petazzoni
thomas.petazzoni at bootlin.com
Sat Sep 7 20:35:30 UTC 2019
Hello,
Thanks for this new version. When sending a new version for a patch, it
is customary to use:
[PATCH v2]
as the patch prefix, which you can do using "git format-patch -v2".
And also, to include a changelog of the changes between the v1 and v2,
below the "---", i.e...
On Sat, 7 Sep 2019 21:11:15 +0100
sam at gpsm.co.uk wrote:
> From: Sam Lancia <sam at gpsm.co.uk>
>
> lrzip is a compression utility that excels at compressing
> large files (usually > 10-50 MB)
>
> Signed-off-by: Sam Lancia <sam at gpsm.co.uk>
> ---
... here.
> diff --git a/package/lrzip/0001-missing-stdarg.patch b/package/lrzip/0001-missing-stdarg.patch
> new file mode 100644
> index 0000000000..7a19bc2834
> --- /dev/null
> +++ b/package/lrzip/0001-missing-stdarg.patch
> @@ -0,0 +1,15 @@
> +Lrzip.h: add missing header for va_list on some platforms
> +
> +Signed-off-by: Sam Lancia <sam at gpsm.co.uk>
We would like patches for projects that are hosted on Git to be
generated using git format-patch. So, grab the Git repo of the upstream
project, and make a proper commit. This way, you can also easily send a
pull request to the upstream lrzip project.
However, was this sufficient to fix all build issues ? Indeed stdarg.h
was missing, but there were also some bswap related problems.
> +comment "lrzip needs a toolchain w/ wchar"
> + depends on BR2_USE_MMU
> + depends on !BR2_USE_WCHAR
> + depends on BR2_TOOLCHAIN_HAS_THREADS
No, that's not the proper way. It should be:
comment "lrzip needs a toolchain w/ wchar, threads"
depends on BR2_USE_MMU
depends on !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
More information about the buildroot
mailing list