Fix build on Ubuntu arm64 platform

[LocutusOfBorg]

This fixes the build when GL* structs are undefined.

[SergioRAgostinho]

I have no way of checking this. Although the change seems harmless I wonder if it's the most correct one. I just it find strange that one needs to manually do it, and that Qt doesn't somehow include those for you. What error were you having from the compiler exactly?

[LocutusOfBorg]

Hi @jspricke @SergioRAgostinho

I have no way of checking this.

in a Debian/Ubuntu is trivial

pbuilder-dist yakkety arm64 create
pbuilder-dist yakkety arm64 login

(you need ubuntu-dev-tools and qemu-user-static IIRC)

Although the change seems harmless I wonder if it's the most correct one. I just it find strange that >one needs to manually do it, and that Qt doesn't somehow include those for you.

I find strange that it fails only arm64, but probably this is because arm64 has different GL implementation

What error were you having from the compiler exactly?

https://launchpad.net/ubuntu/+source/pcl/1.8.0+dfsg1-3ubuntu1

BTW, you are already doing that in your source code, according to
grep OPENGL_IS_A_FRAMEWORK . -R

You use GL structs, you need to include GL for this. Probably some QT implementations has the side effect of including it indirectly, but this isn't a reason to not include it, because the qt implementation can change or not have GL compatibility.
(not sure if the program works on arm64, BTW)

[jspricke]

Hi @LocutusOfBorg

thanks for the explanations and the patch, I've merged it already (although I wasn't able to reproduce it).

[jspricke]

..on Debian, I mean

[LocutusOfBorg]

yes, I saw the merge thanks!
I don't know, maybe some qt folks can answer this question, but I really think the arm64 support is broken in qt somewhere