Skip to content

Conversation

@Andrew-Musa
Copy link

I performed a comprehensive analysis and refactoring of the Simula project to improve its structure, build process, and maintainability. This involved creating a new dedicated main scene, Simula.tscn, and updating the Godot project settings to use it, which provides a clearer entry point. I also improved the Nix build system by externalizing large, embedded shell scripts from flake.nix into a separate nix/ directory and corrected a bug in the justfile's watch command. Finally, I refactored the Haskell source code by reorganizing it into a more logical structure with a top-level Simula module and added extensive Haddock documentation to the most critical files to clarify their purpose and make the codebase easier to understand.

Comment on lines +192 to +193
--prefix PATH : ${lib.makeBinPath passthru.simulaRuntimePrograms} \
--prefix LD_LIBRARY_PATH : ${lib.makeLibraryPath passthru.simulaRuntimeLibs}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

$ nix flake check -L
error: undefined variable 'passthru'
       at /home/haruki/program-dir/Simula/flake.nix:192:51:
          191|               makeWrapper $out/bin/simula-unwrapped $out/bin/simula \
          192|                 --prefix PATH : ${lib.makeBinPath passthru.simulaRuntimePrograms} \
             |                                                   ^
          193|                 --prefix LD_LIBRARY_PATH : ${lib.makeLibraryPath passthru.simulaRuntimeLibs}

Probably you can use nix flake check command. Please try it~

My environment is:

$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.12.53, NixOS, 25.11 (Xantusia), 25.11.20251018.3622652`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.31.2`
 - nixpkgs: `/nix/store/3qb1vcqah87n5hi5pd57m7jmjn78lpkb-source`

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants