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

etesync-dav: 0.20.0 -> 0.30.6 #105519

Merged
merged 3 commits into from Dec 7, 2020
Merged

etesync-dav: 0.20.0 -> 0.30.6 #105519

merged 3 commits into from Dec 7, 2020

Conversation

3699n
Copy link

@3699n 3699n commented Dec 1, 2020

Motivation for this change

etesync-dav was stuck on 0.20.0 due to higher versions depending on python bindings to etebase. This pull request updates etesync-dev and includes the nix expression for etebase.

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.

Copy link
Member

@SuperSandro2000 SuperSandro2000 left a comment

Choose a reason for hiding this comment

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

Also squash the maintainers adding commit with the update.

pkgs/applications/misc/etesync-dav/default.nix Outdated Show resolved Hide resolved
pkgs/applications/misc/etesync-dav/default.nix Outdated Show resolved Hide resolved
pkgs/development/python-modules/etebase/default.nix Outdated Show resolved Hide resolved
pkgs/development/python-modules/etebase/default.nix Outdated Show resolved Hide resolved
pkgs/development/python-modules/etebase/default.nix Outdated Show resolved Hide resolved
pkgs/development/python-modules/etebase/default.nix Outdated Show resolved Hide resolved
@SuperSandro2000
Copy link
Member

Result of nixpkgs-review pr 105519 run on x86_64-linux 1

4 packages built:
  • etesync-dav
  • python37Packages.etebase
  • python38Packages.etebase
  • python39Packages.etebase

@3699n 3699n force-pushed the master branch 2 times, most recently from de2e386 to d3a08ec Compare December 3, 2020 09:49
Copy link
Contributor

@Valodim Valodim left a comment

Choose a reason for hiding this comment

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

lgtm 👍 thanks for putting in the work

Copy link
Contributor

@Valodim Valodim left a comment

Choose a reason for hiding this comment

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

lgtm now. you might wanna add yourself to the maintainer list of etesync-dav too, so you get notifications on related PRs for that

@ofborg ofborg bot requested a review from Valodim December 3, 2020 10:26
@SuperSandro2000
Copy link
Member

Result of nixpkgs-review pr 105519 run on x86_64-linux 1

4 packages built:
  • etesync-dav
  • python37Packages.etebase
  • python38Packages.etebase
  • python39Packages.etebase

@SuperSandro2000
Copy link
Member

SuperSandro2000 commented Dec 3, 2020

patch to compile etebase on darwin:

diff --git a/pkgs/development/python-modules/etebase/default.nix b/pkgs/development/python-modules/etebase/default.nix
index c487aa98eb17..366c447ed942 100644
--- a/pkgs/development/python-modules/etebase/default.nix
+++ b/pkgs/development/python-modules/etebase/default.nix
@@ -1,4 +1,4 @@
-{ lib
+{ stdenv
 , wheel
 , rustPlatform
 , pipInstallHook
@@ -10,6 +10,7 @@
 , perl
 , rustfmt
 , fetchFromGitHub
+, Security
 }:
 
 rustPlatform.buildRustPackage rec {
@@ -34,6 +35,8 @@ rustPlatform.buildRustPackage rec {
     wheel
   ];
 
+  buildInputs = stdenv.lib.optionals stdenv.isDarwin [ Security ];
+
   propagatedBuildInputs = [
     python
     msgpack
@@ -49,7 +52,7 @@ rustPlatform.buildRustPackage rec {
     pipInstallPhase
   '';
 
-  meta = with lib; {
+  meta = with stdenv.lib; {
     homepage = "https://www.etebase.com/";
     description = "A Python client library for Etebase";
     license = licenses.bsd3;
diff --git a/pkgs/top-level/python-packages.nix b/pkgs/top-level/python-packages.nix
index 10328cb08e8c..5010c7dcc8be 100644
--- a/pkgs/top-level/python-packages.nix
+++ b/pkgs/top-level/python-packages.nix
@@ -1951,7 +1951,9 @@ in {
 
   etelemetry = callPackage ../development/python-modules/etelemetry { };
 
-  etebase = callPackage ../development/python-modules/etebase { };
+  etebase = callPackage ../development/python-modules/etebase {
+    inherit (pkgs.darwin.apple_sdk.frameworks) Security;
+  };
 
   etesync = callPackage ../development/python-modules/etesync { };

etesync-dav can be build when pyobjc is merged which is tracked here #105156

@SuperSandro2000
Copy link
Member

Result of nixpkgs-review pr 105519 run on x86_64-darwin 1

1 package failed to build:
  • etesync-dav
3 packages built:
  • python37Packages.etebase
  • python38Packages.etebase
  • python39Packages.etebase

@SuperSandro2000
Copy link
Member

Result of nixpkgs-review pr 105519 run on x86_64-linux 1

4 packages built:
  • etesync-dav
  • python37Packages.etebase
  • python38Packages.etebase
  • python39Packages.etebase

@SuperSandro2000 SuperSandro2000 merged commit 04cb54a into NixOS:master Dec 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants