<div dir="ltr"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr">Le dim. 16 déc. 2018 à 21:51, Thomas Petazzoni <<a href="mailto:thomas.petazzoni@bootlin.com">thomas.petazzoni@bootlin.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello François,<br>
<br>
Even though I applied the series, I have a few comments.<br>
<br>
On Wed, 12 Dec 2018 08:14:52 +0100, Francois Perrad wrote:<br>
> Signed-off-by: Francois Perrad <<a href="mailto:francois.perrad@gadz.org" target="_blank">francois.perrad@gadz.org</a>><br>
> ---<br>
>  .../netsurf/0002-fix-freetype-detection.patch | 28 +++++++++++++++++++<br>
>  package/netsurf/<a href="http://netsurf.mk" rel="noreferrer" target="_blank">netsurf.mk</a>                    |  7 +++++<br>
>  2 files changed, 35 insertions(+)<br>
>  create mode 100644 package/netsurf/0002-fix-freetype-detection.patch<br>
> <br>
> diff --git a/package/netsurf/0002-fix-freetype-detection.patch b/package/netsurf/0002-fix-freetype-detection.patch<br>
> new file mode 100644<br>
> index 000000000..27c0440e1<br>
> --- /dev/null<br>
> +++ b/package/netsurf/0002-fix-freetype-detection.patch<br>
> @@ -0,0 +1,28 @@<br>
> +From 85336f2404b9583267019650ea0112ebf1bb571a Mon Sep 17 00:00:00 2001<br>
> +From: Francois Perrad <<a href="mailto:francois.perrad@gadz.org" target="_blank">francois.perrad@gadz.org</a>><br>
> +Date: Wed, 28 Nov 2018 17:59:44 +0100<br>
> +Subject: [PATCH 2/2] fix freetype detection<br>
<br>
When generating patches, use the -N option of git format-patch to avoid<br>
the numbering: it provides patches that use [PATCH] instead of [PATCH<br>
X/Y]. We prefer just [PATCH] because the X/Y often get invalid. Your<br>
series is a good example: this patch adds a PATCH 2/2 patch, and the<br>
next patch adds a 3/3 patch, and then another patch adds a 4/4 patch.<br>
<br>
Another comment: did you submit all those patches to the upstream<br>
netsurf project ?<br></blockquote><div><br></div><div>I will do.</div><div><br></div><div>note: this package still fail on autobuilders, see fix on <a href="https://patchwork.ozlabs.org/patch/1013055/">https://patchwork.ozlabs.org/patch/1013055/</a><br></div><div><br></div><div>François<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Thanks,<br>
<br>
Thomas<br>
-- <br>
Thomas Petazzoni, CTO, Bootlin<br>
Embedded Linux and Kernel engineering<br>
<a href="https://bootlin.com" rel="noreferrer" target="_blank">https://bootlin.com</a><br>
_______________________________________________<br>
buildroot mailing list<br>
<a href="mailto:buildroot@busybox.net" target="_blank">buildroot@busybox.net</a><br>
<a href="http://lists.busybox.net/mailman/listinfo/buildroot" rel="noreferrer" target="_blank">http://lists.busybox.net/mailman/listinfo/buildroot</a><br>
</blockquote></div></div></div>