Skip to content

Commit

Permalink
Revert "don't overwrite pre-existing INSTALL for target"
Browse files Browse the repository at this point in the history
This reverts commit c0e94dd, as it
introduced a regression for applications that sets an installation
target (on Android), which a lot of our examples do. The installation
target for Android applications/libraries needs to be within in the
application bundle's directory tree, or it won't work.

Task-number: QTBUG-61635
Change-Id: I8c919ef3888d7679b0f9659796f5e590bc1faa57
Reviewed-by: Eskil Abrahamsen Blomfeldt <[email protected]>
  • Loading branch information
cstatic authored and aksalova committed Jun 27, 2017
1 parent 0c02106 commit 11d36bd
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions mkspecs/features/android/android.prf
Original file line number Diff line number Diff line change
@@ -1,6 +1,3 @@
android_install:contains(INSTALLS, target):!isEmpty(target.path): \
CONFIG -= android_install

contains(TEMPLATE, ".*app") {
!android_app {
!contains(TARGET, ".so"): TARGET = lib$${TARGET}.so
Expand All @@ -11,7 +8,7 @@ contains(TEMPLATE, ".*app") {
INSTALLS *= target
}
}
} else: contains(TEMPLATE, "lib"):!static:android_install {
} else: contains(TEMPLATE, "lib"):!static:!QTDIR_build:android_install {
target.path = /libs/$$ANDROID_TARGET_ARCH/
INSTALLS *= target
}

0 comments on commit 11d36bd

Please sign in to comment.