force add /usr/include for protopath

machinekit / QtQuickVcp

A Virtual Control Panel for Machinekit written in Qt/C++/QML

Other

128 stars 74 forks source link

force add /usr/include for protopath #200

Closed melnaquib closed 6 years ago

melnaquib commented 6 years ago

build fails sayinf protocc cannot find .proto files

machinekoder commented 6 years ago

Thank you for the PR. Can you please separate the QUrl fix and the protopath fix.

I removed the PROTOPATH link to /usr/include some commits back since it causes problems on Debian Stretch. This is due to the fact that the /usr/include path is then added twice to the include paths which leads to inversion of the include order - so the compiler starts to include wrong files.

Can you describe on which distribution you experience the problems?

machinekoder commented 6 years ago

Here is the line in the paths file: https://github.com/qtquickvcp/QtQuickVcp/blob/c0949184a634b9d4cdcc46ec6c9bd5a29da7c3a1/paths.pri#L57-L58

machinekoder commented 6 years ago

I close this PR now since it should be fixed with https://github.com/qtquickvcp/QtQuickVcp/pull/201