lf-pico-template updates (#32)

* [init] platform opt, template repo

* [package]
abstract repo template setup

* [backend] set lfc backend for C

* [format] apply

* stub c-cmake

* remove cmake stub

* add tempfile

* format

---------

Co-authored-by: Tassilo Tanneberger <[email protected]>

Author: gundralaa

Cargo.lock

@@ -22,3 +22,4 @@ crossbeam = "*"
 termion = "1.5"
 git2 = "*"
 run_script = "0.10"
+tempfile = "*"

Cargo.toml

@@ -16,6 +16,7 @@ pub enum TargetLanguage {
 pub enum Platform {
     Native,
     Zephyr,
+    RP2040,
 }
 
 #[derive(clap::ValueEnum, Clone, Copy, Debug, Deserialize, Serialize, Eq, PartialEq, Hash)]
@@ -87,16 +88,17 @@ impl ToString for TargetLanguage {
 pub struct InitArgs {
     #[clap(value_enum, short, long)]
     pub language: Option<TargetLanguage>,
-
     #[clap(value_enum, short, long, default_value_t = Platform::Native)]
     pub platform: Platform,
 }
 impl InitArgs {
     pub fn get_target_language(&self) -> TargetLanguage {
         self.language.unwrap_or({
-            // Target language for Zephyr is C, else Cpp.
+            // Target language for Zephyr and RP2040 is C
+            // Else use Cpp.
             match self.platform {
                 Platform::Zephyr => TargetLanguage::C,
+                Platform::RP2040 => TargetLanguage::C,
                 _ => TargetLanguage::Cpp,
             }
         })
@@ -108,7 +110,7 @@ pub enum Command {
     /// initializing a lingua-franca project
     Init(InitArgs),
 
-    /// compiling one ore multiple binaries in a lingua-franca package
+    /// compiling one or multiple binaries in a lingua-franca package
     Build(BuildArgs),
 
     /// Updates the dependencies and potentially build tools

src/args.rs

@@ -16,6 +16,7 @@ fn gen_cmake_files(app: &App, options: &BuildCommandOptions) -> BuildResult {
     fs::create_dir_all(&build_dir)?;
 
     let mut cmake = Command::new("cmake");
+    // cmake args
     cmake.arg(format!(
         "-DCMAKE_BUILD_TYPE={}",
         if options.profile == BuildProfile::Release {

src/backends/cmake.rs

@@ -48,7 +48,7 @@ impl BatchBackend for LFC {
             CommandSpec::Update => todo!(),
             CommandSpec::Clean => {
                 results.par_map(|app| {
-                    fs::remove_dir_all(app.src_gen_dir())?;
+                    crate::util::default_build_clean(&app.output_root)?;
                     Ok(())
                 });
             }
@@ -89,7 +89,10 @@ impl<'a> LfcJsonArgs<'a> {
             .unwrap()
             .as_object_mut()
             .unwrap();
-        properties.insert("no-compile".to_string(), self.no_compile.into());
+        // lfc does not support no-compile:false
+        if self.no_compile {
+            properties.insert("no-compile".to_string(), self.no_compile.into());
+        }
         Ok(value)
     }
 }

src/backends/lfc.rs

@@ -5,9 +5,9 @@ use std::sync::Arc;
 
 use rayon::prelude::*;
 
-use crate::args::Platform;
+use crate::args::{BuildSystem, Platform};
+use crate::package::App;
 use crate::util::errors::{AnyError, BuildResult, LingoError};
-use crate::{args::BuildSystem, package::App};
 
 pub mod cmake;
 pub mod lfc;

src/backends/mod.rs

@@ -13,6 +13,7 @@ use std::{env, io};
 use crate::args::BuildSystem::{CMake, Cargo, LFC};
 use crate::util::errors::{BuildResult, LingoError};
 use git2::Repository;
+use tempfile::tempdir;
 use which::which;
 
 fn is_valid_location_for_project(path: &std::path::Path) -> bool {
@@ -92,6 +93,7 @@ pub struct App {
 impl App {
     pub fn build_system(&self) -> BuildSystem {
         match self.target {
+            TargetLanguage::C => LFC,
             TargetLanguage::Cpp => CMake,
             TargetLanguage::Rust => Cargo,
             TargetLanguage::TypeScript => {
@@ -118,16 +120,16 @@ impl App {
     }
 }
 
-/// Simple or DetailedDependcy
+/// Simple or DetailedDependency
 #[derive(Clone, Deserialize, Serialize)]
-pub enum FileDependcy {
+pub enum FileDependency {
     // the version string
     Simple(String),
     /// version string and source
     Advanced(DetailedDependency),
 }
 
-/// Dependcy with source and version
+/// Dependency with source and version
 #[derive(Clone, Deserialize, Serialize)]
 pub struct DetailedDependency {
     version: String,
@@ -222,23 +224,32 @@ impl ConfigFile {
         Ok(())
     }
 
-    // Sets up a LF project with Zephyr as the target platform.
-    pub fn setup_zephyr(&self) -> BuildResult {
-        // Clone lf-west-template into a temporary directory
-        let tmp_path = Path::new("zephyr_tmp");
-        if tmp_path.exists() {
-            remove_dir_all(tmp_path)?;
-        }
-        let url = "https://github.com/lf-lang/lf-west-template";
+    fn setup_template_repo(&self, url: &str) -> BuildResult {
+        let dir = tempdir()?;
+        let tmp_path = dir.path();
         Repository::clone(url, tmp_path)?;
-
         // Copy the cloned template repo into the project directory
         copy_recursively(tmp_path, Path::new("."))?;
+        // Remove temporary folder
+        dir.close()?;
+        Ok(())
+    }
 
-        // Remove .git, .gitignore ad temporary folder
+    // Sets up a LF project with Zephyr as the target platform.
+    fn setup_zephyr(&self) -> BuildResult {
+        let url = "https://github.com/lf-lang/lf-west-template";
+        self.setup_template_repo(url)?;
         remove_file(".gitignore")?;
         remove_dir_all(Path::new(".git"))?;
-        remove_dir_all(tmp_path)?;
+        Ok(())
+    }
+
+    // Sets up a LF project with RP2040 MCU as the target platform.
+    // Initializes a repo using the lf-pico-template
+    fn setup_rp2040(&self) -> BuildResult {
+        let url = "https://github.com/lf-lang/lf-pico-template";
+        // leave git artifacts
+        self.setup_template_repo(url)?;
         Ok(())
     }
 
@@ -247,6 +258,7 @@ impl ConfigFile {
             match self.apps[0].platform {
                 Some(Platform::Native) => self.setup_native(),
                 Some(Platform::Zephyr) => self.setup_zephyr(),
+                Some(Platform::RP2040) => self.setup_rp2040(),
                 _ => Ok(()),
             }
         } else {