Skip to content
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

martin: 0.9.1 → 0.14.2 #267315

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

martin: 0.9.1 → 0.14.2 #267315

wants to merge 1 commit into from

Conversation

sikmir
Copy link
Member

@sikmir sikmir commented Nov 13, 2023

Description of changes

Changelog

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/)
  • 23.11 Release Notes (or backporting 23.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.

@ofborg ofborg bot requested review from imincik, nh2 and willcohen November 14, 2023 00:31
@sikmir sikmir marked this pull request as draft November 14, 2023 07:16
@imincik
Copy link
Contributor

imincik commented Nov 14, 2023

That's a very interesting tool. I didn't know about it. Thanks for packaging !

@sikmir
Copy link
Member Author

sikmir commented Nov 19, 2023

protoc binary needs to be patched or replaced with one from nixpkgs:

failed to spawn command `"/build/martin-0.10.0-vendor.tar.gz/protoc-bin-vendored-linux-aarch_64/bin/protoc"

@sikmir sikmir changed the title martin: 0.9.1 → 0.10.0 martin: 0.9.1 → 0.11.0 Nov 20, 2023
@sikmir sikmir changed the title martin: 0.9.1 → 0.11.0 martin: 0.9.1 → 0.11.1 Nov 28, 2023
@ofborg ofborg bot requested a review from das-g November 29, 2023 00:33
@sikmir sikmir changed the title martin: 0.9.1 → 0.11.1 martin: 0.9.1 → 0.11.3 Dec 9, 2023
@sikmir
Copy link
Member Author

sikmir commented Dec 9, 2023

@nixos/rust Could you advise a proper way to fix it?

@winterqt
Copy link
Member

winterqt commented Dec 9, 2023

Either patch these binaries, patch martin to not use this crate, or symlink our binaries (from buildPackages) into the respective directories.

@sikmir
Copy link
Member Author

sikmir commented Dec 9, 2023

Either patch these binaries, patch martin to not use this crate, or symlink our binaries (from buildPackages) into the respective directories.

I've tried to patch the binary or symlink to nixpkgs protoc, but I've got:

martin> error: the listed checksum of `/build/martin-0.11.3-vendor.tar.gz/protoc-bin-vendored-linux-x86_64/bin/protoc` has changed:
martin> expected: 239cd7cc1888c8855bc2a7ed571d4af3cb9d255fde86b78c1fc1cdabf9248c14
martin> actual:   636f4f115e97302cda39133d3714414ea41cfd5c415d9da2c060e080fd3e8775
martin> directory sources are not intended to be edited, if modifications are required then it is recommended that `[patch]` is used with a forked copy of the source

cargoHash remained the same.

@ofborg ofborg bot requested a review from nialov December 9, 2023 21:49
@sikmir sikmir changed the title martin: 0.9.1 → 0.11.3 martin: 0.9.1 → 0.12.0 Dec 28, 2023
@ofborg ofborg bot requested a review from r-burns December 28, 2023 02:01
@sikmir sikmir changed the title martin: 0.9.1 → 0.12.0 martin: 0.9.1 → 0.13.0 Jan 1, 2024
@sikmir sikmir changed the title martin: 0.9.1 → 0.13.0 martin: 0.9.1 → 0.14.0 Jun 25, 2024
@ofborg ofborg bot requested a review from l0b0 June 25, 2024 21:38
Copy link
Member

@das-g das-g left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's inconsistent.

pkgs/servers/geospatial/martin/default.nix Outdated Show resolved Hide resolved
@sikmir sikmir changed the title martin: 0.9.1 → 0.14.0 martin: 0.9.1 → 0.14.2 Jun 27, 2024
@imincik
Copy link
Contributor

imincik commented Jun 27, 2024

Is this PR still a draft ?

@sikmir
Copy link
Member Author

sikmir commented Jun 27, 2024

Is this PR still a draft ?

protoc binary needs to be patched or replaced with one from nixpkgs:

failed to spawn command `"/build/martin-0.10.0-vendor.tar.gz/protoc-bin-vendored-linux-aarch_64/bin/protoc"

I still don't know how to do it.

@wegank wegank added the 2.status: merge conflict This PR has merge conflicts with the target branch label Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.status: merge conflict This PR has merge conflicts with the target branch 10.rebuild-darwin: 1-10 10.rebuild-darwin: 1 10.rebuild-linux: 1-10 10.rebuild-linux: 1 11.by: package-maintainer This PR was created by the maintainer of the package it changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants