-
-
Notifications
You must be signed in to change notification settings - Fork 16.6k
comaps: init at 2025.08.13-8 #419820
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
comaps: init at 2025.08.13-8 #419820
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great code, thx for the contribution. A couple of notes
@ofborg build comaps |
Is this ready to review? Whats missing? |
No, the map is missing symbols |
I can download maps just fine but no symbols. Also can't launch over wayland, have to use |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I accidentally submitted a pr for the same package: #432574
There are a few things from there that I'd like to see here as well:
- System dependencies (see cmakeFlags)
- updateScript
- Wayland support (qtwayland)
@fgaz I'm getting the same jansson issue here, #432574 (comment) |
5433fb5
to
e1ddd84
Compare
Okay, I'm getting a new issue now:
New issue after skipping all the downloads:
|
Okay, it seems to be an LTO related thing here: https://codeberg.org/comaps/comaps/src/commit/76ffc99abd714d84dc719c7bf549c54173a739b5/CMakeLists.txt#L125 |
33b9aa8
to
14d56af
Compare
thank you! i guess we should sooner or later make an own derivation for comaps instead of overriding organicmaps derivation here. otherwise LGTM |
Adds CoMaps, a community-led fork of Organic Maps.
https://codeberg.org/comaps/comaps
Closes #419794
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-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.