cgogn / CGoGN_2

n-dimensional Meshes with Combinatorial Maps
https://cgogn.github.io
GNU Lesser General Public License v2.1
20 stars 19 forks source link

Right way of linking to Qt modules #352

Closed untereiner closed 6 years ago

untereiner commented 6 years ago
untereiner commented 6 years ago

ping @pierrekraemer or @sylvainthery

pierrekraemer commented 6 years ago

J'avais juste quelques questions, mais ne suis pas au labo pour y répondre moi-même : Est-ce qu'il n'y a pas d'autre endroit que "topology/examples" où on linkait avec Qt5Widgets ? Est-ce que c'est normal qu'on linke avec la lib de Widgets dans la lib rendering ?

untereiner commented 6 years ago

Il n'y a pas d'autre endroit. D'ailleurs je pense que ce n'est pas une bonne idée. C'est à rendering de gérer cette dépendance.. Sinon dans cgogn c'est le seul endroit ou on le fait explicitement. Par contre, je n'ai pas touché aux third parties... QOpenGLFunctions, QOpenGLBuffer et QColor sont dans Qt5::Gui donc je suppose que rendering pourrait linker contre Qt5::Gui plutôt. Reste à voir si Qt5::Widget est filé de façon transitive par QOGLWiget ou pas... Je peux essayer

untereiner commented 6 years ago

Ça a commité un truc poubelle de sous-module. Je refais une PR propre.