<html><head></head><body><div>Hello Thiago,</div><div>Le jeudi 26 octobre 2017 à 13:14 -0700, Thiago Macieira a écrit :</div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div>On Thursday, 26 October 2017 10:09:49 PDT Jeandet Alexis wrote:</div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div>Hello,</div><div><br></div><div>I already asked this on IRC but I got no answer.</div><div>On Fedora and Ubuntu I can say that "pkg-config --cflags Qt5[any</div><div>module]" does only provides include flags and no defines such as</div><div>-DQT_CORE_LIB or -DQT_GUI_LIB.</div><div><br></div><div>As an example "pkg-config --cflags panelw" gives "-D_GNU_SOURCE</div><div>-D_DEFAULT_SOURCE "</div></blockquote><div><br></div><div>panelw.pc is buggy</div></blockquote><div>In fact I took the first one returning defines :) </div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div><br></div><div>However, we should have -DQT_CORE_LIB in Qt5Core.pc. Please file an issue for </div><div>us.</div><div><br></div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div>So my questions are:</div><div>1) Is this normal/expected?</div><div>2) Why not providing this flags?</div><div>3) Does a patch to provides this flags would be accepted?</div></blockquote><div><br></div><div>Yes, a patch adding QT_${uppercaselib}_LIB to each .pc would be accepted.</div></blockquote><div>Awesome, thank you.</div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div><br></div></blockquote></body></html>