[Pixman] [PATCH] build: Ensure config.h does not undefine `sqrtf`

Peter TB Brett peter at peter-b.co.uk
Thu Oct 6 11:30:57 UTC 2016


On 06/10/2016 12:29, Siarhei Siamashka wrote:
>> The patch I provided makes it considerably easier for other users in our
>> position to get pixman to build correctly, by making it clearer what's
>> going on when reading "config.h" in order to make appropriate edits.
>
> Just a side note. We still can't make any guarantees about using
> the pixman build system in a non-intended way.

Understood. :-) I saw this as a good opportunity to help us with our 
non-supported hackery _and_ make pixman's official build system "more 
correct"!

>> Do you have any specific objections to merging it?
>
> I'm fine with fixing this. But would prefer to get rid of the
> AH_VERBATIM stuff and just directly add the
>
>     #ifndef HAVE_SQRTF
>     #define sqrtf sqrt
>     #endif
>
> chunk into the pixman-compiler.h header file. If this adjustment is
> done, then the patch will be IMHO good enough to be pushed into
> pixman git (unless other people have objections).
>
> You can wait maybe a day (so that more people have a chance to
> comment) and then send a v2 version of the patch. Thanks!

Will do!  Thanks for the feedback.

                                         Peter

-- 
Dr Peter Brett <peter at peter-b.co.uk>


More information about the Pixman mailing list