Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

compilation flags #13

Merged
merged 2 commits into from Jan 8, 2013
Merged

compilation flags #13

merged 2 commits into from Jan 8, 2013

Conversation

ftonello
Copy link
Contributor

@ftonello ftonello commented Jan 8, 2013

Guys,

This patch is useful when cross-compiling and not want to compile everything.

BTW, I'm about to send the OpenEmbedded recipe for libconnman-qt and this patch is necessary.

Also, we kind need to have a better doc. What do you guys think?

Felipe

Felipe F. Tonello added 2 commits January 7, 2013 17:24
This is usefull when there is no need to compile test program and qml plugin.

Signed-off-by: Felipe F. Tonello <ftonello@cercacor.com>
Signed-off-by: Felipe F. Tonello <ftonello@cercacor.com>
@rburchell
Copy link
Contributor

looks ok to me. i'd personally prefer !contains(CONFIG, notest): SUBDIRS += tests - style, but that's just a nitpick

what sort of docs are you thinking?

@lpotter
Copy link
Contributor

lpotter commented Jan 8, 2013

Looks fine to me.

rburchell added a commit that referenced this pull request Jan 8, 2013
@rburchell rburchell merged commit 06f3b03 into nemomobile:master Jan 8, 2013
@ftonello
Copy link
Contributor Author

ftonello commented Jan 8, 2013

I was thinking in something like a qmake help=1 command to show some help. Maybe it's not necessary for now.

@ftonello ftonello deleted the pro-flags branch January 8, 2013 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants