fix: full commit hash in canary compile download (#9166)

denoland · Jan 19, 2021 · cf32026 · cf32026
1 parent 867ce78
commit cf32026
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 5 deletions.
diff --git a/cli/build.rs b/cli/build.rs
@@ -221,7 +221,7 @@ fn git_commit_hash() -> String {
     .output()
   {
     if output.status.success() {
-      std::str::from_utf8(&output.stdout[..7])
+      std::str::from_utf8(&output.stdout[..40])
         .unwrap()
         .to_string()
     } else {

diff --git a/cli/tools/upgrade.rs b/cli/tools/upgrade.rs
@@ -40,9 +40,7 @@ pub async fn upgrade_command(
   let install_version = match version {
     Some(passed_version) => {
       let current_is_passed = if canary {
-        let mut passed_hash = passed_version.clone();
-        passed_hash.truncate(7);
-        crate::version::GIT_COMMIT_HASH == passed_hash
+        crate::version::GIT_COMMIT_HASH == passed_version
       } else if !crate::version::is_canary() {
         crate::version::deno() == passed_version
       } else {

diff --git a/cli/version.rs b/cli/version.rs
@@ -6,7 +6,7 @@ pub const TYPESCRIPT: &str = env!("TS_VERSION");
 pub fn deno() -> String {
   let semver = env!("CARGO_PKG_VERSION");
   option_env!("DENO_CANARY").map_or(semver.to_string(), |_| {
-    format!("{}+{}", semver, GIT_COMMIT_HASH)
+    format!("{}+{}", semver, &GIT_COMMIT_HASH[..7])
   })
 }