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

qmc2 : init at 0.195 #35057

Merged
merged 1 commit into from Mar 10, 2018
Merged

qmc2 : init at 0.195 #35057

merged 1 commit into from Mar 10, 2018

Conversation

bignaux
Copy link
Contributor

@bignaux bignaux commented Feb 16, 2018

Motivation for this change
Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-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 using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@bignaux bignaux force-pushed the qmc2 branch 2 times, most recently from 6119b58 to a9078e8 Compare March 8, 2018 13:13
@bignaux bignaux changed the title qmc2 : init at 0.192 qmc2 : init at 0.195 Mar 8, 2018
@grahamc
Copy link
Member

grahamc commented Mar 8, 2018

@GrahamcOfBorg eval


src = fetchurl {
url = "https://netix.dl.sourceforge.net/project/qmc2/qmc2/${version}/${name}.tar.gz";
#url = "mirror://sourceforge/projects/qmc2/files/qmc2/${version}/${name}.tar.gz/download";
Copy link
Member

@grahamc grahamc Mar 8, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please delete this comment, and instead of using this netix... url use the mirror we have, like this: https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/services/misc/mantisbt.nix#L13

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

problem is that mirror is broken for me, never works and i get their custom 404 ... boring SF.

, minizip, zlib
, qtbase, qtsvg, qtmultimedia, qtwebkit, qttranslations, qtxmlpatterns
, rsync, SDL2, xwininfo
, utillinux #colrm
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please delete the colrm comment

xwininfo zlib utillinux xorg.libxcb ];

makeFlags = [ #"DEFINES-=QMC2_BUNDLED_MINIZIP"
#"DEFINES-=QMC2_BUNDLED_ZLIB"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please delete these commented lines

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i don't sucess to find good makeFlags to add them, so i'll keep in my maintainer notes for improve next time. Btw, it has no real consequences on the output.


meta = with stdenv.lib; {
description = "A Qt frontend for MAME/MESS";
homepage = http://qmc2.batcom-it.net;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This URL supports HTTPS, so please switch to that

@grahamc
Copy link
Member

grahamc commented Mar 8, 2018

lgtm other than these comments

@bignaux
Copy link
Contributor Author

bignaux commented Mar 8, 2018

Thanks for review @grahamc .

@grahamc
Copy link
Member

grahamc commented Mar 8, 2018

@GrahamcOfBorg build qmc2

@GrahamcOfBorg
Copy link

Failure on x86_64-darwin (full log)

Partial log (click to expand)

Package ‘qmc2-0.195’ in /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/misc/emulators/qmc2/default.nix:30 is not supported on ‘x86_64-darwin’, 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

Failure on x86_64-linux (full log)

Partial log (click to expand)

                 from qmc2main.cpp:43:
joystick.h:11:10: fatal error: SDL.h: No such file or directory
 #include <SDL.h>
          ^~~~~~~
compilation terminated.
make[1]: *** [Makefile.qmake:2888: qmc2main.o] Error 1
make[1]: Leaving directory '/tmp/nix-build-qmc2-0.195.drv-0/qmc2'
make: *** [Makefile:1043: qmc2-bin] Error 2
builder for ‘/nix/store/hqzqvpdj7yq51719zqm8ik5iifzfnbap-qmc2-0.195.drv’ failed with exit code 2
error: build of ‘/nix/store/hqzqvpdj7yq51719zqm8ik5iifzfnbap-qmc2-0.195.drv’ failed

@GrahamcOfBorg
Copy link

Failure on aarch64-linux (full log)

Partial log (click to expand)

                 from qmc2main.cpp:43:
joystick.h:11:10: fatal error: SDL.h: No such file or directory
 #include <SDL.h>
          ^~~~~~~
compilation terminated.
make[1]: *** [Makefile.qmake:2888: qmc2main.o] Error 1
make[1]: Leaving directory '/build/qmc2'
make: *** [Makefile:1043: qmc2-bin] Error 2
builder for '/nix/store/gvn1z007pmcm3lkn1jpkx8n2q4sckljz-qmc2-0.195.drv' failed with exit code 2
�[31;1merror:�[0m build of '/nix/store/gvn1z007pmcm3lkn1jpkx8n2q4sckljz-qmc2-0.195.drv' failed

@bignaux
Copy link
Contributor Author

bignaux commented Mar 8, 2018

@GrahamcOfBorg build qmc2

1 similar comment
@grahamc
Copy link
Member

grahamc commented Mar 8, 2018

@GrahamcOfBorg build qmc2

@GrahamcOfBorg
Copy link

Failure on x86_64-darwin (full log)

Partial log (click to expand)

Package ‘qmc2-0.195’ in /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/misc/emulators/qmc2/default.nix:34 is not supported on ‘x86_64-darwin’, 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)

post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/7jilbm0l68gsjffxyr1bsdpqm9yjf21j-qmc2-0.195
shrinking /nix/store/7jilbm0l68gsjffxyr1bsdpqm9yjf21j-qmc2-0.195/bin/qmc2-sdlmame
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/7jilbm0l68gsjffxyr1bsdpqm9yjf21j-qmc2-0.195/bin 
patching script interpreter paths in /nix/store/7jilbm0l68gsjffxyr1bsdpqm9yjf21j-qmc2-0.195
checking for references to /tmp/nix-build-qmc2-0.195.drv-0 in /nix/store/7jilbm0l68gsjffxyr1bsdpqm9yjf21j-qmc2-0.195...
postPatchMkspecs
postPatchMkspecs
/nix/store/7jilbm0l68gsjffxyr1bsdpqm9yjf21j-qmc2-0.195

@GrahamcOfBorg
Copy link

Success on aarch64-linux (full log)

Partial log (click to expand)

post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/jrjz8isgjzdxg8jihvkja78p9lj9pmbl-qmc2-0.195
shrinking /nix/store/jrjz8isgjzdxg8jihvkja78p9lj9pmbl-qmc2-0.195/bin/qmc2-sdlmame
strip is /nix/store/lvx1acn1ig1j2km8jds5x3ggh3f2wa8v-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/jrjz8isgjzdxg8jihvkja78p9lj9pmbl-qmc2-0.195/bin
patching script interpreter paths in /nix/store/jrjz8isgjzdxg8jihvkja78p9lj9pmbl-qmc2-0.195
checking for references to /build in /nix/store/jrjz8isgjzdxg8jihvkja78p9lj9pmbl-qmc2-0.195...
postPatchMkspecs
postPatchMkspecs
/nix/store/jrjz8isgjzdxg8jihvkja78p9lj9pmbl-qmc2-0.195

@7c6f434c 7c6f434c merged commit 00d6d77 into NixOS:master Mar 10, 2018
@bignaux bignaux deleted the qmc2 branch March 10, 2018 13:08
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

4 participants