Merge lp:~jpakkane/unity/build-fixes2 into lp:unity
Proposed by
Jussi Pakkanen
Status: | Merged |
---|---|
Approved by: | Marco Trevisan (Treviño) |
Approved revision: | no longer in the source branch. |
Merged at revision: | 2934 |
Proposed branch: | lp:~jpakkane/unity/build-fixes2 |
Merge into: | lp:unity |
Diff against target: |
571 lines (+35/-46) 32 files modified
UnityCore/CMakeLists.txt (+1/-2) dash/CMakeLists.txt (+1/-4) dash/FilterAllButton.cpp (+1/-0) dash/FilterGenreWidget.cpp (+1/-0) dash/FilterMultiRangeWidget.cpp (+1/-0) dash/FilterRatingsWidget.cpp (+1/-0) dash/LensView.cpp (+1/-0) dash/PlacesGroup.cpp (+1/-0) dash/previews/ApplicationPreview.cpp (+2/-1) dash/previews/CMakeLists.txt (+1/-7) dash/previews/PreviewRatingsWidget.cpp (+1/-0) dash/previews/SocialPreview.cpp (+2/-0) hud/CMakeLists.txt (+1/-4) hud/HudView.cpp (+1/-0) launcher/BFBLauncherIcon.cpp (+1/-0) launcher/CMakeLists.txt (+1/-5) launcher/DesktopLauncherIcon.cpp (+1/-0) launcher/DeviceNotificationDisplayImp.cpp (+1/-0) launcher/ExpoLauncherIcon.cpp (+1/-0) launcher/LauncherController.cpp (+1/-0) launcher/SoftwareCenterLauncherIcon.cpp (+1/-0) launcher/TrashLauncherIcon.cpp (+1/-0) launcher/VolumeLauncherIcon.cpp (+1/-0) panel/CMakeLists.txt (+1/-4) panel/PanelMenuView.cpp (+1/-0) services/CMakeLists.txt (+1/-2) shortcuts/CMakeLists.txt (+1/-4) shortcuts/ShortcutHintPrivate.cpp (+1/-0) shortcuts/StandaloneShortcuts.cpp (+1/-0) unity-shared/CMakeLists.txt (+1/-3) unity-shared/TextInput.h (+2/-1) unity-standalone/CMakeLists.txt (+1/-9) |
To merge this branch: | bzr merge lp:~jpakkane/unity/build-fixes2 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Marco Trevisan (Treviño) | Approve | ||
Stephen M. Webb (community) | Needs Fixing | ||
PS Jenkins bot (community) | continuous-integration | Needs Fixing | |
Review via email: mp+130329@code.launchpad.net |
Commit message
More build fixes.
Description of the change
Second set of build fixes. This has three separate fixes.
1. GETTEXT_PACKAGE is defined in two different places: config.h and as a command line argument that is given to every single compiler invocation. Only use the first one and even then only when it is actually used.
2. Some include directories were (multiply) defined with add_definitions
3. Removed current source directory from include paths. It was added twice for each directory and both were unnecessary, since the source directory is in the search path by default.
To post a comment you must log in.
There was a fourth one as well. There were a lot of constructs like this:
target_ link_library( foo bar)
add_dependency(foo bar)
The latter is unnecessary, target_link_library sets up the dependency automatically. Removed the extra definitions.