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.

Sign up for GitHub

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

Jump to bottom

sequoia: 0.19.0 -> 0.20.0 #101753

Merged
merged 2 commits into from Oct 27, 2020
Merged

sequoia: 0.19.0 -> 0.20.0 #101753

merged 2 commits into from Oct 27, 2020

Conversation

doronbehar
Copy link
Contributor

Motivation for this change
Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@SuperSandro2000
Copy link
Member

SuperSandro2000 commented Oct 26, 2020 •
edited

This diff fixes building on darwin for me:

diff --git a/pkgs/tools/security/sequoia/default.nix b/pkgs/tools/security/sequoia/default.nix
index a54433b92c93..42edff5ca49e 100644
--- a/pkgs/tools/security/sequoia/default.nix
+++ b/pkgs/tools/security/sequoia/default.nix
@@ -62,6 +62,7 @@ rustPlatform.buildRustPackage rec {

   makeFlags = [
     "PREFIX=${placeholder "out"}"
+    "INSTALL=install"
   ];

   buildFlags = [
@@ -100,6 +101,5 @@ rustPlatform.buildRustPackage rec {
     homepage = "https://sequoia-pgp.org/";
     license = licenses.gpl3;
     maintainers = with maintainers; [ minijackson doronbehar ];
-    broken = stdenv.targetPlatform.isDarwin;
   };
 }

@doronbehar
Copy link
Contributor Author

This diff fixes building on darwin for me:

Awesome! Thanks :).

@marsam
Copy link
Contributor

marsam commented Oct 27, 2020

Would you mind checking whether we can drop llvmPackages_10 from buildInputs along with LIBCLANG_PATH?

@doronbehar
Copy link
Contributor Author

Would you mind checking whether we can drop llvmPackages_10 from buildInputs along with LIBCLANG_PATH?

No, fails the same way it used to fail before. FWIW, clang is listed officially as a dependency, so for sure it can't be dropped completely: https://gitlab.com/sequoia-pgp/sequoia/-/tree/v0.20.0#arch-linux . I tried more things - using llvmPackages_10.llvm and llvmPackages_10.llvm.out in inputs...

@doronbehar
Copy link
Contributor Author 

sequoia>    Compiling nettle-sys v2.0.4
sequoia> error: failed to run custom build command for `nettle-sys v2.0.4`
sequoia> Caused by:
sequoia> process didn't exit successfully: `/build/source/target/release/build/nettle-sys-1fcae42ffc344f15/build-script-build` (exit code: 101)
sequoia> --- stdout
sequoia> cargo:rerun-if-env-changed=NETTLE_STATIC
sequoia> cargo:rerun-if-env-changed=NETTLE_PREGENERATED_BINDINGS
sequoia> cargo:rerun-if-env-changed=NETTLE HOGWEED_NO_PKG_CONFIG
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG
sequoia> cargo:rerun-if-env-changed=NETTLE HOGWEED_STATIC
sequoia> cargo:rerun-if-env-changed=NETTLE HOGWEED_DYNAMIC
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_ALL_STATIC
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_ALL_DYNAMIC
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_PATH_x86_64-unknown-linux-gnu
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_PATH_x86_64_unknown_linux_gnu
sequoia> cargo:rerun-if-env-changed=HOST_PKG_CONFIG_PATH
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_PATH
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_LIBDIR_x86_64-unknown-linux-gnu
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_LIBDIR_x86_64_unknown_linux_gnu
sequoia> cargo:rerun-if-env-changed=HOST_PKG_CONFIG_LIBDIR
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_LIBDIR
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_SYSROOT_DIR_x86_64-unknown-linux-gnu
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_SYSROOT_DIR_x86_64_unknown_linux_gnu
sequoia> cargo:rerun-if-env-changed=HOST_PKG_CONFIG_SYSROOT_DIR
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_SYSROOT_DIR
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_SYSROOT_DIR
sequoia> cargo:rerun-if-env-changed=SYSROOT
sequoia> cargo:rerun-if-env-changed=NETTLE HOGWEED_STATIC
sequoia> cargo:rerun-if-env-changed=NETTLE HOGWEED_DYNAMIC
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_ALL_STATIC
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_ALL_DYNAMIC
sequoia> cargo:rustc-link-search=native=/nix/store/6qp2gdi9jc3hcjpa4bqmxalzxwvgb7sa-nettle-3.6/lib
sequoia> cargo:rustc-link-lib=nettle
sequoia> cargo:rustc-link-lib=hogweed
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG
sequoia> cargo:rerun-if-env-changed=NETTLE HOGWEED_STATIC
sequoia> cargo:rerun-if-env-changed=NETTLE HOGWEED_DYNAMIC
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_ALL_STATIC
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_ALL_DYNAMIC
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_PATH_x86_64-unknown-linux-gnu
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_PATH_x86_64_unknown_linux_gnu
sequoia> cargo:rerun-if-env-changed=HOST_PKG_CONFIG_PATH
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_PATH
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_LIBDIR_x86_64-unknown-linux-gnu
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_LIBDIR_x86_64_unknown_linux_gnu
sequoia> cargo:rerun-if-env-changed=HOST_PKG_CONFIG_LIBDIR
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_LIBDIR
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_SYSROOT_DIR_x86_64-unknown-linux-gnu
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_SYSROOT_DIR_x86_64_unknown_linux_gnu
sequoia> cargo:rerun-if-env-changed=HOST_PKG_CONFIG_SYSROOT_DIR
sequoia> cargo:rerun-if-env-changed=PKG_CONFIG_SYSROOT_DIR
sequoia> cargo:include=/nix/store/bfsnmn88nf3b54fh1n884lq6wp99j3ba-nettle-3.6-dev/include
sequoia> cargo:rustc-link-lib=dylib=nettle
sequoia> cargo:rustc-link-lib=dylib=hogweed
sequoia> cargo:rustc-link-search=native=/nix/store/6qp2gdi9jc3hcjpa4bqmxalzxwvgb7sa-nettle-3.6/lib
sequoia> cargo:rustc-link-lib=dylib=gmp
sequoia> --- stderr
sequoia> thread 'main' panicked at 'Unable to find libclang: "couldn\'t find any valid shared libraries matching: [\'libclang.so\', \'libclng-*.so\', \'libclang.so.*\'], set the `LIBCLANG_PATH` environment variable to a path where one of these files can be found (invalid: [])"'
 /build/sequoia-0.20.0-vendor.tar.gz/bindgen/src/lib.rs:1731:31
sequoia> note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
sequoia> warning: build failed, waiting for other jobs to finish...
sequoia> error: build failed
sequoia> make[1]: *** [Makefile:74: build-release] Error 101
sequoia> make[1]: Leaving directory '/build/source/openpgp-ffi'
sequoia> make: *** [Makefile:100: build-release] Error 2

@doronbehar doronbehar merged commit ccf24bf into NixOS:master Oct 27, 2020
@doronbehar doronbehar deleted the pkg/sequoia branch October 27, 2020 07:09
@puzzlewolf puzzlewolf mentioned this pull request Mar 3, 2021
Closed
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
10.rebuild-darwin: 1-10 10.rebuild-linux: 1-10 11.by: package-maintainer
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@doronbehar @SuperSandro2000 @marsam