Skip to content

Commit de69665

Browse files
docs(Mountain): Update build.rs comments to document PascalCase naming
Clarify the build script comments to document the PascalCase environment variable naming convention introduced in the 2026-04-29 migration. The comments now specify that `Profile`, `Pack`, `Bundle`, and `Compiler` (PascalCase) are the current env var names, replacing the legacy `LAND_*` names that were retired. This ensures clarity for developers debugging build configuration resolution at runtime.
1 parent 5b111b1 commit de69665

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

build.rs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,9 +75,11 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
7575
// Profile sentinel: Build.sh exports `Browser=true` / `Mountain=true` /
7676
// `Electron=true` / `Bundle` / `Compiler` / `Profile` into the shell
7777
// that invokes cargo. These shell env vars don't survive to the resulting
78-
// binary; only `cargo:rustc-env` does. Bake a LAND_* set into the binary so
78+
// binary; only `cargo:rustc-env` does. Bake a PascalCase Land env set
79+
// (`Profile`, `Pack`, `Bundle`, `Compiler`) into the binary so
7980
// `option_env!("Profile")` resolves after launch without depending on
80-
// the shell the user runs the binary from.
81+
// the shell the user runs the binary from. Legacy `LAND_*` names were
82+
// retired in the 2026-04-29 PascalCase migration.
8183
//
8284
// Follow-up to playbook item #3 - previously the sentinel logged
8385
// `Active profile=unknown` because it ran `std::env::var` at runtime.

0 commit comments

Comments
 (0)