Skip to content

Conversation

ryand56
Copy link
Member

@ryand56 ryand56 commented Jun 25, 2025

Adds CoMaps, a community-led fork of Organic Maps.
https://codeberg.org/comaps/comaps

Closes #419794

Things done

  • 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/)
  • Nixpkgs 25.11 Release Notes (or backporting 24.11 and 25.05 Nixpkgs Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
  • NixOS 25.11 Release Notes (or backporting 24.11 and 25.05 NixOS Release notes)
    • (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, pkgs/README.md, maintainers/README.md and other contributing documentation in corresponding paths.

Add a 👍 reaction to pull requests you find important.

@nixpkgs-ci nixpkgs-ci bot added 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 Jun 25, 2025
Copy link
Contributor

@acid-bong acid-bong left a 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

@nixpkgs-ci nixpkgs-ci bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Jun 26, 2025
@ryand56 ryand56 marked this pull request as ready for review June 26, 2025 07:36
@ryand56
Copy link
Member Author

ryand56 commented Jun 26, 2025

@ofborg build comaps

@ryand56 ryand56 marked this pull request as draft June 26, 2025 22:56
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. and removed 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. labels Jun 28, 2025
@nixpkgs-ci nixpkgs-ci bot added 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. and removed 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. labels Jul 1, 2025
@ryand56 ryand56 changed the title comaps: init at 2025.06.22-5 comaps: init at 2025.06.30-22 Jul 1, 2025
@onny
Copy link
Contributor

onny commented Jul 29, 2025

Is this ready to review? Whats missing?

@ryand56
Copy link
Member Author

ryand56 commented Jul 29, 2025

Is this ready to review? Whats missing?

No, the map is missing symbols

@ryand56 ryand56 changed the title comaps: init at 2025.06.30-22 comaps: init at 2025.07.23-4 Jul 30, 2025
@ryand56 ryand56 marked this pull request as ready for review July 30, 2025 06:35
@ryand56
Copy link
Member Author

ryand56 commented Jul 30, 2025

I can download maps just fine but no symbols. Also can't launch over wayland, have to use QT_QPA_PLATFORM=xcb

@ryand56 ryand56 added the 8.has: package (new) This PR adds a new package label Jul 31, 2025
@acid-bong acid-bong mentioned this pull request Aug 10, 2025
13 tasks
Copy link
Member

@fgaz fgaz left a 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)

@ryand56
Copy link
Member Author

ryand56 commented Aug 11, 2025

@fgaz I'm getting the same jansson issue here, #432574 (comment)
Apparently, the libs are in jansson.out but it doesn't seem to detect it

@ryand56 ryand56 changed the title comaps: init at 2025.07.23-4 comaps: init at 2025.08.10 Aug 11, 2025
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. and removed 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. labels Aug 11, 2025
@ryand56 ryand56 marked this pull request as draft August 11, 2025 01:10
@ryand56 ryand56 force-pushed the init/comaps branch 2 times, most recently from 5433fb5 to e1ddd84 Compare August 16, 2025 11:16
@ryand56 ryand56 changed the title comaps: init at 2025.08.10 comaps: init at 2025.08.13-8 Aug 16, 2025
@ryand56
Copy link
Member Author

ryand56 commented Aug 17, 2025

Okay, I'm getting a new issue now:

comaps> -- Configuring done (18.1s)
comaps> -- Generating done (0.4s)
comaps> -- Build files have been written to: /build/source/out/release
comaps> [ 50%] Building CXX object base/CMakeFiles/base.dir/fast_math.cpp.o
comaps> [ 50%] Building CXX object base/CMakeFiles/base.dir/Unity/unity_1_cxx.cxx.o
comaps> [ 50%] Building CXX object base/CMakeFiles/base.dir/Unity/unity_0_cxx.cxx.o
comaps> [ 50%] Building CXX object 3party/gflags/CMakeFiles/gflags_nothreads_static.dir/Unity/unity_0_cxx.cxx.o
comaps> [ 50%] Linking CXX static library libgflags_nothreads.a
comaps> qcs: invalid option -- 'g'
comaps> Usage: /nix/store/pbqah1qk4b5y14fqinr1h8zvhqy71v81-gcc-wrapper-14.3.0/bin/ar [emulation options] [-]{dmpqrstx}[abcDfilMNoOPsSTuvV] [--plugin <name>] [member-name] [count] archive-file file...
comaps>        /nix/store/pbqah1qk4b5y14fqinr1h8zvhqy71v81-gcc-wrapper-14.3.0/bin/ar -M [<mri-script]
comaps>  commands:
comaps>   d            - delete file(s) from the archive
comaps>   m[ab]        - move file(s) in the archive
comaps>   p            - print file(s) found in the archive
comaps>   q[f]         - quick append file(s) to the archive
comaps>   r[ab][f][u]  - replace existing or insert new file(s) into the archive
comaps>   s            - act as ranlib
comaps>   t[O][v]      - display contents of the archive
comaps>   x[o]         - extract file(s) from the archive
comaps>  command specific modifiers:
comaps>   [a]          - put file(s) after [member-name]
comaps>   [b]          - put file(s) before [member-name] (same as [i])
comaps>   [D]          - use zero for timestamps and uids/gids (default)
comaps>   [U]          - use actual timestamps and uids/gids
comaps>   [N]          - use instance [count] of name
comaps>   [f]          - truncate inserted file names
comaps>   [P]          - use full path names when matching
comaps>   [o]          - preserve original dates
comaps>   [O]          - display offsets of files in the archive
comaps>   [u]          - only replace files that are newer than current archive contents
comaps>  generic modifiers:
comaps>   [c]          - do not warn if the library had to be created
comaps>   [s]          - create an archive index (cf. ranlib)
comaps>   [l <text> ]  - specify the dependencies of this library
comaps>   [S]          - do not build a symbol table
comaps>   [T]          - deprecated, use --thin instead
comaps>   [v]          - be verbose
comaps>   [V]          - display the version number
comaps>   @<file>      - read options from <file>
comaps>   --target=BFDNAME - specify the target object format as BFDNAME
comaps>   --output=DIRNAME - specify the output directory for extraction operations
comaps>   --record-libdeps=<text> - specify the dependencies of this library
comaps>   --thin       - make a thin archive
comaps>  optional:
comaps>   --plugin <p> - load the specified plugin
comaps>  emulation options:
comaps>   No emulation specific options
comaps> /nix/store/pbqah1qk4b5y14fqinr1h8zvhqy71v81-gcc-wrapper-14.3.0/bin/ar: supported targets: elf64-x86-64 elf32-i386 elf32-iamcu elf32-x86-64 pei-i386 pe-x86-64 pei-x86-64 elf64-little elf64-big elf32-little elf32-big srec symbolsrec verilog tekhex binary ihex plugin
comaps> make[3]: *** [3party/gflags/CMakeFiles/gflags_nothreads_static.dir/build.make:101: 3party/gflags/libgflags_nothreads.a] Error 1
comaps> make[2]: *** [CMakeFiles/Makefile2:2042: 3party/gflags/CMakeFiles/gflags_nothreads_static.dir/all] Error 2
comaps> make[2]: *** Waiting for unfinished jobs....
comaps> [ 50%] Linking CXX static library libbase.a
comaps> qcs: invalid option -- 'e'
comaps> Usage: /nix/store/pbqah1qk4b5y14fqinr1h8zvhqy71v81-gcc-wrapper-14.3.0/bin/ar [emulation options] [-]{dmpqrstx}[abcDfilMNoOPsSTuvV] [--plugin <name>] [member-name] [count] archive-file file...
comaps>        /nix/store/pbqah1qk4b5y14fqinr1h8zvhqy71v81-gcc-wrapper-14.3.0/bin/ar -M [<mri-script]
comaps>  commands:
comaps>   d            - delete file(s) from the archive
comaps>   m[ab]        - move file(s) in the archive
comaps>   p            - print file(s) found in the archive
comaps>   q[f]         - quick append file(s) to the archive
comaps>   r[ab][f][u]  - replace existing or insert new file(s) into the archive
comaps>   s            - act as ranlib
comaps>   t[O][v]      - display contents of the archive
comaps>   x[o]         - extract file(s) from the archive
comaps>  command specific modifiers:
comaps>   [a]          - put file(s) after [member-name]
comaps>   [b]          - put file(s) before [member-name] (same as [i])
comaps>   [D]          - use zero for timestamps and uids/gids (default)
comaps>   [U]          - use actual timestamps and uids/gids
comaps>   [N]          - use instance [count] of name
comaps>   [f]          - truncate inserted file names
comaps>   [P]          - use full path names when matching
comaps>   [o]          - preserve original dates
comaps>   [O]          - display offsets of files in the archive
comaps>   [u]          - only replace files that are newer than current archive contents
comaps>  generic modifiers:
comaps>   [c]          - do not warn if the library had to be created
comaps>   [s]          - create an archive index (cf. ranlib)
comaps>   [l <text> ]  - specify the dependencies of this library
comaps>   [S]          - do not build a symbol table
comaps>   [T]          - deprecated, use --thin instead
comaps>   [v]          - be verbose
comaps>   [V]          - display the version number
comaps>   @<file>      - read options from <file>
comaps>   --target=BFDNAME - specify the target object format as BFDNAME
comaps>   --output=DIRNAME - specify the output directory for extraction operations
comaps>   --record-libdeps=<text> - specify the dependencies of this library
comaps>   --thin       - make a thin archive
comaps>  optional:
comaps>   --plugin <p> - load the specified plugin
comaps>  emulation options:
comaps>   No emulation specific options
comaps> /nix/store/pbqah1qk4b5y14fqinr1h8zvhqy71v81-gcc-wrapper-14.3.0/bin/ar: supported targets: elf64-x86-64 elf32-i386 elf32-iamcu elf32-x86-64 pei-i386 pe-x86-64 pei-x86-64 elf64-little elf64-big elf32-little elf32-big srec symbolsrec verilog tekhex binary ihex plugin
comaps> make[3]: *** [base/CMakeFiles/base.dir/build.make:133: base/libbase.a] Error 1
comaps> make[2]: *** [CMakeFiles/Makefile2:2522: base/CMakeFiles/base.dir/all] Error 2
comaps> make[1]: *** [CMakeFiles/Makefile2:6245: skin_generator/CMakeFiles/skin_generator_tool.dir/rule] Error 2
comaps> make: *** [Makefile:1385: skin_generator_tool] Error 2

New issue after skipping all the downloads:

comaps> FAILED: 3party/bsdiff-courgette/libbsdiff.a
comaps> : && /nix/store/dx4bdrs7mq3jfviqhszrc7l35ps9kg64-cmake-3.31.7/bin/cmake -E rm -f 3party/bsdiff-courgette/libbsdiff.a && /nix/store/pbqah1qk4b5y14fqinr1h8zvhqy71v81-gcc-wrapper-14.3.0/bin/ar --plugin  qcs 3party/bsdiff-courgette/libbsdiff.a 3party/bsdiff-courgette/CMakeFiles/bsdiff.dir/Unity/unity_0_cxx.cxx.o && /nix/store/pbqah1qk4b5y14fqinr1h8zvhqy71v81-gcc-wrapper-14.3.0/bin/ranlib --plugin  3party/bsdiff-courgette/libbsdiff.a && :
comaps> qcs: invalid option -- '3'
comaps> Usage: /nix/store/pbqah1qk4b5y14fqinr1h8zvhqy71v81-gcc-wrapper-14.3.0/bin/ar [emulation options] [-]{dmpqrstx}[abcDfilMNoOPsSTuvV] [--plugin <name>] [member-name] [count] archive-file file...
comaps>        /nix/store/pbqah1qk4b5y14fqinr1h8zvhqy71v81-gcc-wrapper-14.3.0/bin/ar -M [<mri-script]
comaps>  commands:
comaps>   d            - delete file(s) from the archive
comaps>   m[ab]        - move file(s) in the archive
comaps>   p            - print file(s) found in the archive
comaps>   q[f]         - quick append file(s) to the archive
comaps>   r[ab][f][u]  - replace existing or insert new file(s) into the archive
comaps>   s            - act as ranlib
comaps>   t[O][v]      - display contents of the archive
comaps>   x[o]         - extract file(s) from the archive
comaps>  command specific modifiers:
comaps>   [a]          - put file(s) after [member-name]
comaps>   [b]          - put file(s) before [member-name] (same as [i])
comaps>   [D]          - use zero for timestamps and uids/gids (default)
comaps>   [U]          - use actual timestamps and uids/gids
comaps>   [N]          - use instance [count] of name
comaps>   [f]          - truncate inserted file names
comaps>   [P]          - use full path names when matching
comaps>   [o]          - preserve original dates
comaps>   [O]          - display offsets of files in the archive
comaps>   [u]          - only replace files that are newer than current archive contents
comaps>  generic modifiers:
comaps>   [c]          - do not warn if the library had to be created
comaps>   [s]          - create an archive index (cf. ranlib)
comaps>   [l <text> ]  - specify the dependencies of this library
comaps>   [S]          - do not build a symbol table
comaps>   [T]          - deprecated, use --thin instead
comaps>   [v]          - be verbose
comaps>   [V]          - display the version number
comaps>   @<file>      - read options from <file>
comaps>   --target=BFDNAME - specify the target object format as BFDNAME
comaps>   --output=DIRNAME - specify the output directory for extraction operations
comaps>   --record-libdeps=<text> - specify the dependencies of this library
comaps>   --thin       - make a thin archive
comaps>  optional:
comaps>   --plugin <p> - load the specified plugin
comaps>  emulation options:
comaps>   No emulation specific options
comaps> /nix/store/pbqah1qk4b5y14fqinr1h8zvhqy71v81-gcc-wrapper-14.3.0/bin/ar: supported targets: elf64-x86-64 elf32-i386 elf32-iamcu elf32-x86-64 pei-i386 pe-x86-64 pei-x86-64 elf64-little elf64-big elf32-little elf32-big srec symbolsrec verilog tekhex binary ihex plugin

@ryand56
Copy link
Member Author

ryand56 commented Aug 17, 2025

@ryand56 ryand56 force-pushed the init/comaps branch 2 times, most recently from 33b9aa8 to 14d56af Compare August 17, 2025 11:34
@ryand56 ryand56 mentioned this pull request Aug 17, 2025
13 tasks
@ryand56 ryand56 added 2.status: blocked by pr/issue Another PR or issue is preventing this from being completed and removed 2.status: blocked by pr/issue Another PR or issue is preventing this from being completed labels Aug 21, 2025
@ryand56 ryand56 marked this pull request as ready for review August 23, 2025 07:19
@onny
Copy link
Contributor

onny commented Aug 23, 2025

thank you!

i guess we should sooner or later make an own derivation for comaps instead of overriding organicmaps derivation here. otherwise LGTM

@onny onny merged commit 89dd666 into NixOS:master Aug 23, 2025
25 of 27 checks passed
@ryand56 ryand56 deleted the init/comaps branch August 23, 2025 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.has: package (new) This PR adds a new package 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 12.approvals: 1 This PR was reviewed and approved by one person.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Package request] CoMaps
4 participants