i #313 Minor Fix: Change srcml_path to use tools.yml

Signed-off-by: Dao McGill <dmcgill@hawaii.edu>
sailuh · Nov 5, 2024 · 68e2331 · 68e2331
1 parent 1cc63c8
commit 68e2331
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/conf/maven.yml b/conf/maven.yml
@@ -14,7 +14,7 @@ filter:
 
 srcml:
   # path to tool binary (change this location later)
-  srcml_path: /usr/local/bin/srcml
+  # srcml_path: /usr/local/bin/srcml
   # path to the folder to analyze
   src_folder: ../../maven/maven-artifact/src/main/java/org/apache/maven/artifact/
   srcml_filepath: ../../analysis/maven/srcml_output.xml
diff --git a/vignettes/syntax_extractor.Rmd b/vignettes/syntax_extractor.Rmd
@@ -101,6 +101,12 @@ srcml:
   srcml_filepath: ../../analysis/maven/srcml_output.xml
 ```
 
+The path to the srcML binary should be defined in tools.yml:
+
+```         
+srcml: /usr/local/bin/srcml
+```
+
 ### Explanation
 
 -   website: URL to the project repository, helpful for reference or project metadata.
@@ -127,7 +133,7 @@ tool <- yaml::read_yaml("../tools.yml")
 conf <- yaml::read_yaml("../conf/maven.yml")
 
 # Paths from config
-srcml_path <- conf[["srcml"]][["srcml_path"]]
+srcml_path <- tool[["srcml"]]
 src_folder <- conf[["srcml"]][["src_folder"]]
 # srcml_output_path <- conf[["tool"]][["srcml_output_path"]]
 srcml_filepath <- conf[["srcml"]][["srcml_filepath"]]