| 19 Apr 2025 |
alexfmpe | nope, and I can't reproduce either | 16:14:56 |
alexfmpe | package is unchanged on hackage since 2019 | 16:15:07 |
alexfmpe | maybe something else was borked? or bit flip I dunno | 16:15:25 |
maralorn | My intuition was that we added some new hydra job which creates an invalid combination. | 16:19:22 |
alexfmpe | oh wait a sec | 16:20:50 |
alexfmpe | that's the native build | 16:20:57 |
alexfmpe | hostPlatform.config = "x86_64-unknown-linux-gnu"
package.meta.platforms = [
"javascript-ghcjs"
]
package.meta.badPlatforms = [ ]
| 16:21:58 |
alexfmpe | shouldn't it, like, not try to eval? | 16:22:30 |
maralorn | Well the problem is that it has a dependency which doesn't eval. | 16:26:10 |
maralorn | So the correct solution seems to be to restrict the plattforms of miso-action-logger if that's factually correct. | 16:27:09 |
alexfmpe | but they are? | 16:27:32 |
alexfmpe | the error message says so | 16:27:43 |
alexfmpe | building locally I also get
a) To temporarily allow packages that are unsupported for this system, you can use an environment variable
for a single invocation of the nix tools.
$ export NIXPKGS_ALLOW_UNSUPPORTED_SYSTEM=1
| 16:28:04 |
maralorn | In reply to @alexfmpe:matrix.org but they are? No, the error message is showing a part of the ghcjs-base derivation. The miso-action-logger derivation has no restrictions. | 16:31:32 |
alexfmpe | oooh | 16:31:44 |
maralorn | I wonder if we have to null ghcjs-base or something... | 16:32:22 |
alexfmpe | I'll try a few things | 16:32:50 |
alexfmpe | IIRC miso itself didn't build for native in nixpkgs until recently, because cabal would find no libraries or anything after going through the conditionals | 16:33:33 |
alexfmpe | it just built for me now though, technically was cached | 16:33:53 |
alexfmpe | think that was changed by the 1.8.7 release | 16:34:13 |
alexfmpe | not enough because the package always unconditionally declares the dependency and imports from it | 16:37:43 |
alexfmpe | https://github.com/Lermex/miso-action-logger/issues/1 | 16:38:02 |
alexfmpe | I'll just flag as ghcjs only | 16:38:08 |
alexfmpe | someone put up a PR for this on large-records just 2 hours ago, but still needs sanity check so I'm not leaning towards even applying as patch until it's reviewed
I don't think we should block merge to staging on it, can just be PR'd directly to staging if upstream rubber stamps without an immediate release? | 16:45:09 |
alexfmpe |
https://github.com/well-typed/large-records/pull/171 | 16:45:16 |
alexfmpe | https://github.com/NixOS/nixpkgs/pull/400121 | 16:54:25 |
alexfmpe | hey we broke 400k | 16:54:29 |
sterni (he/him) | agree & agree & agree | 18:40:27 |
| alexfmpe changed their profile picture. | 23:55:28 |
| 20 Apr 2025 |
maralorn | In reply to @emilazy:matrix.org the 23rd is the nominal breaking change deadline and I assume there are breaking changes here If no one else does, I will finalize our merge on Tuesday. | 23:03:04 |