Merge branch 'master' into catalyst_api_2_structured_blocks

.github/workflows/build_external_lib.yml

@@ -9,6 +9,10 @@ on:
     - master
   workflow_dispatch:
 
+concurrency:  
+  group: ${{ github.workflow}}-${{ github.head_ref }}  
+  cancel-in-progress: true
+
 jobs:
 
   build-deps:

.github/workflows/build_netcdf_no_hdf5.yml

@@ -10,6 +10,10 @@ on:
     branches:
     - master
 
+concurrency:  
+  group: ${{ github.workflow}}-${{ github.head_ref }}  
+  cancel-in-progress: true
+
 jobs:
 
   build-deps:

.github/workflows/build_test.yml

@@ -8,6 +8,10 @@ on:
     branches:
     - master
 
+concurrency:  
+  group: ${{ github.workflow}}-${{ github.head_ref }}  
+  cancel-in-progress: true
+
 jobs:
 
   build-deps:

.github/workflows/build_variant.yml

@@ -10,8 +10,11 @@ on:
     branches:
     - master
 
-jobs:
+concurrency:  
+  group: ${{ github.workflow}}-${{ github.head_ref }}  
+  cancel-in-progress: true
 
+jobs:
   build-deps:
 
     runs-on: ubuntu-latest

.github/workflows/docker-exodus.yml

@@ -2,6 +2,11 @@ name: docker-exodus
 on:
   push:
     branches: [ "master" ]
+
+concurrency:  
+  group: ${{ github.workflow}}-${{ github.head_ref }}  
+  cancel-in-progress: true
+
 jobs:
   build-latest:
     runs-on: ubuntu-latest

.github/workflows/docker-seacas.yml

@@ -2,6 +2,11 @@ name: docker-seacas
 on:
   push:
     branches: [ "master" ]
+
+concurrency:  
+  group: ${{ github.workflow}}-${{ github.head_ref }}  
+  cancel-in-progress: true
+
 jobs:
   build-latest:
     runs-on: ubuntu-latest

.github/workflows/intel-build.yml

@@ -8,6 +8,11 @@ on:
     branches:
     - master
 
+
+concurrency:  
+  group: ${{ github.workflow}}-${{ github.head_ref }}  
+  cancel-in-progress: true
+
 jobs:
   build-deps:
 
@@ -25,6 +30,14 @@ jobs:
         shell: bash --noprofile --norc {0}
 
     steps:
+    - name: Maximize build space
+      uses: easimon/maximize-build-space@master
+      with:
+        root-reserve-mb: 30000
+        remove-dotnet: 'true'
+        remove-android: 'true'
+        remove-haskell: 'true'
+        remove-codeql:  'true'
     - uses: actions/checkout@v3
     - name: setup repo
       run: |
@@ -79,6 +92,14 @@ jobs:
         cgns: [ 4.3.0 ]
     steps:
 
+      - name: Maximize build space
+        uses: easimon/maximize-build-space@master
+        with:
+          root-reserve-mb: 30000
+          remove-dotnet: 'true'
+          remove-android: 'true'
+          remove-haskell: 'true'
+          remove-codeql:  'true'
       - uses: actions/checkout@v3
       - name: setup repo
         run: |

.github/workflows/msys2.yml

@@ -7,6 +7,10 @@ on:
     branches:
     - master
 
+concurrency:  
+  group: ${{ github.workflow}}-${{ github.head_ref }}  
+  cancel-in-progress: true
+
 jobs:
   build:
     runs-on: windows-latest

TPLsList.cmake

