Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Rolling release] Do not forward legacy cmd.exe variables to the server #24500

Conversation

keertk
Copy link
Member

@keertk keertk commented Nov 26, 2024

This fixes startup failures such as:

ERROR: While parsing option --client_env==C:=C:\Users\wyv: Variable definitions must be in the form of a 'name=value' assignment

Fixes #24448

Closes #24451.

PiperOrigin-RevId: 699115724
Change-Id: Id7a2445e82b24384e9ade0c92ddb343f00e9ab2b

This fixes startup failures such as:
```
ERROR: While parsing option --client_env==C:=C:\Users\wyv: Variable definitions must be in the form of a 'name=value' assignment
```

Fixes #24448

Closes #24451.

PiperOrigin-RevId: 699115724
Change-Id: Id7a2445e82b24384e9ade0c92ddb343f00e9ab2b
@keertk keertk requested review from Wyverald and fweikert November 26, 2024 20:58
@github-actions github-actions bot added team-Rules-CPP Issues for C++ rules awaiting-review PR is awaiting review from an assigned reviewer labels Nov 26, 2024
@keertk keertk enabled auto-merge (squash) November 26, 2024 20:58
@Wyverald
Copy link
Member

Looks like there was a bad merge somewhere. Could you go to this file src/test/shell/integration/run_test.sh and delete the whole function test_run_under_command_change_preserves_cache() { block?

@keertk keertk merged commit 4754de2 into bazelbuild:release-9.0.0-pre.20241119.2rc1 Nov 26, 2024
45 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Nov 26, 2024
keertk pushed a commit that referenced this pull request Nov 27, 2024
Baseline: ebeab8c

Cherry picks:

   + 650142f:
     Automated rollback of commit
     4515bb6.
   + 4754de2:
     [Rolling release] Do not forward legacy `cmd.exe` variables to
     the server (#24500)
   + 94fb35b:
     Fix rolling release (#24513)

Incompatible changes:

  - Changing --test_env no longer invalidates the analysis cache.
    `ctx.configuration.test_env` may be empty for non-test rules and
    should not be used by such rules.

Important changes:

  - Add a set data type to Starlark, guarded by the
    --experimental_enable_starlark_set flag.
  - symbolic macro attribute inheritance is now marked experimental;
    set --experimental_enable_macro_inherit_attrs flag to enable it.
  - Changing any part of `--run_under` that isn't the label (such as
    the shell command) no longer invalidates the analysis cache.
  - The progress of completed configured aspects is now shown in the
    UI, next to the number of completed targets. Example: "(100
    packages loaded, 10000 targets and 500 aspects configured)".

This release contains contributions from many people at Google, as well as Fabian Meumertzheim, kalvdans, Xùdōng Yáng.
copybara-service bot pushed a commit that referenced this pull request Nov 27, 2024
Baseline: ebeab8c

Cherry picks:

   + 650142f:
     Automated rollback of commit
     4515bb6.
   + 4754de2:
     [Rolling release] Do not forward legacy `cmd.exe` variables to
     the server (#24500)
   + 94fb35b:
     Fix rolling release (#24513)

Incompatible changes:

  - Changing --test_env no longer invalidates the analysis cache.
    `ctx.configuration.test_env` may be empty for non-test rules and
    should not be used by such rules.

Important changes:

  - Add a set data type to Starlark, guarded by the
    --experimental_enable_starlark_set flag.
  - symbolic macro attribute inheritance is now marked experimental;
    set --experimental_enable_macro_inherit_attrs flag to enable it.
  - Changing any part of `--run_under` that isn't the label (such as
    the shell command) no longer invalidates the analysis cache.
  - The progress of completed configured aspects is now shown in the
    UI, next to the number of completed targets. Example: "(100
    packages loaded, 10000 targets and 500 aspects configured)".

This release contains contributions from many people at Google, as well as Fabian Meumertzheim, kalvdans, Xùdōng Yáng.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Rules-CPP Issues for C++ rules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants