[Buildroot] [PATCH v4 3/5] package/gutenprint: Un-deprecate and bump version to 5.2.10
Arnout Vandecappelle
arnout at mind.be
Sun Jan 24 14:26:01 UTC 2016
On 24-01-16 14:50, Olivier Schonken wrote:
> Hi Thomas, All
>
> Attached is a patch that worked for fixing the build issue you had with
> Gutenprint. Seems they have a bit of issues with code quality control/review
> upstream.
> For the print-olympus.c file the way in which they checked the endianess
> differed from the rest of the project, and the also had a variable in genppd.c
> that was dependant on a HAVE_ZLIB define, but not encased in an #ifdef block.
>
> If the changes are in order I will resubmit using the usual methods.
>
A few small comments:
It should be two separate patches, because they do two unrelated things.
The patch commit log for the endianness fix could be improved a little, e.g.
print-olympus.c: fix endianness detection
The __LITTLE_ENDIAN__ and __BIG_ENDIAN__ macros are not a reliable way to detect
endianness. Instead, rely on __BYTE_ORDER like we do in the rest of the code.
There's a redundant newline in the patch description.
And finally, don't forget to send these patches upstream.
Thanks,
Arnout
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
More information about the buildroot
mailing list