Support multiple output directories #18

Allow multiple -o flags by introducing Preferences.output_dirs (replacing single output_dir). CLI parsing now appends each output path into output_dirs and validates at least one is provided. PEX backend writes generated .pex files to the first output dir and copies them to any additional dirs. Original backend now warns when multiple output dirs are used (only the first dir is used). Added a new warning message constant and adjusted error/message string formatting. Updated tests to use output_dirs and added test_multiple_output_dirs. CHANGELOG updated to mention the new feature.

Author: russo-2025

CHANGELOG.md

@@ -1,8 +1,10 @@
 ## Next Release
 
-...
+### New Features
 
-## V 0.0.4
+- Added support for multiple output directories - you can now specify multiple `-o` flags to copy compiled .pex files to multiple locations. #18
+
+...
 
 ### New Features
 

modules/builder/builder.v

@@ -68,7 +68,12 @@ pub fn (mut b Builder) run() bool {
 	b.print("used header dirs ${b.pref.header_dirs}")
 
 	b.files, b.files_names = find_all_src_files(b.pref.paths)
-	
+
+	// Warn if using multiple output directories with -original flag
+	if b.pref.output_dirs.len > 1 && b.pref.backend == .original {
+		b.print("Warning: Using multiple output directories with -original flag is not fully supported. Files will only be compiled to the first directory.")
+	}
+
 	match b.pref.backend {
 		.check,
 		.pex {

modules/builder/original.v

@@ -15,7 +15,7 @@ fn (mut b Builder) compile_original() {
 	header_dirs = header_dirs[..header_dirs.len-1]
 
 	for file in b.files {
-		cmd := '"${compiler_exe_path}" "${file}" -quiet -i="${header_dirs}" -o="${b.pref.output_dir}" -f="${compiler_flags_path}"'
+		cmd := '"${compiler_exe_path}" "${file}" -quiet -i="${header_dirs}" -o="${b.pref.output_dirs[0]}" -f="${compiler_flags_path}"'
 
 		b.print("executing: `${cmd}`")
 

modules/builder/pex.v

@@ -99,7 +99,8 @@ fn (mut b Builder) compile_pex() {
 fn (mut b Builder) gen_to_pex_file(mut parsed_file ast.File, mut buff_bytes pex.Buffer) {
 	if is_outdated(parsed_file, b.pref) {
 		output_file_name := parsed_file.file_name + ".pex"
-		output_file_path := os.join_path(b.pref.output_dir, output_file_name)
+		// Use first output directory for writing
+		output_file_path := os.join_path(b.pref.output_dirs[0], output_file_name)
 
 		mut pex_file := b.generator.gen(mut parsed_file)
 
@@ -114,6 +115,16 @@ fn (mut b Builder) gen_to_pex_file(mut parsed_file ast.File, mut buff_bytes pex.
 			util.fatal_error("failed to write file: ${err}")
 		}
 		file.close()
+
+		// Copy to additional output directories
+		if b.pref.output_dirs.len > 1 {
+			for j in 1 .. b.pref.output_dirs.len {
+				copy_path := os.join_path(b.pref.output_dirs[j], output_file_name)
+				os.cp(output_file_path, copy_path) or {
+					util.fatal_error("failed to copy file to ${copy_path}: ${err}")
+				}
+			}
+		}
 	}
 }
 

modules/papyrus/errors/errors.v

@@ -35,19 +35,20 @@ pub:
 	reporter  Reporter
 }
 
-pub const msg_invalid_output_path = "Error: Invalid path specified for the -o flag. Please provide a valid directory path where compiled files should be saved.";
-pub const msg_invalid_input_path = "Error: Invalid path specified for the -i flag. Please provide a valid directory or file path containing the .psc scripts to compile.";
-pub const msg_invalid_headers_path = "Error: Invalid path specified for the -h flag. Please provide a valid directory path containing header/import .psc files.";
+pub const msg_invalid_output_path = "Error: Invalid path specified for the -o flag. Please provide a valid directory path where compiled files should be saved."
+pub const msg_invalid_input_path = "Error: Invalid path specified for the -i flag. Please provide a valid directory or file path containing the .psc scripts to compile."
+pub const msg_invalid_headers_path = "Error: Invalid path specified for the -h flag. Please provide a valid directory path containing header/import .psc files."
 pub const msg_duplicate_input_flag = "Error: Duplicate -i flag detected with the same path. Please ensure each -i flag points to a unique directory or file."
-pub const msg_duplicate_output_flag = "Error: The -o flag has already been specified. Please provide only one output directory.";
-pub const msg_missing_input = "Error: Missing mandatory -i flag. Please specify the directory or file containing .psc scripts to compile.";
-pub const msg_missing_output = "Error: Missing mandatory -o flag. Please specify the directory where compiled .pex files should be saved.";
+pub const msg_duplicate_output_flag = "Error: The -o flag has already been specified. Please provide only one output directory."
+pub const msg_missing_input = "Error: Missing mandatory -i flag. Please specify the directory or file containing .psc scripts to compile."
+pub const msg_missing_output = "Error: Missing mandatory -o flag. Please specify the directory where compiled .pex files should be saved."
+pub const msg_warning_multiple_output_with_original = "Warning: Using multiple output directories with -original flag is not fully supported. Files will only be compiled to the first directory."
 //TODO
-pub const msg_wrong_number_of_arguments = "Error: Incorrect number of arguments for the command. Please refer to the documentation for the correct usage.";
+pub const msg_wrong_number_of_arguments = "Error: Incorrect number of arguments for the command. Please refer to the documentation for the correct usage."
 //TODO
-pub const msg_missing_or_incorrect_argument = "Error: Missing or incorrect argument. Please check the syntax of your command.";
-pub const msg_missing_or_incorrect_command = "Error: Invalid command. Please use one of the following commands: compile, read, disassembly, create-dump, help.";
-pub const msg_invalid_path_disassembly = "Error: Invalid path specified for the disassembly command. Please provide a valid file path for the .pex file.";
-pub const msg_invalid_path_read = "Error: Invalid path specified for the read command. Please provide a valid file path for the .pex file.";
-pub const msg_invalid_path_create_dump = "Error: Invalid path specified for create-dump. Please provide a valid directory path containing .pex files.";
+pub const msg_missing_or_incorrect_argument = "Error: Missing or incorrect argument. Please check the syntax of your command."
+pub const msg_missing_or_incorrect_command = "Error: Invalid command. Please use one of the following commands: compile, read, disassembly, create-dump, help."
+pub const msg_invalid_path_disassembly = "Error: Invalid path specified for the disassembly command. Please provide a valid file path for the .pex file."
+pub const msg_invalid_path_read = "Error: Invalid path specified for the read command. Please provide a valid file path for the .pex file."
+pub const msg_invalid_path_create_dump = "Error: Invalid path specified for create-dump. Please provide a valid directory path containing .pex files."
 

modules/pref/pref.v

@@ -47,7 +47,7 @@ pub fn (m RunMode) str() string {
 pub struct Preferences {
 pub mut:
 	paths				[]string	// folders with files to compile
-	output_dir			string		// folder for output files
+	output_dirs			[]string	// folders for output files
 	mode				RunMode = .compile
 	backend				Backend = .pex
 	no_cache			bool
@@ -71,7 +71,9 @@ pub fn (p Preferences) cmd_str() string {
 		b.write_string("-i \"${path}\" ")
 	}
 
-	b.write_string("-o \"${p.output_dir}\" ")
+	for dir in p.output_dirs {
+		b.write_string("-o \"${dir}\" ")
+	}
 
 	for dir in p.header_dirs {
 		b.write_string("-h \"${dir}\" ")
@@ -124,17 +126,13 @@ fn (mut p Preferences) parse_compile_args(args []string) {
 			"-output" {
 				i++
 
-				if p.output_dir != "" {
-					error(errors.msg_duplicate_output_flag) // path
-				}
-
 				path := os.real_path(args[i])
 
 				if !os.is_dir(path) {
 					error(errors.msg_invalid_output_path) // path
 				}
 
-				p.output_dir = path
+				p.output_dirs << path
 				i++
 			}
 			"-h",
@@ -192,7 +190,7 @@ fn (mut p Preferences) parse_compile_args(args []string) {
 		error(errors.msg_missing_input)
 	}
 
-	if p.output_dir == "" {
+	if p.output_dirs.len == 0 {
 		error(errors.msg_missing_output)
 	}
 }

modules/tests/projects_test.v

@@ -7,7 +7,7 @@ import builder
 fn get_prefs(input_dir string, header_dirs []string, output_dir string) pref.Preferences {
 	return pref.Preferences {
 		paths: [ input_dir ]
-		output_dir: output_dir
+		output_dirs: [ output_dir ]
 		mode: .compile
 		backend: .check
 		no_cache: true

modules/tests/selective_headers_loading_test.v

@@ -35,7 +35,7 @@ fn test_selective_headers_loading() {
 
 	prefs := pref.Preferences {
 		paths: [ src_file ]
-		output_dir: output_dir
+		output_dirs: [ output_dir ]
 		mode: .compile
 		backend: .check
 		no_cache: true

modules/tests/vm_tests_test.v

@@ -26,17 +26,17 @@ fn test_builder() {
 
 	prefs := pref.Preferences {
 		paths: [ input_dir ]
-		output_dir: output_dir
+		output_dirs: [ output_dir ]
 		mode: .compile
 		backend: .pex
 		no_cache: true
 		header_dirs: []
 		output_mode: .silent
 	}
 
-	out_file1 := os.join_path(prefs.output_dir, "AAATestObject.pex")
-	out_file2 := os.join_path(prefs.output_dir, "LatentTest.pex")
-	out_file3 := os.join_path(prefs.output_dir, "OpcodesTest.pex")
+	out_file1 := os.join_path(prefs.output_dirs[0], "AAATestObject.pex")
+	out_file2 := os.join_path(prefs.output_dirs[0], "LatentTest.pex")
+	out_file3 := os.join_path(prefs.output_dirs[0], "OpcodesTest.pex")
 
 	if os.is_file(out_file1) {
 		os.rm(out_file1) or { assert false, "failed to delete file" }
@@ -100,4 +100,65 @@ fn test_builder() {
 	for func in pex3.objects[0].states[0].functions {
 		assert pex3.get_string(func.name) in pex3_string_table
 	}
+}
+
+fn test_multiple_output_dirs() {
+	input_dir := os.real_path('test-files/vm-tests')
+	output_dir1 := os.real_path('test-files/compiled/multi1')
+	output_dir2 := os.real_path('test-files/compiled/multi2')
+
+	file1 := os.join_path(input_dir, "AAATestObject.psc")
+
+	if !os.is_file(file1) {
+		assert false, "invalid input file ${file1}"
+	}
+
+	// Create output directories
+	if !os.is_dir(output_dir1) {
+		os.mkdir(output_dir1, os.MkdirParams{}) or { assert false, "failed to create output_dir1" }
+	}
+	if !os.is_dir(output_dir2) {
+		os.mkdir(output_dir2, os.MkdirParams{}) or { assert false, "failed to create output_dir2" }
+	}
+
+	prefs := pref.Preferences {
+		paths: [ input_dir ]
+		output_dirs: [ output_dir1, output_dir2 ]
+		mode: .compile
+		backend: .pex
+		no_cache: true
+		header_dirs: []
+		output_mode: .silent
+	}
+
+	out_file1 := os.join_path(prefs.output_dirs[0], "AAATestObject.pex")
+	out_file2 := os.join_path(prefs.output_dirs[1], "AAATestObject.pex")
+
+	// Clean up previous runs
+	if os.is_file(out_file1) {
+		os.rm(out_file1) or { assert false, "failed to delete file" }
+	}
+	if os.is_file(out_file2) {
+		os.rm(out_file2) or { assert false, "failed to delete file" }
+	}
+
+	builder.compile(&prefs)
+
+	// Verify file exists in first output directory
+	assert os.is_file(out_file1), "pex file should exist in first output directory"
+
+	// Verify file was copied to second output directory
+	assert os.is_file(out_file2), "pex file should exist in second output directory (copied)"
+
+	// Verify both files are identical
+	pex1_content := os.read_bytes(out_file1) or { panic("failed to read pex1") }
+	pex2_content := os.read_bytes(out_file2) or { panic("failed to read pex2") }
+	assert pex1_content.len == pex2_content.len, "both pex files should have same size"
+	assert pex1_content == pex2_content, "both pex files should be identical"
+
+	// Clean up
+	os.rm(out_file1) or {}
+	os.rm(out_file2) or {}
+	os.rmdir(output_dir1) or {}
+	os.rmdir(output_dir2) or {}
 }