issue #10987 Doxygen 1.11.0 for macOS reports 'sh: latex: command not found'

- Renamed EXTERNAL_TOOL_PATHS to EXTERNAL_TOOL_PATH
- Prepend directories to PATH instead of appending them.

Author: doxygen

src/config.xml

@@ -1343,16 +1343,23 @@ FILE_VERSION_FILTER = "cleartool desc -fmt \%Vn"
 ]]>
       </docs>
     </option>
-    <option type='list' id='EXTERNAL_TOOL_PATHS' format='dir' defval=''>
+    <option type='list' id='EXTERNAL_TOOL_PATH' format='dir' defval=''>
       <docs>
 <![CDATA[
- The \c EXTERNAL_TOOL_PATHS tag can be used to extend the search path (PATH environment variable)
- so that external tools such as \c latex and \c gs can be found.
+ The \c EXTERNAL_TOOL_PATH tag can be used to extend the search path (PATH environment variable)
+ so that external tools such as \c latex and \c gs can be found. 
+ \note Directories specified with EXTERNAL_TOOL_PATH are added in front of the path already specified by
+ the PATH variable, and are added in the order specified.
  \note This option is particularly useful for macOS version 14 (Sonoma) and higher,
  when running Doxygen from Doxywizard, because in this case any user-defined changes to the PATH
  are ignored. A typical example on macOS is to set
 \verbatim
-EXTERNAL_TOOL_PATHS = /Library/TeX/texbin /usr/local/bin
+EXTERNAL_TOOL_PATH = /Library/TeX/texbin /usr/local/bin
+\endverbatim
+ together with the standard path, the full search path used by doxygen when launching external tools
+ will then become
+\verbatim
+PATH=/Library/TeX/texbin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin
 \endverbatim
 ]]>
       </docs>

src/doxygen.cpp

@@ -11788,7 +11788,7 @@ void parseInput()
   AUTO_TRACE();
   std::atexit(exitDoxygen);
 
-  Portable::correctPath(Config_getList(EXTERNAL_TOOL_PATHS));
+  Portable::correctPath(Config_getList(EXTERNAL_TOOL_PATH));
 
 #if USE_LIBCLANG
   Doxygen::clangAssistedParsing = Config_getBool(CLANG_ASSISTED_PARSING);

src/portable.cpp

@@ -526,18 +526,26 @@ bool Portable::isAbsolutePath(const QCString &fileName)
 void Portable::correctPath(const StringVector &extraPaths)
 {
   QCString p = Portable::getenv("PATH");
+  bool first=true;
+  QCString result;
 #if defined(_WIN32) && !defined(__CYGWIN__)
-  QCString result = substitute(p,"/","\\");
   for (const auto &path : extraPaths)
   {
-    result+=";\""+substitute(QCString(path),"/","\\")+"\"";
+    if (!first) result+=';';
+    first=false;
+    result += substitute(QCString(path),"/","\\");
   }
+  if (!result.isEmpty() && !p.isEmpty()) result+=';';
+  result += substitute(p,"/","\\");
 #else
-  QCString result=p;
   for (const auto &path : extraPaths)
   {
-    result+=":"+QCString(path);
+    if (!first) result+=':';
+    first=false;
+    result += QCString(path);
   }
+  if (!result.isEmpty() && !p.isEmpty()) result+=':';
+  result += p;
 #endif
   if (result!=p) Portable::setenv("PATH",result.data());
   //printf("settingPath(%s) #extraPaths=%zu\n",Portable::getenv("PATH").data(),extraPaths.size());