johnnyzhangzhao / monav

Automatically exported from code.google.com/p/monav
0 stars 0 forks source link

Pull Request: NOGUI qmake Improvement #59

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
I've had trouble building the routing daemon because the nogui config was not 
being passed around to some of the subprojects causing them to be built with 
calls into QtGui. This created a problem when linking because the QtGui library 
was not linked in.

To get around this I created additional pro files that specifically defined the 
nogui config option and specified that those be used to build the routing 
daemon.

This has the side effect that plugins are built twice -- one gui version and 
one non-gui version. However, the major benefit is that users can use qmake 
directly without specifying defines manually.

I've committed to my clone of monav:

http://code.google.com/r/adob321-monav/source/detail?r=babf3c6fdb8b50978acfeeed6
7136396e3f0ac51

repo: https://adob321@code.google.com/r/adob321-monav/ 
rev: babf3c6fdb8b

Original issue reported on code.google.com by adob...@gmail.com on 19 Jul 2011 at 7:33

GoogleCodeExporter commented 8 years ago
why not symply add:
CONFIG += nogui

in monavroutingdaemon.pro - seems all other project files can handle it. (trunk)

Original comment by chake.h...@gmail.com on 20 Jul 2011 at 4:08

GoogleCodeExporter commented 8 years ago
Adding a CONFIG += nogui to monavroutingdaemon.pro will have not work because 
CONFIG properties are not passed down to the subprojects.

Original comment by adob...@gmail.com on 20 Jul 2011 at 5:22

GoogleCodeExporter commented 8 years ago
I like the solution, thanks. You have to build the plugins twice anyhow. It 
always bugged me, that we could not pass down variable's contents.

I will also use this for the preprocessor.

Original comment by veaac.fd...@gmail.com on 21 Jul 2011 at 3:59