forked from qt-creator/qt-creator
Android: Use native paths format when setting an environment
ANDROID_HOME, ANDROID_SDK_ROOT and JAVA_HOME should be set as native paths in an environment to follow common practice. Change-Id: I073d125c93e27a02cc2b909e8c641ea61b3e9f8e Reviewed-by: <github-actions-qt-creator@cristianadam.eu> Reviewed-by: Assam Boudjelthia <assam.boudjelthia@qt.io>
This commit is contained in:
@@ -120,14 +120,14 @@ void AndroidToolChain::addToEnvironment(Environment &env) const
|
|||||||
env.set(QLatin1String("ANDROID_NDK_HOST"), config.toolchainHostFromNdk(m_ndkLocation));
|
env.set(QLatin1String("ANDROID_NDK_HOST"), config.toolchainHostFromNdk(m_ndkLocation));
|
||||||
const Utils::FilePath javaHome = config.openJDKLocation();
|
const Utils::FilePath javaHome = config.openJDKLocation();
|
||||||
if (javaHome.exists()) {
|
if (javaHome.exists()) {
|
||||||
env.set(QLatin1String("JAVA_HOME"), javaHome.toString());
|
env.set(QLatin1String("JAVA_HOME"), javaHome.toUserOutput());
|
||||||
const FilePath javaBin = javaHome.pathAppended("bin");
|
const FilePath javaBin = javaHome.pathAppended("bin");
|
||||||
const FilePath currentJavaFilePath = env.searchInPath("java");
|
const FilePath currentJavaFilePath = env.searchInPath("java");
|
||||||
if (!currentJavaFilePath.isChildOf(javaBin))
|
if (!currentJavaFilePath.isChildOf(javaBin))
|
||||||
env.prependOrSetPath(javaBin);
|
env.prependOrSetPath(javaBin);
|
||||||
}
|
}
|
||||||
env.set(QLatin1String("ANDROID_HOME"), config.sdkLocation().toString());
|
env.set(QLatin1String("ANDROID_HOME"), config.sdkLocation().toUserOutput());
|
||||||
env.set(QLatin1String("ANDROID_SDK_ROOT"), config.sdkLocation().toString());
|
env.set(QLatin1String("ANDROID_SDK_ROOT"), config.sdkLocation().toUserOutput());
|
||||||
}
|
}
|
||||||
|
|
||||||
bool AndroidToolChain::fromMap(const QVariantMap &data)
|
bool AndroidToolChain::fromMap(const QVariantMap &data)
|
||||||
|
|||||||
Reference in New Issue
Block a user