Skip to content

mudlet: 4.17.2 -> 4.19.1#389652

Closed
TheWanderingCrow wants to merge 7 commits intoNixOS:masterfrom
TheWanderingCrow:update-mudlet
Closed

mudlet: 4.17.2 -> 4.19.1#389652
TheWanderingCrow wants to merge 7 commits intoNixOS:masterfrom
TheWanderingCrow:update-mudlet

Conversation

@TheWanderingCrow
Copy link
Contributor

Things Done

Updated mudlet from 4.17.2 to 4.19.1
Removed MacOS patch as that was fixed upstream

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@github-actions github-actions bot added the 6.topic: games Gaming on NixOS label Mar 13, 2025
@NixOSInfra NixOSInfra added the 12.first-time contribution This PR is the author's first one; please be gentle! label Mar 13, 2025
@github-actions github-actions bot added 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Mar 13, 2025
@nix-owners nix-owners bot requested review from felixalbrigtsen and pstn March 13, 2025 21:24
@FliegendeWurst
Copy link
Member

ofborg hit a different error for Darwin:

> /tmp/nix-build-mudlet-4.19.1.drv-0/source/src/sparkleupdater.mm:23:9: fatal error: 'Sparkle/Sparkle.h' file not found
   >    23 | #import <Sparkle/Sparkle.h>
   >       |         ^~~~~~~~~~~~~~~~~~~
   > 1 error generated.

@felixalbrigtsen
Copy link
Member

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...

@nixpkgs-ci nixpkgs-ci bot added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Sep 14, 2025
@netcrns netcrns mentioned this pull request Oct 18, 2025
13 tasks
@iedame
Copy link
Contributor

iedame commented Oct 22, 2025

Hello! Still working on this?
You might want to apply patch Mudlet/Mudlet@933f255#diff-148715d6ea0c0ea0a346af3f6bd610d010d490eca35ac6a9b408748f7ca9e3f4 to build on cmake4.

@TheWanderingCrow
Copy link
Contributor Author

I can take a look at it again, my biggest issue is not having a Mac around to test the darwin build

@nixpkgs-ci nixpkgs-ci bot removed the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Oct 22, 2025
@iedame
Copy link
Contributor

iedame commented Oct 22, 2025

Feel free to ping me and I can help testing on darwin, I'm interested to try this package. :)

@nixpkgs-ci nixpkgs-ci bot added the 2.status: merge conflict This PR has merge conflicts with the target branch label Oct 23, 2025
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. and removed 2.status: merge conflict This PR has merge conflicts with the target branch 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. labels Oct 23, 2025
@TheWanderingCrow
Copy link
Contributor Author

@iedame Applied the patch, if you want to try building that on your side

@iedame
Copy link
Contributor

iedame commented Oct 23, 2025

@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?

@nixpkgs-ci nixpkgs-ci bot added the 8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` label Oct 23, 2025
@iedame
Copy link
Contributor

iedame commented Oct 23, 2025

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 389652
Commit: 3cd4f72dc8c32607e501f8e75d46a6e6e67fb3de (subsequent changes)
Merge: 33599ffb039db1e36af388a20a07ca47f29f552d

Logs: https://github.com/iedame/nixpkgs-review-gha/actions/runs/18750815502


x86_64-linux

No rebuilds


aarch64-linux

No rebuilds


x86_64-darwin

No rebuilds


aarch64-darwin

No rebuilds

@iedame
Copy link
Contributor

iedame commented Oct 23, 2025

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:

  • maintainers: add thewanderingcrow
  • mudlet: 4.17.2 -> 4.19.1
  • mudlet: add maintainer thewanderingcrow

@TheWanderingCrow
Copy link
Contributor Author

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

@iedame
Copy link
Contributor

iedame commented Oct 23, 2025

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?
I understand it is a little confusing at first, you will get used to it. For now you can read the CONTRIBUTING.md, it explains many things.

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.

@TheWanderingCrow
Copy link
Contributor Author

Yeah that's completely fine with me

@iedame
Copy link
Contributor

iedame commented Oct 23, 2025

Nice, thanks! Feel free to ping me when you open the PR to add yourself as maintainer, I can help review it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: games Gaming on NixOS 8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. 12.first-time contribution This PR is the author's first one; please be gentle!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants