10 Jun 2025 |
alexfmpe | in the meantime you should be able to workaround by using jsaddle-warp for reflex-dom | 15:14:35 |
maralorn | Uh, probably an excellent idea anyway. | 15:27:18 |
maralorn | Buuut, I guess we shouldn’t do that in nixpkgs, should we? 😄 | 15:30:38 |
maralorn | Huh, I wonder what is the correct way to detect that we are targeting js. ghc.isGhcjs does not exist anymore. | 15:37:58 |
alexfmpe | stdenv.hostPlatform.isGhcjs | 15:39:08 |
alexfmpe | but in the reflex-platform case, I thought warp was the default jsaddle | 15:39:57 |
alexfmpe | https://github.com/reflex-frp/reflex-platform/blob/2e0d31bc1b565eed02a3848cde0e0dbc4f33a2d4/project/default.nix#L120 | 15:39:58 |
alexfmpe | er, or rather, in r-p there's a nix setting for it | 15:40:11 |
alexfmpe | and in obelisk it was made the default | 15:40:16 |
maralorn | Nope | 15:40:37 |
maralorn | stdenv.targetPlatform.isJavascript | 15:40:55 |
alexfmpe | what | 15:41:08 |
alexfmpe | oh are you checking on the compiler or on the package? | 15:41:26 |
alexfmpe | because for the package it's https://github.com/NixOS/nixpkgs/blob/50015e7b86465fd5bd10d79ffe9567b4483d6279/pkgs/development/haskell-modules/generic-builder.nix#L24 | 15:41:33 |
maralorn | on the compiler | 15:41:37 |
maralorn | Ah, I see. | 15:41:45 |
alexfmpe | then it's shifted | 15:42:08 |
alexfmpe | so there it's targetPlatform.isGhcjs | 15:42:15 |
alexfmpe | because the compiler targets the platform the package it compiles is ran/hosted on | 15:43:27 |
alexfmpe | did I allucinate this? could have sworn it was the case but can't find it | 15:43:55 |
maralorn | I probably bricked that logic anyway by removing the reflex-plattform overrides | 15:45:43 |
maralorn | * | 15:45:55 |
alexfmpe | https://github.com/reflex-frp/reflex-platform/commit/93cb8ec441ebac3d68a9f982a1aa3ca55eb509e1 | 15:45:57 |
alexfmpe | this is probably what confused me | 15:46:00 |
maralorn | Okay good to know. Now I know how to solve this properly. 🙏 | 15:47:18 |
alexfmpe | probably not, though I was just wondering if the reflex-dom package should even have the jsaddle switch logic | 15:48:21 |
alexfmpe | after all, miso or whatever needs the exact same cabal if pyramid and cpp imports no? | 15:48:55 |
alexfmpe | shouldn't there be a jsaddle-cross or whatever where you always import your run :: JSM () -> IO () or so | 15:49:18 |
alexfmpe | and what that ends up as depends on platform and/or flags to that jsaddle package | 15:50:20 |
| dmjio changed their display name from dmjio to dmjio 🍜. | 17:04:02 |