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
sdcv: 0.4.2 -> 0.5.2 #31493
sdcv: 0.4.2 -> 0.5.2 #31493
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.
Thanks for your help refreshing an old package. :)
@@ -1,7 +1,9 @@ | |||
{ stdenv, fetchurl, pkgconfig, glib, gettext }: | |||
{ stdenv, fetchurl, pkgconfig, glib, gettext, fetchFromGitHub, cmake, |
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.
fetchFromGitHub
should replace fetchurl
since you are not using the latter anymore
|
||
stdenv.mkDerivation rec { | ||
name= "sdcv-0.4.2"; | ||
name = "sdcv-${version}"; | ||
version = "0.5.2"; | ||
|
||
meta = { | ||
homepage = http://sdcv.sourceforge.net/; |
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 page now redirects to https://dushistov.github.io/sdcv/ so we should set meta.homepage
to this new home.
preBuild = '' | ||
sed -i 's/guint32 page_size/size_t page_size/' src/lib/lib.cpp | ||
preInstall = '' | ||
touch locale |
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.
Just curious: why is this necessary?
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.
Without this:
CMake Error at cmake_install.cmake:60 (file):
file INSTALL cannot find
"/tmp/nix-build-sdcv-0.5.2.drv-0/source/build/locale".
#define _UTILS_HPP_ | ||
|
||
#include <string> | ||
+#include <cstdlib> |
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.
I don't see this patch at upstream: https://github.com/Dushistov/sdcv/blob/v0.5.2/src/utils.hpp
It's used on Darwin, and my concern is that dropping this particular patch will break Darwin build. I guess let's wait what Travis CI has to say on that.
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.
Travis passes.
Motivation for this change
The build was no longer working on latest fork of nixos-unstable.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)