Bug#798053: Bug#824079: qgis: FTBFS on non-Linux: 'TCGETS' was not declared in this scope

2016-05-13 Thread aga...@siduction.org
Hi Bas, thanks for the patch - i will integrate it upstream. > [0] > https://anonscm.debian.org/cgit/pkg-grass/qgis.git/tree/debian/patches/qtermwidget-kfreebsd.patch?h=debian/2.14.2%2bdfsg-2 > [1] https://github.com/qgis/QGIS/pull/3068 I will close this bug with the next upload - and leave the

Bug#798053: Bug#824079: qgis: FTBFS on non-Linux: 'TCGETS' was not declared in this scope

2016-05-13 Thread Sebastiaan Couwenberg
Control: tags 824079 pending On 05/13/2016 03:34 AM, Aaron M. Ucko wrote: > Sebastiaan Couwenberg writes: > >> Adding " || defined(__FreeBSD_kernel__)" to the #elif statements in >> kpty.cpp should be sufficient to not need the CMakeLists.txt changes. > > Indeed, that approach (my initial

Bug#798053: Bug#824079: qgis: FTBFS on non-Linux: 'TCGETS' was not declared in this scope

2016-05-12 Thread Aaron M. Ucko
Sebastiaan Couwenberg writes: > Adding " || defined(__FreeBSD_kernel__)" to the #elif statements in > kpty.cpp should be sufficient to not need the CMakeLists.txt changes. Indeed, that approach (my initial suggestion in #798053) also works, Alf Gaida's opening comment in

Bug#798053: Bug#824079: qgis: FTBFS on non-Linux: 'TCGETS' was not declared in this scope

2016-05-12 Thread Aaron M. Ucko
Sebastiaan Couwenberg writes: > https://quickgit.kde.org/?p=kpty.git=commitdiff=35ea45b588db9afcbd796576833ac338c6b4b8e8 > > Is this your suggested patch? And if so, can you test it on kfreebsd-*? That's a fair question. The code change works if I hardcode -DHAVE_TCGETATTR