Revert "chore(manifest) impl hash for build manifest"

This reverts commit 67ebbac.

Hashing is done per file (raw), and not per
build_manifest struct due to Blake3 not supporting it.

Author: Taxrosdev

src/build/mod.rs

@@ -4,7 +4,7 @@ mod sources;
 
 use anyhow::{Context, Result, bail};
 use std::{
-    collections::BTreeMap,
+    collections::HashMap,
     fs,
     path::{Path, PathBuf},
     process::Command,
@@ -18,7 +18,7 @@ use crate::{
 use hash::calc_build_hash;
 use sources::get_sources;
 
-#[derive(serde::Deserialize, serde::Serialize, Clone, Hash)]
+#[derive(serde::Deserialize, serde::Serialize, Clone)]
 struct BuildManifest {
     /// ID of this package, the main alias
     id: String,
@@ -46,11 +46,10 @@ struct BuildManifest {
     /// Useful for SDKs.
     sdks: Option<Vec<String>>,
     /// RUNTIME environment variables
-    /// This is a `BTreeMap` due to `HashMap` not having the `hash` trait.
-    env: Option<BTreeMap<String, String>>,
+    env: Option<HashMap<String, String>>,
 }
 
-#[derive(serde::Deserialize, serde::Serialize, Clone, Hash)]
+#[derive(serde::Deserialize, serde::Serialize, Clone)]
 struct Source {
     /// Should either be git, tar or local
     kind: String,
@@ -199,7 +198,7 @@ fn include_all(
     build_dir: &Path,
     repo_path: &Path,
     chunk_store_path: &Path,
-    envs: &mut BTreeMap<String, String>,
+    envs: &mut HashMap<String, String>,
 ) -> Result<()> {
     for dependency in packages {
         let result = include(
@@ -224,7 +223,7 @@ fn include(
     path_to_include_at: &Path,
     repo_path: &Path,
     chunk_store_path: &Path,
-) -> Result<BTreeMap<String, String>> {
+) -> Result<HashMap<String, String>> {
     let dependency_build_manifest_path = search_path.join(dependency);
     let dependency_build_manifest: BuildManifest =
         serde_yaml::from_str(&fs::read_to_string(dependency_build_manifest_path)?)?;

src/repo/types.rs

@@ -1,4 +1,4 @@
-use std::{collections::BTreeMap, path::PathBuf};
+use std::{collections::HashMap, path::PathBuf};
 
 use crate::chunks::{Chunk, HashKind};
 
@@ -20,13 +20,13 @@ pub struct PackageManifest {
     pub chunks: Vec<Chunk>,
     pub commands: Vec<PathBuf>,
     /// Runtime environment variables
-    pub env: Option<BTreeMap<String, String>>,
+    pub env: Option<HashMap<String, String>>,
     #[serde(default = "build_hash_default")]
     pub build_hash: String,
 }
 
 /// All of these are user visible, and should carry no actual weight.
-#[derive(serde::Deserialize, serde::Serialize, Debug, Clone, PartialEq, Eq, Hash)]
+#[derive(serde::Deserialize, serde::Serialize, Debug, Clone, PartialEq, Eq)]
 pub struct Metadata {
     pub title: Option<String>,
     pub description: Option<String>,

src/run/mod.rs

@@ -2,7 +2,7 @@ pub mod quicklaunch;
 
 use anyhow::{Context, Result, bail};
 use std::{
-    collections::BTreeMap,
+    collections::HashMap,
     ffi::OsStr,
     path::{Path, PathBuf},
     process::{Command, ExitStatus},
@@ -44,7 +44,7 @@ pub fn start<S: AsRef<OsStr>>(
         let entrypoint = entrypoint.to_string_lossy();
         let entrypoint: &str = entrypoint.trim_start_matches('/');
 
-        let mut envs: BTreeMap<String, String> = package_manifest.env.unwrap_or_default();
+        let mut envs: HashMap<String, String> = package_manifest.env.unwrap_or_default();
 
         // I hate I have to do this.
         let keys_to_update: Vec<String> = envs