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

Update isync-git, with several other improvements #22872

Merged
merged 4 commits into from Feb 17, 2017
Merged

Conversation

ttuegel
Copy link
Member

@ttuegel ttuegel commented Feb 16, 2017

Motivation for this change

Although there has not been an isync release in several years, development continues upstream at a moderate pace, so it is important that we keep our Git-based version up-to-date.

Besides updating the package, I:

  • added myself to the maintainers
  • changed the src to download over HTTPS, and
  • added zlib support
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" (Not Applicable)
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

The extra dashes are removed from the version number because Nix treats only the
first component as the version, i.e. `2015-11-08` is version `2015` with two
tags `-11` and `-08`.
@mention-bot
Copy link

@ttuegel, thanks for your PR! By analyzing the history of the files in this pull request, we identified @the-kenny and @dezgeg to be potential reviewers.

@globin globin merged commit a6eb466 into NixOS:master Feb 17, 2017
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

3 participants