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

unison: fix on darwin #54448

Closed
wants to merge 1 commit into from

Conversation

ryanartecona
Copy link
Contributor

Motivation for this change

Unison is currently broken by default under Darwin due to a build
failure in libglade, which is a transient dependency for unison's gtk2
GUI.

Unison technically has a native macOS UI, so it should in principle be
possible to package it here and enable it instead of the gtk2 UI by
default in darwin, but I struggled to get there. Short of that, a
working unison install with a CLI but no GUI is preferable to no unison
at all on darwin.

Things done

The UISTYLE=text option tells the unison makefile to build without any
GUI support. This makes the package flag enableX11 = false effectively
disable any other unison GUI too, as a simple way of supporting the
darwin case above.


  • Tested using sandboxing (nix.useSandbox on NixOS, or option 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/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Assured whether relevant documentation is up to date
  • Fits CONTRIBUTING.md.

Unison is currently broken by default under Darwin due to a build
failure in libglade, which is a transient dependency for unison's gtk2
GUI.

Unison technically has a native macOS UI, so it should in principle be
possible to package it here and enable it instead of the gtk2 UI by
default in darwin, but I struggled to get there. Short of that, a
working unison install with a CLI but no GUI is preferable to no unison
at all on darwin.

The 'UISTYLE=text' option tells the unison makefile to build without any
GUI support. This makes the package flag 'enableX11 = false' effectively
disable any other unison GUI too, as a simple way of supporting the
darwin case above.
@matthewbauer
Copy link
Member

This is probably ok for now, but ideally we could fix libglade.

@vbgl
Copy link
Contributor

vbgl commented Jan 22, 2019

#54321 suggests a different fix. Any opinion?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants