feat(config): enable reuse of mfm configuration variables

This change introduces a two-pass parsing approach to allow reuse of mfm
configuration variables within other configuration parts. This initial
implementation does not handle recursive dependencies, which is
acceptable for the current requirements.

- Implemented a two-pass parsing mechanism for configuration files.
- Added mfm configuration variables to the Crinja render context in both
  passes.
- Ensured the global mount point base is safely retrieved and validated
  in each pass.

Signed-off-by: Glenn Y. Rolland <glenux@glenux.net>

Makefile

@@ -24,7 +24,7 @@ test:
 install:
 	install \
 		-m 755 \
-		bin/code-preloader \
+		bin/mfm \
 		$(PREFIX)/bin
 
 .PHONY: spec test build all prepare install

README.md

@@ -180,5 +180,5 @@ By contributing, you agree to our code of conduct and license terms.
 
 ## License
 
-GNU GPL-3
+GNU GPL-3 
 

src/cli.cr

@@ -16,7 +16,7 @@ module GX
   class Cli
     Log = ::Log.for("cli")
 
-    @config : GX::Config
+   @config : GX::Config
 
     def initialize
       # Main execution starts here

src/config.cr

@@ -94,14 +94,37 @@ module GX
         exit(1)
       end
 
+      ## PASS 1
       file_data = File.read(config_path)
-      file_patched = Crinja.render(file_data, {"env" => ENV.to_h})
-
+      file_patched = Crinja.render(
+        file_data, 
+        {
+          "env" => ENV.to_h, 
+          "mfm" => {
+            "global" => {"mount_point_base" => "" }
+          }
+        }
+      )
       root = Models::RootConfig.from_yaml(file_patched)
 
       mount_point_base_safe = root.global.mount_point_base
       raise Models::InvalidMountpointError.new("Invalid global mount point") if mount_point_base_safe.nil?
 
+      ## PASS 2
+      file_patched = Crinja.render(
+        file_data, 
+        {
+          "env" => ENV.to_h, 
+          "mfm" => {
+            "global" => {"mount_point_base" => mount_point_base_safe }
+          }
+        }
+      )
+
+      root = Models::RootConfig.from_yaml(file_patched)
+      mount_point_base_safe = root.global.mount_point_base
+      raise Models::InvalidMountpointError.new("Invalid global mount point") if mount_point_base_safe.nil?
+
       root.filesystems.each do |selected_filesystem|
         if !selected_filesystem.mount_point?
           selected_filesystem.mount_point =

src/main.cr

@@ -31,6 +31,6 @@ Log.setup do |config|
   end
 end
 
-app = GX::Cli.new
-app.parse_command_line(ARGV)
-app.run
+cli = GX::Cli.new
+cli.parse_command_line(ARGV)
+cli.run

src/models/sshfs_config.cr

@@ -13,6 +13,7 @@ module GX::Models
     getter remote_user : String = ""
     getter remote_host : String = ""
     getter remote_port : String = "22"
+    getter options : Array(String) = [] of String
 
     include Concerns::Base
 
@@ -28,13 +29,19 @@ module GX::Models
       mount_point_safe = @mount_point
       raise InvalidMountpointError.new("Invalid mount point") if mount_point_safe.nil?
 
+      options = [] of String
+      # merge sshfs options
+      @options.each do |option|
+        options.push("-o", option)
+      end
+      options.push("-p", remote_port)
+      options.push(
+        "#{@remote_user}@#{@remote_host}:#{@remote_path}",
+          mount_point_safe
+      )
       process = Process.new(
         "sshfs",
-        [
-          "-p", remote_port,
-          "#{@remote_user}@#{@remote_host}:#{@remote_path}",
-          mount_point_safe,
-        ],
+        options,
         input: STDIN,
         output: STDOUT,
         error: STDERR
@@ -43,3 +50,4 @@ module GX::Models
     end
   end
 end
+