feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity

McuSupport: Use env variable as package's primary value

Browse Source 
Use env variable value as primary source before values from settings
and fallback value from JSON.

Change-Id: I0c0a4c2cdac65a075d9f38aa1557a9d303b30a5e
Reviewed-by: Yasser Grimes <yasser.grimes@qt.io>
Reviewed-by: <github-actions-qt-creator@cristianadam.eu>
Reviewed-by: Dawid Śliwa <dawid.sliwa@qt.io>
Reviewed-by: hjk <hjk@qt.io>
This commit is contained in:
Piotr Mućko
2022-07-14 15:44:54 +02:00
parent 23d9ef1adb
commit ca9e652716
2 changed files with 8 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/plugins/mcusupport/mcupackage.cpp
View File

@@ -75,8 +75,12 @@ McuPackage::McuPackage(const SettingsHandler::Ptr &settingsHandler,
, m_downloadUrl(downloadUrl)
, m_addToSystemPath(addToSystemPath)
{
m_path = FilePath::fromString(
qEnvironmentVariable(m_environmentVariableName.toStdString().c_str()));
if (!m_path.exists()) {
m_path = this->settingsHandler->getPath(settingsKey, QSettings::UserScope, m_defaultPath);
}
}
QString McuPackage::label() const
{
@@ -125,7 +129,7 @@ FilePath McuPackage::path() const
FilePath McuPackage::defaultPath() const
{
return m_defaultPath;
return m_defaultPath.cleanPath();
}
FilePath McuPackage::detectionPath() const

3
src/plugins/mcusupport/test/armgcc_nxp_1050_json.h
View File

@@ -81,7 +81,7 @@ constexpr auto armgcc_nxp_1050_json = R"({
"label": "CMake Toolchain File",
"cmakeVar": "CMAKE_TOOLCHAIN_FILE",
"type": "file",
"defaultValue": "/opt/qtformcu/2.2/lib/cmake/Qul/toolchain/armgcc.cmake",
"defaultValue": "$Qul_ROOT//lib/cmake/Qul/toolchain/armgcc.cmake",
"visible": false,
"optional": false
}
@@ -109,6 +109,7 @@ constexpr auto armgcc_nxp_1050_json = R"({
"id": "NXP_FREERTOS_DIR",
"label": "FreeRTOS SDK for MIMXRT1050-EVK",
"cmakeVar": "FREERTOS_DIR",
"envVar": "IMXRT1050_FREERTOS_DIR",
"defaultValue": "$QUL_BOARD_SDK_DIR/rtos/freertos/freertos_kernel",
"type": "path",
"optional": false