chore(xtask): Dont update run submodule update

alvr/session/build.rs

@@ -4,7 +4,7 @@ use std::{env, fmt::Write, fs, path::PathBuf};
 fn main() {
     let openvr_driver_header_string =
         fs::read_to_string(alvr_filesystem::workspace_dir().join("openvr/headers/openvr_driver.h"))
-            .unwrap();
+            .expect("Missing openvr header files, did you clone the submodule?\n");
 
     let property_finder = Regex::new(
         r"\tProp_([A-Za-z\d_]+)_(Bool|Int32|Uint64|Float|String|Vector3)[\t ]+= ([0-9]+)",

alvr/xtask/src/build.rs

@@ -374,7 +374,7 @@ pub fn build_android_client(profile: Profile) {
     {
         let keystore_path = build_dir.join("debug.keystore");
         if !keystore_path.exists() {
-            let keytool = PathBuf::from(env::var("JAVA_HOME").unwrap())
+            let keytool = PathBuf::from(env::var("JAVA_HOME").expect("Env var JAVA_HOME not set"))
                 .join("bin")
                 .join(afs::exec_fname("keytool"));
             let pass = "alvrclient";

alvr/xtask/src/dependencies.rs

@@ -9,14 +9,6 @@ pub enum OpenXRLoadersSelection {
     All,
 }
 
-pub fn update_submodules(sh: &Shell) {
-    let dir = sh.push_dir(afs::workspace_dir());
-    cmd!(sh, "git submodule update --init --recursive")
-        .run()
-        .unwrap();
-    std::mem::drop(dir);
-}
-
 pub fn choco_install(sh: &Shell, packages: &[&str]) -> Result<(), xshell::Error> {
     cmd!(
         sh,
@@ -85,8 +77,6 @@ pub fn prepare_ffmpeg_windows(deps_path: &Path) {
 pub fn prepare_windows_deps(skip_admin_priv: bool) {
     let sh = Shell::new().unwrap();
 
-    update_submodules(&sh);
-
     let deps_path = afs::deps_dir().join("windows");
     sh.remove_path(&deps_path).ok();
     sh.create_dir(&deps_path).unwrap();
@@ -113,8 +103,6 @@ pub fn prepare_windows_deps(skip_admin_priv: bool) {
 pub fn prepare_linux_deps(enable_nvenc: bool) {
     let sh = Shell::new().unwrap();
 
-    update_submodules(&sh);
-
     let deps_path = afs::deps_dir().join("linux");
     sh.remove_path(&deps_path).ok();
     sh.create_dir(&deps_path).unwrap();
@@ -293,11 +281,7 @@ pub fn build_ffmpeg_linux(enable_nvenc: bool, deps_path: &Path) {
     cmd!(sh, "make install").run().unwrap();
 }
 
-pub fn prepare_macos_deps() {
-    let sh = Shell::new().unwrap();
-
-    update_submodules(&sh);
-}
+pub fn prepare_macos_deps() {}
 
 pub fn prepare_server_deps(
     platform: Option<BuildPlatform>,
@@ -390,8 +374,6 @@ pub fn build_android_deps(
 ) {
     let sh = Shell::new().unwrap();
 
-    update_submodules(&sh);
-
     if cfg!(windows) && !skip_admin_priv {
         choco_install(&sh, &["unzip", "llvm"]).unwrap();
     }

wiki/Building-From-Source.md

@@ -6,7 +6,10 @@ Preferred IDE (optional): Visual Studio Code with rust-analyzer extension
 
 You need to install [rustup](https://www.rust-lang.org/tools/install).
 
-On Windows you need also [Chocolatey](https://chocolatey.org/install).
+On Windows you also need [Chocolatey](https://chocolatey.org/install).
+
+To clone the repository use `git clone --recurse-submodules https://github.com/alvr-org/ALVR.git`.
+If you previously cloned the repo without submodules, simply run `git submodule update --init --checkout --recursive` in it.
 
 # Streamer Building