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

Do not forward legacy cmd.exe variables to the server #24451

Closed
wants to merge 1 commit into from

Conversation

fmeum
Copy link
Collaborator

@fmeum fmeum commented Nov 22, 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

@fmeum fmeum force-pushed the 24448-client-env branch 2 times, most recently from 3262743 to 8332050 Compare November 22, 2024 09:34
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
```
@fmeum fmeum changed the title Debug Windows failure Do not forward legacy cmd.exe variables to the server Nov 22, 2024
@fmeum fmeum requested a review from meteorcloudy November 22, 2024 10:34
@fmeum fmeum marked this pull request as ready for review November 22, 2024 10:35
@fmeum fmeum requested a review from Wyverald November 22, 2024 10:35
@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 22, 2024
@fmeum
Copy link
Collaborator Author

fmeum commented Nov 22, 2024

GitHub has a partial outage.

@meteorcloudy
Copy link
Member

Thanks! I'll import this one now

@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Nov 22, 2024
bazel-io pushed a commit to bazel-io/bazel that referenced this pull request Nov 22, 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 bazelbuild#24448

Closes bazelbuild#24451.

PiperOrigin-RevId: 699115724
Change-Id: Id7a2445e82b24384e9ade0c92ddb343f00e9ab2b
@fmeum fmeum deleted the 24448-client-env branch November 22, 2024 11:51
github-merge-queue bot pushed a commit that referenced this pull request Nov 22, 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

Commit
59eb519

Co-authored-by: Fabian Meumertzheim <fabian@meumertzhe.im>
keertk added a commit that referenced this pull request Nov 26, 2024
…rver (#24500)

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

Co-authored-by: Fabian Meumertzheim <fabian@meumertzhe.im>
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.

Bazel always errors with bad --client_env on Windows for HEAD Bazel
2 participants