ISLE build-script check logic: Canonicalize All The Things!
writeln!() to a string on Windows to generate the expected manifest needs newline-character canonicalization, too.
This commit is contained in:
@@ -272,7 +272,7 @@ fn maybe_rebuild_isle(
|
||||
// Canonicalize Windows line-endings into Unix line-endings in
|
||||
// the manifest text itself.
|
||||
let manifest = manifest.replace("\r\n", "\n");
|
||||
let expected_manifest = compilation.compute_manifest()?;
|
||||
let expected_manifest = compilation.compute_manifest()?.replace("\r\n", "\n");
|
||||
if manifest != expected_manifest {
|
||||
rebuild_compilations.push((compilation, expected_manifest));
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user