[flashrom] [PATCH 1/2] Add missing header for musl compatibility

Stefan Tauner stefan.tauner at alumni.tuwien.ac.at
Thu Oct 29 00:32:08 CET 2015


On Tue, 27 Oct 2015 08:12:24 +0100
"gwenhael.goavec" <gwenj at trabucayre.com> wrote:

> Hi Stefan
> On Mon, 26 Oct 2015 23:59:38 +0100
> Stefan Tauner <stefan.tauner at alumni.tuwien.ac.at> wrote:
> 
> > On Wed, 14 Oct 2015 10:43:07 +0200
> > Gwenhael Goavec-Merou <gwenj at trabucayre.com> wrote:
> > 
> > > From: Gwenhael Goavec-Merou <gwenhael.goavec-merou at trabucayre.com>
> > > 
> The compile problem is shown in this build report :
> > > http://autobuild.buildroot.net/results/3de/3de936d9be79e151e66af15193084d82a0f2c04a/build-end.log
> This problem is clearly related to flashrom.

Well, yes true... however, I am not sure if that's actually a problem
with the includes in flashrom or the actual headers they are using.
After all we do not use the undefined macro directly, so it should be
brought in by the header file that uses it IMHO.

flashrom does not support non-glibc C libraries yet anyway (this also
affects your other patch) so this needs more investigation in any case.
I am currently building a musl "crosscompiler" toolchain on our flashrom
buildbot to allow for that (because the musl-gcc wrapper does not work
with libraries like libpci). Don't expect any results soon but I'll put
it on my todo list... thanks for the heads-up.

-- 
Kind regards/Mit freundlichen Grüßen, Stefan Tauner




More information about the flashrom mailing list