[all-commits] [llvm/llvm-project] 42d06b: [compiler-rt][test] Change tests to remove the use...

[Harini0924 via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 42d06b8e555727e8e043d5ea9240ad103d950192
      https://github.com/llvm/llvm-project/commit/42d06b8e555727e8e043d5ea9240ad103d950192
  Author: Harini0924 <79345568+Harini0924 at users.noreply.github.com>
  Date:   2024-08-22 (Thu, 22 Aug 2024)

  Changed paths:
    M compiler-rt/test/fuzzer/afl-driver-close-fd-mask.test
    M compiler-rt/test/fuzzer/afl-driver-stderr.test

  Log Message:
  -----------
  [compiler-rt][test] Change tests to remove the use of `unset` command in lit internal shell  (#104880)

This patch rewrites tests to remove the use of the `unset` command,
which is not supported in the lit internal shell. The tests now use the
`env -u` to unset environment variables.

The `unset` command is used in shell environments to remove the
environment variable. However, because the lit internal shell does not
support the `unset` command, using it in tests would result in errors or
other unexpected behavior. To overcome this limitation, the tests have
been updated to use the `env -u` command instead. `env -u` is supported
by lit and effectively removes specified environment variables. This
allows the tests to achieve the same goal of unsetting environment
variables while ensuring compatibility with the lit internal shell.

This change is relevant for [[RFC] Enabling the Lit Internal Shell by
Default](https://discourse.llvm.org/t/rfc-enabling-the-lit-internal-shell-by-default/80179/3)
Fixes: #102397



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