Merge pull request #678 from axodotdev/ignore-script-hints

fix: don't respect cargo-dist's install hints for curl-sh exprs
axodotdev · Dec 20, 2023 · b11f405 · b11f405
2 parents aba8a57 + 60ab1fd
commit b11f405
Show file tree

Hide file tree

Showing 4 changed files with 115 additions and 112 deletions.
diff --git a/src/data/cargo_dist.rs b/src/data/cargo_dist.rs
@@ -61,11 +61,14 @@ impl ReleaseArtifacts {
                             {
                                 preference = InstallerPreference::Custom;
                                 None
+                            } else if id.ends_with(".sh") || id.ends_with(".ps1") {
+                                // We have more info to do a better job than cargo-dist on
+                                // `curl | sh` exprs, inference will handle this for us!
+                                continue;
                             } else {
                                 preference = InstallerPreference::Script;
                                 file
                             };
-
                             method = InstallMethod::Run {
                                 file,
                                 run_hint: install_hint.clone(),