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
inspectrum: 20160403 -> 20170218 #23903
Conversation
liquid-dsp is a dependency for the latest git version of inspectrum, therefore I added it in order to be able to upgrade inspectrum.
src = fetchFromGitHub { | ||
owner = "jgaeddert"; | ||
repo = "liquid-dsp"; | ||
rev = "8c1978fa4f5662b8849fe712be716958f29cec0e"; |
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.
v1.3.0 is too old?
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.
v1.3.0 fails to compile:
/nix/store/ndqahbwrnabiaszk0s8cblhgwhid5kn0-gcc-5.4.0/lib/gcc/x86_64-unknown-linux-gnu/5.4.0/include/pmmintrin.h:50:1: error: inlining failed in call to always_inline '_mm_addsub_ps': target specific option mismatch
_mm_addsub_ps (__m128 __X, __m128 __Y)
^
src/dotprod/src/dotprod_cccf.mmx.c:255:11: error: called from here
s = _mm_addsub_ps( ci, cq );
^
In file included from src/dotprod/src/dotprod_cccf.mmx.c:49:0:
This seems to be fixed in the version I referenced.
|
||
stdenv.mkDerivation rec { | ||
name = "inspectrum-${version}"; | ||
version = "20160403"; | ||
version = "20170218"; |
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.
How about version 0.2, which was released on 3. February?
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.
Version 0.2 has some bugs, which are fixed in latest git. So the question is if it's preferred to have the version with the tag or the one containing the bugfix...
I would vote bugfix one but it's up to someone that can merge this. Just voting 😀 |
And as a dependency:
liquid-dsp: init at 20170307