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

dropbox: use buildFHSUserEnv #31140

Merged
merged 1 commit into from
Nov 2, 2017
Merged

dropbox: use buildFHSUserEnv #31140

merged 1 commit into from
Nov 2, 2017

Conversation

ttuegel
Copy link
Member

@ttuegel ttuegel commented Nov 2, 2017

Instead of adapting Dropbox to NixOS with patchelf, NixOS is adapted to Dropbox
with an FHS user environment. A crash due to missing libXert (#15356) is
fixed. The client's automatic updater is fixed; this obviates the need to
update Dropbox in Nixpkgs every time the client is updated upstream!

Resolves: #15356

Testing

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

Instead of adapting Dropbox to NixOS with patchelf, NixOS is adapted to Dropbox
with an FHS user environment. A crash due to missing libXert (NixOS#15356) is
fixed. The client's automatic updater is fixed; this obviates the need to
update Dropbox in Nixpkgs every time the client is updated upstream!

Resolves: NixOS#15356
@ttuegel ttuegel added 0.kind: bug Something is broken 0.kind: enhancement Add something new 8.has: package (update) This PR updates a package to a newer version 9.needs: port to stable A PR needs a backport to the stable release. labels Nov 2, 2017
@ttuegel ttuegel self-assigned this Nov 2, 2017
@GrahamcOfBorg GrahamcOfBorg added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild 10.rebuild-linux: 0 This PR does not cause any packages to rebuild labels Nov 2, 2017
@ttuegel
Copy link
Member Author

ttuegel commented Nov 2, 2017

The CI tests pass on Linux. I cancelled the tests on macOS because it is not in meta.platforms.

@ttuegel ttuegel merged commit a513c07 into NixOS:master Nov 2, 2017
@ttuegel ttuegel deleted the dropbox branch November 2, 2017 17:42
@samueldr samueldr removed the 9.needs: port to stable A PR needs a backport to the stable release. label Apr 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.kind: bug Something is broken 0.kind: enhancement Add something new 8.has: package (update) This PR updates a package to a newer version 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild 10.rebuild-linux: 0 This PR does not cause any packages to rebuild
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot start Dropbox due to missing libXert
3 participants