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
Fix grab-site and python3Packages.lmdb #120785
Conversation
This makes no sense and yet was added in 65eccfa This was breaking the build of python3Packages.lmdb because ludios_wpull does not build on 3.8, only 3.7.
This has a fix for https://bugs.openldap.org/show_bug.cgi?id=9461, thus fixing pythonPackages.lmdb: jnwatson/py-lmdb#278
The C part looks good to me; I'm not really familiar with the python package. |
@ofborg build python3.pkgs.lmdb |
Result of 61 packages marked as broken and skipped:
1 package failed to build:48 packages skipped due to time constraints:
44 packages built successfully:
2 suggestions:
Note that build failures may predate this PR, and could be nondeterministic or hardware dependent. |
On Mon, Apr 26, 2021 at 07:57:58PM +0000, Ivan Kozik wrote:
This makes no sense and yet was added in 65eccfa
This was breaking the build of python3Packages.lmdb because ludios_wpull does not
build on 3.8, only 3.7
- propogatedBuildInputs = [ ludios_wpull ];
-
weird, good catch. ACK.
|
@@ -2,12 +2,12 @@ | |||
|
|||
stdenv.mkDerivation rec { | |||
pname = "lmdb"; | |||
version = "0.9.28"; | |||
version = "0.9.29"; | |||
|
|||
src = fetchgit { |
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.
This could be converted to fetchFromGitLab.
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.
Sure, but I hope that doesn't need to hold this up
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 please update the homepage to https://symas.com/lmdb/technical/
Motivation for this change
This fixes:
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)cc maintainers @copumpkin @jb55 @vcunat
I confirmed that grab-site now builds and runs on NixOS master.