| 6 Aug 2025 |
dramforever | git archive generates empty directory for submodule | 04:21:29 |
dramforever | so i'm starting to think empty directory is "correct" | 04:21:41 |
cleverca22 | so fixing nix, would remove the one case where it works, and then pcsx-redux definitely needs a PR to fix it | 04:22:18 |
dramforever | yes | 04:24:36 |
dramforever | even though i guess it would be slightly weird now | 04:24:56 |
dramforever | because until nix gets fixed you kinda need to handle both cases | 04:25:15 |
cleverca22 | you just need to rmdir, and handle if its already missing | 04:25:33 |
cleverca22 | and then copy the submodule into place | 04:25:37 |
cleverca22 | and all of this mess, is only because flakes strip submodules! | 04:25:59 |
cleverca22 | https://github.com/grumpycoders/pcsx-redux/blob/main/pcsx-redux.nix#L39-L117 | 04:26:45 |
cleverca22 | this whole mess of code, is there to recreate the submodules | 04:26:55 |
dramforever | probably add -T to cp? | 04:27:32 |
dramforever | well you can get flakes to do submodules | 04:28:15 |
cleverca22 | ive tried before, and it doesnt actually do it for self | 04:28:36 |
dramforever | but you have to use git+something and then ?submodules=1 | 04:28:36 |
dramforever | like .?submodules=1 | 04:28:43 |
cleverca22 | for an input you can do it but for self, you cant | 04:28:47 |
dramforever | you can't do it by default | 04:29:09 |
cleverca22 | and that just complicates matters, now the user cant just nix build .# like every other project | 04:29:14 |
dramforever | yes | 04:29:24 |
cleverca22 | the flake.nix needs to be able to request that on its own | 04:29:26 |
dramforever | you also can't use github: for this | 04:30:02 |
cleverca22 | yeah, that does seem to work | 04:30:03 |
cleverca22 | dramforever: https://github.com/NixOS/nix/issues/13698 if you want to add any other comments or follow it | 04:37:07 |
cleverca22 | also added a TLDR, 2 sets of shell commands, where it fails to build twice with one
but succeeds the build with both on the other | 04:43:52 |
cleverca22 | the only difference, is the order of the nix build's | 04:43:59 |
emily | In reply to @cleverca22:matrix.org for an input you can do it but for self, you cant you can | 13:23:19 |
emily | a cursed syntax was added for it | 13:23:24 |
emily | involves multiple clones too | 13:23:30 |
emily | inputs.self.submodules | 13:23:37 |