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
etesync-dav: 0.20.0 -> 0.30.6 #105519
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.
Also squash the maintainers adding commit with the update.
0d2e145
to
24a4df8
Compare
Result of 4 packages built:
|
de2e386
to
d3a08ec
Compare
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.
lgtm 👍 thanks for putting in the work
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.
lgtm now. you might wanna add yourself to the maintainer list of etesync-dav too, so you get notifications on related PRs for that
Result of 4 packages built:
|
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 |
Result of 1 package failed to build:
3 packages built:
|
Result of 4 packages built:
|
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
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)