Skip to content

feat: add support for patching icedos modules#59

Merged
IceDBorn merged 1 commit into
mainfrom
feat/module-patches
Jun 27, 2026
Merged

feat: add support for patching icedos modules#59
IceDBorn merged 1 commit into
mainfrom
feat/module-patches

Conversation

@IceDBorn

@IceDBorn IceDBorn commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Add support for patching icedos modules and expose module input patching to users.

Module patching (renames celluloid input name):

--- a/modules/celluloid/icedos.nix
+++ b/modules/celluloid/icedos.nix
@@ -10,7 +10,7 @@
 in
 {
   inputs = {
-    celluloid-shader = {
+    celluloid-shader-repopatched = {
       url = "path://${toFile shaderFileName shader}";
       flake = false;
     };
@@ -33,7 +33,7 @@
 
               {
                 xdg.configFile."celluloid/celluloid.conf".source = ./celluloid.conf;
-                xdg.configFile."celluloid/shaders/FSR.glsl".source = inputs.celluloid-shader;
+                xdg.configFile."celluloid/shaders/FSR.glsl".source = inputs.celluloid-shader-repopatched;
 
                 dconf.settings = {
                   "io/github/celluloid-player/celluloid" = {
[[icedos.repositories]]
url = "github:icedos/apps"
patches = ["celluloid-rename.patch"]
modules = [ "celluloid" ]

Module input patching (adds a file to the prefixer input flake):

--- /dev/null	1970-01-01 00:00:00.000000000 +0000
+++ b/ICEDOS_CONSUMER_INPUT_PATCH	2026-06-03 00:00:00.000000000 +0000
@@ -0,0 +1 @@
+prefixer input patched from config (no fork)
[[icedos.repositories.inputPatches]]
module = "prefixer"
input = "prefixer"
patches = ["prefixer-consumer.patch"]

Paths are relative to the config root. So, these patches should be next to config.toml. You can specify any relative path you want, for example, "patches/patch.patch", as long as it exists under config root.

@IceDBorn

IceDBorn commented Jun 7, 2026

Copy link
Copy Markdown
Contributor Author

Any reviewers? @CrazyStevenz @jim3692

@IceDBorn IceDBorn force-pushed the feat/module-patches branch 2 times, most recently from 1cbe7cc to f39e0cc Compare June 14, 2026 15:18
@IceDBorn IceDBorn force-pushed the main branch 5 times, most recently from 1448afa to 8496519 Compare June 18, 2026 11:19
@IceDBorn IceDBorn force-pushed the feat/module-patches branch from f39e0cc to 74a21cd Compare June 18, 2026 16:50
@IceDBorn

IceDBorn commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

Tests to be done:

  • Ensure all configs working from config root
  • Ensure flake input patches work from module root's path

@IceDBorn IceDBorn force-pushed the feat/module-patches branch 2 times, most recently from a36185d to 145942c Compare June 26, 2026 23:10
@IceDBorn IceDBorn force-pushed the feat/module-patches branch from 145942c to f5c2f57 Compare June 27, 2026 17:59
@IceDBorn IceDBorn merged commit b828a33 into main Jun 27, 2026
@IceDBorn IceDBorn deleted the feat/module-patches branch June 27, 2026 18:42
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