@@ -53,6 +53,9 @@
 #
 
 SET( Seacas_TPLS_FINDMODS_CLASSIFICATIONS
+  X11             "cmake/TPLs/"    ST
+  DLlib           "cmake/TPLs/"    ST
+  fmt             "cmake/TPLs/"    ST
   GTest           "cmake/TPLs/"    PT
   Zlib            "cmake/TPLs/"    ST
   Pthread         "cmake/TPLs/"    PT
@@ -66,10 +69,7 @@ SET( Seacas_TPLS_FINDMODS_CLASSIFICATIONS
   ParMETIS        "cmake/TPLs/"    ST
   Pamgen          "cmake/TPLs/"    ST
   Matio           "cmake/TPLs/"    ST
-  X11             "cmake/TPLs/"    ST
-  DLlib           "cmake/TPLs/"    ST
   CUDA            "cmake/TPLs/"    ST
-  fmt             "cmake/TPLs/"    ST
   Kokkos          "${${PROJECT_NAME}_TRIBITS_DIR}/common_tpls/"  ST
   Faodel          "${PROJECT_SOURCE_DIR}/packages/seacas/cmake/tpls/"    ST
   Cereal          "${PROJECT_SOURCE_DIR}/packages/seacas/cmake/tpls/"    ST

cmake-config

@@ -540,8 +540,7 @@ ${FAODEL_SYMBOLS} \
 -D Metis_ROOT:PATH=${METIS_PATH} \
 -D ParMETIS_ROOT:PATH=${PARMETIS_PATH} \
 -D PNetCDF_ROOT:PATH=${PNETCDF_PATH} \
--D fmt_LIBRARY_DIRS:PATH=${FMT_PATH}/lib \
--D fmt_INCLUDE_DIRS:PATH=${FMT_PATH}/include \
+-D fmt_ROOT:PATH=${FMT_PATH} \
 $EXTRA_ARGS \
 ${ACCESS}
 

cmake/TPLs/FindTPLfmt.cmake

@@ -53,8 +53,15 @@
 # ************************************************************************
 # @HEADER
 
+message("-- Using find_package(fmt CONFIG) ...")
+find_package(fmt CONFIG)
+if (fmt_FOUND)
+  message("-- Found fmt_CONFIG=${fmt_CONFIG}")
+  message("-- Generating fmt::all_libs and fmtConfig.cmake")
+  tribits_extpkg_create_imported_all_libs_target_and_config_file(
+    fmt
+    INNER_FIND_PACKAGE_NAME  fmt
+    IMPORTED_TARGETS_FOR_ALL_LIBS  fmt::fmt)
+  set(TPL_fmt_NOT_FOUND FALSE)
+endif()
 
-TRIBITS_TPL_FIND_INCLUDE_DIRS_AND_LIBRARIES( fmt
-  REQUIRED_HEADERS fmt/format.h
-  REQUIRED_LIBS_NAMES  fmt
-  )

cmake/tribits/CHANGELOG.md

@@ -2,43 +2,6 @@
 ChangeLog for TriBITS
 ----------------------------------------
 
-## 2023-06-22:
-
-* **Added:** Packages are now determined to be missing if their dependencies
-  file `<packageDir>/cmake/Dependencies.cmake` is missing.  If the package
-  directory `<packageDir>` exists but the dependencies file is missing, the
-  package is determined to be missing but a warning is printed.  (This expands
-  behavior to gracefully deal with a situation where a package source
-  directory is only partially removed, such as with `git rm -r <packageDir>`,
-  but the base directory still exists.  Therefore, this allows the project to
-  gracefully configure with the package being considered missing and avoids a
-  fatal error in this case.)
-
-## 2023-06-02:
-
-* **Added/Deprecated:** External packages/TPLs can now be (and should be)
-  listed in the `[TEST|LIB]_[REQUIRED|OPTIONAL]_PACKAGES` arguments/lists in
-  the macro `tribits_package_define_dependencies()` and the
-  `[TEST|LIB]_[REQUIRED|OPTIONAL]_TPLS` arguments/lists are deprecated (but
-  with no deprecation warning yet).  This makes it easier to write
-  `<packageDir>/cmake/Dependencies.cmake` files for packages where the set of
-  internal and external upstream dependent packages is dynamic and changes
-  depending on the TriBITS project where these package are configured under.
-  (And conceptually, a downstream package should not care if an upstream
-  dependent package is pulled in as an external package or built as an
-  internal package.)
-
-## 2023-05-03:
-
-* **Added:** Added support for non-fully TriBITS-compatible external packages.
-  Now, a `<Package>Config.cmake` file need not define
-  `<UpstreamPkg>::all_libs` targets for all of its upstream dependencies.  The
-  updated macro `tribits_process_enabled_tpls()` will find any missing
-  upstream external packages/TPLs as needed (see updated documentation in the
-  section "TriBITS-Compliant External Packages" in the "TriBITS Users Guide"
-  and the section "Processing of external packages/TPLs and TriBITS-compliant
-  external packages" in the "TriBITS Maintainers Guide").
-
 ## 2023-02-24:
 
 * **Changed:** Upgraded minimum required CMake version from 3.17 to 3.23.

cmake/tribits/README.DIRECTORY_CONTENTS.rst

@@ -22,10 +22,9 @@ TriBITS refactorings of TriBITS.
 
 .. _TriBITS Core:
 
-**core/**: Core TriBITS test support and package-based architecture for CMake
-projects. This only depends on raw CMake and contains just the minimal support
-for building, testing, installing, and deployment.  This CMake code depends
-only on CMake and nothing else.
+**core/**: Core TriBITS package-based architecture for CMake projects. This
+only depends on raw CMake and contains just the minimal support for building,
+testing, installing, and deployment.  Only depends on CMake and nothing else.
 
 **python_utils/**: Some basic Python utilities that are not specific to
 TriBITS but are used in TriBITS CI and testing support software.  There are
@@ -87,49 +86,3 @@ subdirectory. It supports the argument ``--components`` with values ``core``,
 * ``examples`` => (external tribits installation)
 * ``doc`` => ``core``, ``ci_support``, ``examples``
 * ``devtools_install`` => ``python_utils``
-
-
-TriBITS Core Directory Contents
-...............................
-
-The TriBITS ``core/`` directory is broken down into several subdirectories of
-its own:
-
-**core/utils**: General CMake utilities that are not specific to the TriBITS
-system and can be reused in any CMake project.
-
-**core/common**: As small set of common modules that the different TriBITS
-Core module files in different directories depend on.  These include things
-like common TriBITS constants and TriBITS CMake policies.
-
-**core/test_support**: Modules that help define CTest tests using functions
-like `tribits_add_test()`_ and `tribits_add_advanced_test()`_.  These can be
-used in CMake projects that are not full-blown TriBITS projects.
-
-**core/config_tests**: Some basic configure-time tests used by the TriBITS
-package architecture framework.
-
-**core/std_tpls**: Some ``Find<tplName>.cmake`` files for key external
-dependencies handled as TriBITS TPLs but are more central to the TriBITS
-system.  (Examples include CUDA and MPI support.)
-
-**core/installation**: A collection of ``*.cmake.in`` and related Cmake code
-supporting installations.
-
-**core/package_arch**: Modules for the full-blown TriBITS package architecture
-framework including package dependency management, multi-repository support,
-installations (including the generation of ``<Package>Config.cmake`` files),
-etc.
-
-The dependencies between these different TriBITS `core` subdirectories are:
-
-* ``core/utils`` => (external CMake)
-* ``core/common`` => ``core/utils``
-* ``core/test_support`` =>   ``core/utils``, ``core/common``
-* ``core/config_tests`` => (external CMake)
-* ``core/std_tpls`` => (external CMake)
-* ``core/installation`` <=> ``core/package_arch`` (bidirectional)
-* ``core/package_arch`` => ``core/utils``, ``core/common``,
-  ``core/test_support``, ``core/config_tests``, ``core/std_tpls``,
-  ``core/installation``
-

cmake/tribits/ci_support/CDashQueryAnalyzeReport.py

@@ -40,11 +40,9 @@
 try:
   # Python 2
   from urllib2 import urlopen
-  from urllib2 import quote as urlquote
 except ImportError:
   # Python 3
   from urllib.request import urlopen
-  from urllib.parse import quote as urlquote
 
 import sys
 import hashlib
@@ -718,45 +716,41 @@ def getAndCacheCDashQueryDataOrReadFromCache(
   return cdashQueryData
 
 
-def normalizeUrlStrings(*args):
-  return [urlquote(x) for x in args]
-
-
 # Construct full cdash/api/v1/index.php query URL to pull data down given the
 # pieces
 def getCDashIndexQueryUrl(cdashUrl, projectName, date, filterFields):
-  # for legacy reasons, this function assumes we normalized projectName
-  projectName, = normalizeUrlStrings(projectName,)
   if date: dateArg = "&date="+date
   else: dateArg = ""
   return cdashUrl+"/api/v1/index.php?project="+projectName+dateArg \
-      + "&"+filterFields
+    + "&"+filterFields
 
 
 # Construct full cdash/index.php browser URL given the pieces
 def getCDashIndexBrowserUrl(cdashUrl, projectName, date, filterFields):
-  # for legacy reasons, this function assumes we normalized projectName
-  projectName, = normalizeUrlStrings(projectName,)
   if date: dateArg = "&date="+date
   else: dateArg = ""
   return cdashUrl+"/index.php?project="+projectName+dateArg \
-      + "&"+filterFields
+    + "&"+filterFields
 
 
 # Construct full cdash/api/v1/queryTests.php query URL given the pieces
 def getCDashQueryTestsQueryUrl(cdashUrl, projectName, date, filterFields):
-  # for legacy reasons, this function assumes we normalized projectName
-  projectName, = normalizeUrlStrings(projectName,)
   if date: dateArg = "&date="+date
   else: dateArg = ""
   cdashTestUrl = cdashUrl+"/api/v1/queryTests.php?project="+projectName+dateArg+"&"+filterFields
-  return cdashTestUrl
+  return replaceNonUrlCharsInUrl(cdashTestUrl)
+
+
+# Replace non-URL chars and return new URL string
+def replaceNonUrlCharsInUrl(url):
+  urlNew = url
+  urlNew = urlNew.replace(' ', '%20')
+  # ToDo: Replace other chars as needed
+  return urlNew
 
 
 # Construct full cdash/queryTests.php browser URL given the pieces
 def getCDashQueryTestsBrowserUrl(cdashUrl, projectName, date, filterFields):
-  # for legacy reasons, this function assumes we normalized projectName
-  projectName, = normalizeUrlStrings(projectName,)
   if date: dateArg = "&date="+date
   else: dateArg = ""
   return cdashUrl+"/queryTests.php?project="+projectName+dateArg+"&"+filterFields
@@ -1707,33 +1701,29 @@ def __call__(self, testDict):
     dateRangeEndDateStr = self.__date
     beginEndUrlFields = "begin="+dateRangeBeginDateStr+"&end="+dateRangeEndDateStr
 
-    # normalize names for query
-    projectName_url, buildName_url, testname_url, site_url = normalizeUrlStrings(
-      projectName, buildName, testname, site)
-
     # Define queryTests.php query filters for test history
     testHistoryQueryFilters = \
       beginEndUrlFields+"&"+\
       "filtercombine=and&filtercombine=&filtercount=3&showfilters=1&filtercombine=and"+\
-      "&field1=buildname&compare1=61&value1="+buildName_url+\
-      "&field2=testname&compare2=61&value2="+testname_url+\
-      "&field3=site&compare3=61&value3="+site_url
+      "&field1=buildname&compare1=61&value1="+buildName+\
+      "&field2=testname&compare2=61&value2="+testname+\
+      "&field3=site&compare3=61&value3="+site
 
     # URL used to get the history of the test in JSON form
     testHistoryQueryUrl = \
-      getCDashQueryTestsQueryUrl(cdashUrl, projectName_url, None, testHistoryQueryFilters)
+      getCDashQueryTestsQueryUrl(cdashUrl, projectName, None, testHistoryQueryFilters)
 
     # URL to embed in email to show the history of the test to humans
     testHistoryBrowserUrl = \
-      getCDashQueryTestsBrowserUrl(cdashUrl, projectName_url, None, testHistoryQueryFilters)
+      getCDashQueryTestsBrowserUrl(cdashUrl, projectName, None, testHistoryQueryFilters)
 
     # URL for to the build summary on index.php page
     buildHistoryEmailUrl = getCDashIndexBrowserUrl(
-      cdashUrl, projectName_url, None,
+      cdashUrl, projectName, None,
       beginEndUrlFields+"&"+\
       "filtercombine=and&filtercombine=&filtercount=2&showfilters=1&filtercombine=and"+\
-      "&field1=buildname&compare1=61&value1="+buildName_url+\
-      "&field2=site&compare2=61&value2="+site_url
+      "&field1=buildname&compare1=61&value1="+buildName+\
+      "&field2=site&compare2=61&value2="+site
       )
     # ToDo: Replace this with the the URL to just this one build the index.php
     # page.  To do that, get the build stamp from the list of builds on CDash