Add flag for rtl target to files without target

Author: micprog

CHANGELOG.md

@@ -8,6 +8,9 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/) a
 ### Fixed
 - Put `vcs`, `vsim`, and `riviera` defines in quotes.
 
+### Added
+- Add flag for `rtl` target to files without target in script and sources.
+
 ## 0.28.1 - 2024-02-22
 ### Added
 - Add `flist-plus` script format for file list with plusargs.

src/cmd/script.rs

@@ -188,6 +188,13 @@ pub fn new() -> Command {
                 .num_args(1)
                 .value_parser(value_parser!(String)),
         )
+        .arg(
+            Arg::new("assume_rtl")
+                .long("assume-rtl")
+                .help("Add the `rtl` target to any fileset without a target specification")
+                .num_args(0)
+                .action(ArgAction::SetTrue)
+        )
 }
 
 fn get_package_strings<I>(packages: I) -> IndexSet<String>
@@ -245,6 +252,11 @@ pub fn run(sess: &Session, matches: &ArgMatches) -> Result<()> {
             )
         })
         .unwrap_or_else(|| TargetSet::new(format_targets));
+
+    if matches.get_flag("assume_rtl") {
+        srcs = srcs.assign_target("rtl".to_string());
+    }
+
     srcs = srcs
         .filter_targets(&targets)
         .unwrap_or_else(|| SourceGroup {

src/cmd/sources.rs

@@ -62,6 +62,13 @@ pub fn new() -> Command {
                 .action(ArgAction::Append)
                 .value_parser(value_parser!(String)),
         )
+        .arg(
+            Arg::new("assume_rtl")
+                .long("assume-rtl")
+                .help("Add the `rtl` target to any fileset without a target specification")
+                .num_args(0)
+                .action(ArgAction::SetTrue),
+        )
         .arg(
             Arg::new("raw")
                 .long("raw")
@@ -100,6 +107,11 @@ pub fn run(sess: &Session, matches: &ArgMatches) -> Result<()> {
         .get_many::<String>("target")
         .map(TargetSet::new)
         .unwrap_or_else(TargetSet::empty);
+
+    if matches.get_flag("assume_rtl") {
+        srcs = srcs.assign_target("rtl".to_string());
+    }
+
     srcs = srcs
         .filter_targets(&targets)
         .unwrap_or_else(|| SourceGroup {

src/src.rs

@@ -107,6 +107,35 @@ impl<'ctx> SourceGroup<'ctx> {
         )
     }
 
+    /// Assigns target to SourceGroup without target
+    pub fn assign_target(&self, target: String) -> SourceGroup<'ctx> {
+        let files = self
+            .files
+            .iter()
+            .filter_map(|file| match *file {
+                SourceFile::Group(ref group) => Some(group.assign_target(target.clone()))
+                    .map(|g| SourceFile::Group(Box::new(g))),
+                ref other => Some(other.clone()),
+            })
+            .collect();
+
+        SourceGroup {
+            package: self.package,
+            independent: self.independent,
+            target: if self.target.is_wildcard() {
+                TargetSpec::Name(target)
+            } else {
+                self.target.clone()
+            },
+            include_dirs: self.include_dirs.clone(),
+            export_incdirs: self.export_incdirs.clone(),
+            defines: self.defines.clone(),
+            files,
+            dependencies: self.dependencies.clone(),
+            version: self.version.clone(),
+        }
+    }
+
     /// Recursively get dependency names.
     fn get_deps(
         &self,