Make sure that if everything fails, to check /etc/dc for thedc-binary.

DownloadDCFirmware/main.cpp

@@ -134,10 +134,13 @@ int main(int argc, char **argv) {
     QString const &customerRepo
         = QDir::cleanPath(workingDir + QDir::separator() + QString("customer_%1").arg(customerNr));
 */
-    QString const &psaDcDir = internal::customerRepoDcDir();
+    QString psaDcDir = internal::customerRepoDcDir();
-    QString const &psaRepoRootDir = internal::customerRepoRoot();
+    QString psaRepoRootDir = internal::customerRepoRoot();
-    QString const &psaRepoDir = internal::customerRepoDir();
+    QString psaRepoDir = internal::customerRepoDir();
-    QString const &branchName = internal::branchName();
+    QString branchName = internal::branchName();
+    QString rootDir = "/etc/dc/";
+    // bool usbDetected = false;
+    // bool sdCardDetected = false;
 
     bool debug = false;
     bool noaction = true;
@@ -151,8 +154,17 @@ int main(int argc, char **argv) {
         workingDir = settings->value("workingdir", "/tmp").toString();
         libca = settings->value("libca", "/usr/lib/libCAslave.so").toString();
         settings->endGroup();
+    } else {
+        qCritical() << "count not read ATBUpdateTool.ini";
+        return -1;
     }
 
+    QString dcDir = parser.dcDir();
+    if (!dcDir.isEmpty()) {
+        psaDcDir = dcDir;
+    }
+    debug = parser.debug();
+
     if (debug) {
         qInfo() << "           using dc directory: " << psaDcDir;
         qInfo() << "using psa repo root directory: " << psaRepoRootDir;
@@ -164,26 +176,33 @@ int main(int argc, char **argv) {
     std::optional<QString> mountPath = System::checkForUSBStick(psaDcDir);
     QFileInfo fi;
     if (mountPath.has_value()) {
+        // usbDetected = true;
+        // mountPath/etc/dc must exists !
+        // cannot override with commandline parameter --dc-directory
         if (debug) {
             qInfo() << "using mount path (USB): " << mountPath.value();
         }
     } else
     if ((mountPath = System::checkForSDCard(psaDcDir)).has_value()) {
+        // sdCardDetected = true;
+        // mountPath/etc/dc must exists !
+        // cannot override with commandline parameter --dc-directory
         if (debug) {
             qInfo() << "using mount path (SD): " << mountPath.value();
         }
     } else {
+        rootDir = "";
+        mountPath = QDir::cleanPath(psaRepoRootDir + QDir::separator() + psaDcDir);
         if (debug) {
-            qInfo() << "using customer repository" << psaRepoDir;
+            qInfo() << "using as dc-directory" << mountPath.value();
         }
-        mountPath = "";
     }
 
     if (debug) {
-        qInfo() << "using mount path: " << mountPath.value();
+        qInfo() << "using as mount path: " << mountPath.value();
     }
 
-    std::unique_ptr<QString> c = internal::dcCandidateToInstall("/etc/dc/", mountPath.value());
+    std::unique_ptr<QString> c = internal::dcCandidateToInstall(rootDir, mountPath.value());
     if (c) {
         fi.setFile(*c);
         if (fi.exists() == false) {

UpdatePTUDevCtrl/main.cpp

@@ -74,6 +74,7 @@
  * - \page ATBUpdateSync
  * - \page ATBUpdateOpkg
  * - \page ATBUpdateDC
+ *      Typical call: ./ATBUpdateDC --dc-directory /etc/dc/ --debug true
  * - \page ATBUpdateJsonFiles
  * - \page ATBUpdateShow
  */

common/src/utils_internal.cpp

@@ -62,7 +62,7 @@ QString branchName() {
 }
 
 std::unique_ptr<QSettings> readSettings(QString const &optionalDirName) {
-    std::unique_ptr<QSettings> settings{std::make_unique<QSettings>()};
+    std::unique_ptr<QSettings> settings;
 
     //QString const fileName{settings->applicationName() + ".ini"};
     QString const fileName{"ATBUpdateTool.ini"};
@@ -71,33 +71,17 @@ std::unique_ptr<QSettings> readSettings(QString const &optionalDirName) {
     if (!optionalDirName.isEmpty()) {
         d = QDir{optionalDirName};
         if (d.exists()) { // try to find ini-file under optionalDirname
-            QFileInfo fi{d, optionalDirName};
+            QFileInfo fi{QDir::cleanPath(optionalDirName + QDir::separator() + fileName)};
             if (fi.exists()) {
                 settings.reset(new QSettings(fi.absoluteFilePath(), QSettings::IniFormat));
                 return settings;
             } else {
-                qCritical().noquote() << "read" << fileName << ":" << fi.absoluteFilePath() << "not found."
+                qCritical().noquote() << "read" << fileName << ":" << fi.absoluteFilePath() << "not found.";
-                            << "Try" << internal::DEFAULT_INI_DIR;
             }
         } else {
-            qCritical().noquote() << "read" << fileName << ":" << optionalDirName << "not found."
+            qCritical().noquote() << optionalDirName << "not found. Try" << internal::DEFAULT_INSTALL_DIR;
-                        << "Try" << internal::DEFAULT_INSTALL_DIR;
         }
     }
-    d = internal::DEFAULT_INI_DIR;
-    if (d.exists()) { // try to find ini-file under /etc/tools/atbupdate
-        QFileInfo fi{d, fileName};
-        if (fi.exists()) {
-            settings.reset(new QSettings(fi.absoluteFilePath(), QSettings::IniFormat));
-            return settings;
-        } else {
-            qCritical().noquote() << "read" << fileName << ":" << fi.absoluteFilePath() << "not found."
-                        << "Try" << internal::DEFAULT_INSTALL_DIR;
-        }
-    } else {
-        qCritical().noquote() << "read" << fileName << ":" << internal::DEFAULT_INI_DIR << "not found."
-                    << "Try" << internal::DEFAULT_INSTALL_DIR;
-    }
     d = QDir{internal::DEFAULT_INSTALL_DIR};
     if (d.exists()) { // try to find ini-file under /opt/app/tools/atbupdate
         QFileInfo fi{d, fileName};
@@ -108,12 +92,27 @@ std::unique_ptr<QSettings> readSettings(QString const &optionalDirName) {
             qCritical().noquote() << "read" << fileName << ":" << fi.absoluteFilePath() << "not found.";
         }
     } else {
-        qCritical().noquote() << "read" << fileName << ":" << internal::DEFAULT_INSTALL_DIR << "not found.";
+        qCritical().noquote() << internal::DEFAULT_INSTALL_DIR << "not found.";
     }
 
     return settings;
 }
 
+/*! \brief Find dc-binary to install.
+ *
+ *  If rootDir is empty, and if dcDirectory is empty as well, use etc/dc in
+ *  custome repository (e,.g. /opt/app/tools/atbupdate/customer_337/etc/dc).
+ *  If dcDirectory is not empty, then use it.
+ *  If rootDir is \b not empty,check if it alreay ends with "/etc/dc/". in this
+ *  case, use rootDir as dcDir. Otherwise, if dcDirectory is empty, use rootDir/etc/dc
+ *  (e,.g. /opt/app/tools/atbupdate/customer_337/etc/dc). If dcDirectory
+ *  is not empty, use roorDir/dcDirectory.
+ *  As last resort, try "/etc/dc".
+ *
+ *  \param[in] dcDirectory directory of device controller binary. Maybe empty.
+ *  \param[in] rootDir root directory for dcDirectory. Might be empty.
+ *  \return name of the found candidate, or null if not found.
+ */
 std::unique_ptr<QString> dcCandidateToInstall(QString const &dcDirectory,
                                               QString const &rootDir) {
     std::unique_ptr<QString> dcCandidate{nullptr};
@@ -125,18 +124,22 @@ std::unique_ptr<QString> dcCandidateToInstall(QString const &dcDirectory,
             dcDir = customerRepoDcDir();
         } else {
             dcDir = dcDirectory;
-
         }
     } else {
-        if (dcDirectory.isEmpty()) {
+        if (!rootDir.endsWith("/etc/dc") && !rootDir.endsWith("/etc/dc/")) {
-            dcDir = QDir::cleanPath(rootDir + QDir::separator() + customerRepoDcDir());
+            if (dcDirectory.isEmpty()) {
+                dcDir = QDir::cleanPath(rootDir + QDir::separator() + "etc/dc/");
+            } else {
+                dcDir = QDir::cleanPath(rootDir + QDir::separator() + dcDirectory);
+            }
         } else {
-            dcDir = QDir::cleanPath(rootDir + QDir::separator() + dcDirectory);
+            dcDir = QDir::cleanPath(rootDir + QDir::separator());
         }
     }
 
     if (dcDir.exists()) {
-        qCritical() << __func__ << __LINE__ << dcDir;
+
+        qInfo() << "use" << dcDir.absolutePath() << "as directory (containing dc-binary)";
 
         QFileInfoList fileInfoList =
             dcDir.entryInfoList(QStringList("*.bin"),
@@ -144,7 +147,6 @@ std::unique_ptr<QString> dcCandidateToInstall(QString const &dcDirectory,
 
         QFileInfo dc2cbin{dcDir.absoluteFilePath("dc2c.bin")};
 
-
         if (dc2cbin.exists()) {
 
             QCryptographicHash md5gen(QCryptographicHash::Md5);
@@ -176,7 +178,11 @@ std::unique_ptr<QString> dcCandidateToInstall(QString const &dcDirectory,
             }
         }
     } else {
-        qCritical() << __func__ << __LINE__ << dcDir << "does not exist";
+        qCritical() << dcDir.absolutePath() << "does not exist";
+        if (dcDir.absolutePath() != QDir("/etc/dc/").absolutePath()) {
+            qInfo() << __func__ << __LINE__ << "Try /etc/dc/";
+            return dcCandidateToInstall("/etc/dc/", "");
+        }
     }
 
     return dcCandidate;