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

Update VapourSynth #35882

Merged
merged 1 commit into from Mar 5, 2018
Merged

Update VapourSynth #35882

merged 1 commit into from Mar 5, 2018

Conversation

rnhmjoj
Copy link
Contributor

@rnhmjoj rnhmjoj commented Feb 27, 2018

Motivation for this change

Usual version bump

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS)
  • 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
  • Tested execution of all binary files
  • Fits CONTRIBUTING.md.

@nlewo
Copy link
Member

nlewo commented Mar 2, 2018

@GrahamcOfBorg build vapoursynth

@GrahamcOfBorg
Copy link

Failure on aarch64-linux (full log)

Partial log (click to expand)

src/core/cpufeatures.c: In function 'getCPUFeatures':
src/core/cpufeatures.c:97:2: error: #error Do not know how to get CPU features on Linux.
 #error Do not know how to get CPU features on Linux.
  ^~~~~
src/core/cpufeatures.c:78:24: warning: unused variable 'hwcap' [-Wunused-variable]
     unsigned long long hwcap = getauxval(AT_HWCAP);
                        ^~~~~
make: *** [Makefile:1325: src/core/libvapoursynth_la-cpufeatures.lo] Error 1
builder for '/nix/store/66v0aw6v0alkd5szx3qy976b0wjlwmbd-vapoursynth-R43.drv' failed with exit code 2
�[31;1merror:�[0m build of '/nix/store/66v0aw6v0alkd5szx3qy976b0wjlwmbd-vapoursynth-R43.drv' failed

@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Partial log (click to expand)

shrinking /nix/store/9r857nl3bq1zfr4hbqlriik5s98vb03g-vapoursynth-R43/lib/vapoursynth/libremovegrain.so
shrinking /nix/store/9r857nl3bq1zfr4hbqlriik5s98vb03g-vapoursynth-R43/lib/vapoursynth/libmorpho.so
shrinking /nix/store/9r857nl3bq1zfr4hbqlriik5s98vb03g-vapoursynth-R43/lib/vapoursynth/libmiscfilters.so
shrinking /nix/store/9r857nl3bq1zfr4hbqlriik5s98vb03g-vapoursynth-R43/lib/libvapoursynth-script.so.0.0.0
shrinking /nix/store/9r857nl3bq1zfr4hbqlriik5s98vb03g-vapoursynth-R43/lib/python3.6/site-packages/vapoursynth.so
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/9r857nl3bq1zfr4hbqlriik5s98vb03g-vapoursynth-R43/lib  /nix/store/9r857nl3bq1zfr4hbqlriik5s98vb03g-vapoursynth-R43/bin 
patching script interpreter paths in /nix/store/9r857nl3bq1zfr4hbqlriik5s98vb03g-vapoursynth-R43
checking for references to /tmp/nix-build-vapoursynth-R43.drv-0 in /nix/store/9r857nl3bq1zfr4hbqlriik5s98vb03g-vapoursynth-R43...
/nix/store/9r857nl3bq1zfr4hbqlriik5s98vb03g-vapoursynth-R43

@nlewo
Copy link
Member

nlewo commented Mar 5, 2018

Could you please set the platform attribute to platforms = [ "x86_64-linux" ];?

@rnhmjoj
Copy link
Contributor Author

rnhmjoj commented Mar 5, 2018

darwin and windows should also be supported, so maybe platforms.x86_64.

@nlewo
Copy link
Member

nlewo commented Mar 5, 2018

@rnhmjoj yes, we can try this!

@GrahamcOfBorg build vapoursynth

@GrahamcOfBorg
Copy link

Failure on aarch64-linux (full log)

Partial log (click to expand)

Package ‘vapoursynth-R43’ in /var/lib/gc-of-borg/nix-test-rs-26/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-26/pkgs/development/libraries/vapoursynth/default.nix:39 is not supported on ‘aarch64-linux’, refusing to evaluate.

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Partial log (click to expand)

shrinking /nix/store/zdf63lhq7qx8w1yq2qbd8jszszz65pm5-vapoursynth-R43/lib/vapoursynth/libmiscfilters.so
shrinking /nix/store/zdf63lhq7qx8w1yq2qbd8jszszz65pm5-vapoursynth-R43/lib/vapoursynth/libimwri.so
shrinking /nix/store/zdf63lhq7qx8w1yq2qbd8jszszz65pm5-vapoursynth-R43/lib/libvapoursynth-script.so.0.0.0
shrinking /nix/store/zdf63lhq7qx8w1yq2qbd8jszszz65pm5-vapoursynth-R43/lib/python3.6/site-packages/vapoursynth.so
shrinking /nix/store/zdf63lhq7qx8w1yq2qbd8jszszz65pm5-vapoursynth-R43/lib/libvapoursynth.so
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/zdf63lhq7qx8w1yq2qbd8jszszz65pm5-vapoursynth-R43/lib  /nix/store/zdf63lhq7qx8w1yq2qbd8jszszz65pm5-vapoursynth-R43/bin 
patching script interpreter paths in /nix/store/zdf63lhq7qx8w1yq2qbd8jszszz65pm5-vapoursynth-R43
checking for references to /tmp/nix-build-vapoursynth-R43.drv-0 in /nix/store/zdf63lhq7qx8w1yq2qbd8jszszz65pm5-vapoursynth-R43...
/nix/store/zdf63lhq7qx8w1yq2qbd8jszszz65pm5-vapoursynth-R43

@rnhmjoj
Copy link
Contributor Author

rnhmjoj commented Mar 5, 2018

@LnL7 fixed the package some time ago to build on darwin. I suppose it still works.

@nlewo
Copy link
Member

nlewo commented Mar 5, 2018

Ok. Thanks.

@nlewo nlewo merged commit 53554bd into NixOS:master Mar 5, 2018
@rnhmjoj rnhmjoj deleted the vapoursynth branch February 23, 2019 09:39
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