Conversation
|
ofborg hit a different error for Darwin: |
7c439cd to
285416a
Compare
|
The breakage on Darwin looks very similar to last time: even though we disable the built-in updater SparkleUpdater, it still seems to include sparkle in the build process. I have tried to override both the USE_UPDATER and WITH_UPDATER flags with no success yet... |
|
Hello! Still working on this? |
|
I can take a look at it again, my biggest issue is not having a Mac around to test the darwin build |
|
Feel free to ping me and I can help testing on darwin, I'm interested to try this package. :) |
|
@iedame Applied the patch, if you want to try building that on your side |
CI error likely to GitHub instability, can you re-trigger the CI checks please? |
|
|
I'm running nixpkgs-review but the diff looks really weird with all the commits and it will likely just say no rebuilds. After the results are posted here if you could please squash commits:
|
|
I'm not too certain about how to go about squashing commits, I did some googling but it's looking like it wants me to rebase ~68k commits which doesn't seem right |
Considering this is a very old PR and your first contribution, perhaps we could help in this PR: #451846 instead? Is it okay if we close this PR and re-open the one by @netcrns that is very recent and ready-to-go? You can open a new PR just adding yourself to the maintainers list and as maintainer to mudlet. |
|
Yeah that's completely fine with me |
|
Nice, thanks! Feel free to ping me when you open the PR to add yourself as maintainer, I can help review it. |
Things Done
Updated mudlet from 4.17.2 to 4.19.1
Removed MacOS patch as that was fixed upstream
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Add a 👍 reaction to pull requests you find important.