Skip to content

fix(github-actions): use stable TMPDIR for rules_sass compiler compilation#3535

Merged
alan-agius4 merged 1 commit intoangular:mainfrom
alan-agius4:sass-compiler-tmp
Mar 13, 2026
Merged

fix(github-actions): use stable TMPDIR for rules_sass compiler compilation#3535
alan-agius4 merged 1 commit intoangular:mainfrom
alan-agius4:sass-compiler-tmp

Conversation

@alan-agius4
Copy link
Contributor

@alan-agius4 alan-agius4 commented Mar 13, 2026

Forcing a stable temporary directory prevents the Dart compiler on macOS from embedding random absolute paths into the compiled binaries.

Previously, the random temp paths (e.g., /var/folders/.../random_hash/snapshot.aot) caused the binaries to differ on every build run even when the source code was identical. Using a workspace-relative TMPDIR ensures deterministic binary output.

…ation

Forcing a stable temporary directory prevents the Dart compiler on macOS from embedding random absolute paths into the compiled binaries.

Previously, the random temp paths (e.g., /var/folders/.../random_hash/snapshot.aot)
caused the binaries to differ on every build run even when the source code was identical. Using a workspace-relative TMPDIR ensures deterministic binary output.
@alan-agius4 alan-agius4 changed the title sadsad fix(github-actions): use stable TMPDIR for rules_sass compiler compilation Mar 13, 2026
@alan-agius4 alan-agius4 marked this pull request as ready for review March 13, 2026 12:51
@gemini-code-assist
Copy link

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label Mar 13, 2026
Copy link
Member

@josephperrott josephperrott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alan-agius4 alan-agius4 merged commit b9ac8bc into angular:main Mar 13, 2026
13 checks passed
@alan-agius4
Copy link
Contributor Author

This PR was merged into the repository. The changes were merged into the following branches:

@alan-agius4 alan-agius4 deleted the sass-compiler-tmp branch March 13, 2026 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants