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

virtualbox: 5.1.18 -> 5.1.22 #25368

Merged
merged 1 commit into from Jun 23, 2017
Merged

Conversation

bachp
Copy link
Member

@bachp bachp commented Apr 30, 2017

Motivation for this change

Update to latest upstream version.

Unfortunalty it seems it is still not compatible with X.org ABI 1.19.

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.

@mention-bot
Copy link

@bachp, thanks for your PR! By analyzing the history of the files in this pull request, we identified @edolstra, @globin and @aszlig to be potential reviewers.

@bachp
Copy link
Member Author

bachp commented May 7, 2017

ping

@bachp
Copy link
Member Author

bachp commented May 15, 2017

@globin could you have a look at this PR?

@max-wittig
Copy link
Contributor

Would be a really cool change!. Guest additions are also updated. The guest additions notification also annoys me. This PR would help fix it.

@bachp
Copy link
Member Author

bachp commented Jun 14, 2017

Is there a reason this PR doesn't get any attention?

@NeQuissimus NeQuissimus merged commit 6fb9f89 into NixOS:master Jun 23, 2017
@aszlig
Copy link
Member

aszlig commented Jun 23, 2017

This actually doesn't build nor do any of the tests succeed (because of the former).

@aszlig
Copy link
Member

aszlig commented Jun 23, 2017

Working on fixing that patch...

@aszlig
Copy link
Member

aszlig commented Jun 23, 2017

Fixed in 63fb845.

@NeQuissimus
Copy link
Member

Oh, thank you @aszlig, I had only paid attention to the SHA passing

@bachp
Copy link
Member Author

bachp commented Jun 23, 2017

@aszlig thanks for the fix

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

6 participants