Skip to content

genymotion: 2.7.2 -> 2.8.0 and add menu item #20749

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

Merged
merged 2 commits into from Nov 28, 2016
Merged

genymotion: 2.7.2 -> 2.8.0 and add menu item #20749

merged 2 commits into from Nov 28, 2016

Conversation

ghost
Copy link

@ghost ghost commented Nov 27, 2016

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
    • Linux
  • 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.

Sorry, something went wrong.

Alex Ivanov added 2 commits November 27, 2016 12:21

Verified

This commit was signed with the committer’s verified signature.
bagder Daniel Stenberg

Verified

This commit was signed with the committer’s verified signature.
bagder Daniel Stenberg
@mention-bot
Copy link

@gnidorah, thanks for your PR! By analyzing the history of the files in this pull request, we identified @puffnfresh to be a potential reviewer.

@@ -67,7 +75,7 @@ stdenv.mkDerivation rec {
'';
homepage = https://www.genymotion.com/;
license = stdenv.lib.licenses.unfree;
platforms = stdenv.lib.platforms.linux;
platforms = ["x86_64-linux"];
Copy link
Member

Choose a reason for hiding this comment

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

Does this not work on i686?

Copy link
Member

Choose a reason for hiding this comment

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

The 64-bit version is downloaded above:

name = "genymotion-${version}-linux_x64.bin";

Copy link
Author

Choose a reason for hiding this comment

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

And 32-bit version is not available :-(

@pSub pSub added the 8.has: package (update) This PR updates a package to a newer version label Nov 28, 2016
@grahamc grahamc merged commit ef138dc into NixOS:master Nov 28, 2016
@ghost ghost deleted the master3 branch November 28, 2016 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.has: package (update) This PR updates a package to a newer version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants