Skip to content

feat: add support for patching icedos modules#59

Open
IceDBorn wants to merge 1 commit into
mainfrom
feat/module-patches
Open

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

Conversation

@IceDBorn
Copy link
Copy Markdown
Contributor

@IceDBorn IceDBorn commented Jun 3, 2026

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 force-pushed the feat/module-patches branch from 255d2b3 to 24c4984 Compare June 5, 2026 17:02
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