<div><div><div dir="auto">On Tue, 10 Dec 2019 at 10:51 pm, Simon McVittie wrote:</div></div></div><div><div><div class="gmail_quote"></div></div></div><div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
With which of my patches? I sent three merge requests (!3, !4, !5) with<br>
different ways to solve multiarch coinstallability, each with different<br>
advantages and disadvantages, in the hope that at least one of them would<br>
be considered acceptable and unblock progress on this bug.</blockquote><div dir="auto"><br></div></div><div><div dir="auto">Sorry, I should have said I applied !5.</div></div><div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
I would expect that !5 (the one that Felix has now merged, using<br>
$PKG_CONFIG) and !4 (closer to what you and Adrian suggested, using $CC)<br>
would break the neverball use-case when used as-is, as Felix pointed out.<br>
This is because they do not have -I/usr/include/SDL2 in<br>
`sdl2-config --cflags`, so #include <SDL_ttf.h> won't work. To avoid that<br>
regression, if !5 (or !4) is still the preferred option, one option is<br>
to hard-code -I/usr/include/SDL2 in addition to -I${libdir}/SDL2.</blockquote><div dir="auto"><br></div></div><div><div dir="auto">Yes, adding the extra include path fixed the build failure.</div></div><div dir="auto"><br></div><div><div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
However, I would have expected !3 (which would now mean reverting the<br>
main commit from !5 and applying !3 instead) to work OK for neverball's<br>
use-case, because it leaves -I/usr/include/SDL2 in `sdl2-config --cflags`,<br>
the same as in the non-multiarch-capable implementation currently in<br>
the archive; so I'd be surprised if this breaks neverball. If this is<br>
the one you tested, please could you indicate the exact patch or git<br>
commit you used?<br>
<br>
When I get a chance to work on this again, I'll add an autopkgtest that<br>
does what neverball relies on, and re-test with that.<br>
<br>
I still like !3 best, but I'm happy to fix !5 (or even !4) to avoid the<br>
neverball regression if the SDL2 maintainers would prefer that.<br>
</blockquote><div dir="auto"><br></div><div dir="auto">At this point, I don’t see anything preventing us from adding the extra include path, although I’m going to do some tests builds of various SDL2-based programs to be certain.</div><div dir="auto"><br></div><div dir="auto">Neverball could also be patched to call pkg-config with SDL2_ttf, which, IIRC, has both include paths already (since SDL2_ttf requires SDL2). Unfortunately, this won’t resolve the broader problem.</div><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
> Also, this small addition has far less overhead than multiple copies<br>
> of the SDL2 headers.</blockquote><div dir="auto"><br></div><div dir="auto">I think I was commenting on the small change to the pkg-config file, but as you point out, !4 and !5 already duplicate the headers. So I was obviously thinking about something else when I wrote that.</div><div dir="auto"><br></div><div dir="auto">Hugh</div></div></div>
</div>