[all-commits] [llvm/llvm-project] 9fb237: [libcxx] [test] Merge the MinGW static/shared test...

[Martin Storsjö via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 9fb2378ad52927cafa918b51f2054c47108f2f24
      https://github.com/llvm/llvm-project/commit/9fb2378ad52927cafa918b51f2054c47108f2f24
  Author: Martin Storsjö <martin at martin.st>
  Date:   2024-10-10 (Thu, 10 Oct 2024)

  Changed paths:
    M libcxx/CMakeLists.txt
    A libcxx/test/configs/llvm-libc++-mingw.cfg.in
    R libcxx/test/configs/llvm-libc++-shared-mingw.cfg.in
    R libcxx/test/configs/llvm-libc++-static-mingw.cfg.in

  Log Message:
  -----------
  [libcxx] [test] Merge the MinGW static/shared test config files (#111759)

These were split in 0e8208eca1c316b7302de7803ab0d85a1dd77076, with the
only functional difference between them at the time being `--prepend_env
PATH=%{lib-dir}` in the static config and `--prepend_env
PATH=%{install-prefix}/bin` in the shared library config.

However this difference is unnecessary - the static library config
doesn't need any `--prepend_env` argument at all. Before
0e8208eca1c316b7302de7803ab0d85a1dd77076, both configurations used the
same config file, where the `--prepend_env` argument was unnecessary but
benign in the static case.

Reduce the unnecessary config duplication in this case, and return these
configs to using one single config file for both setups.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications