Skip to content
Snippets Groups Projects

ci: use JSON to transfer environment variables

Merged Ben Boeckel requested to merge ben.boeckel/diy:ci-vcvarsall-fix into master

The set command in cmd has terrible properties in that there's no real structure to it. Sensitive sequences in values or variable names that are escaping sequences can be interpreted at the wrong time or things like newlines in values are not escaped at all and cause ambiguities in parsing. Avoid all of that and use PowerShell to use JSON as a communication mechanism.


Also remove executable permissions from .ps1 files.

Merge request reports

Merge request pipeline #421649 passed

Merge request pipeline passed for e5fecfc9

Approved by

Merged by Kitware RobotKitware Robot 5 months ago (Oct 31, 2024 3:55pm UTC)

Merge details

  • Changes merged into master with 1d0bd0a2.
  • Deleted the source branch.

Pipeline #421683 passed

Pipeline passed for 1d0bd0a2 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading