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

purple-lurch: init at 0.6.7 #40795

Merged
merged 2 commits into from May 20, 2018
Merged

purple-lurch: init at 0.6.7 #40795

merged 2 commits into from May 20, 2018

Conversation

emmanuelrosa
Copy link
Contributor

Motivation for this change

This change adds the pidgin/libpurple plugin lurch, which provides an implementation for XEP-0384: OMEMO Encryption. As long as the XMPP server of both parties supports OMEMO, this plugin allows for encrypted messaging. See https://github.com/gkdr/lurch

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 Technically should build on macOS, but I have no way of confirming.
    • 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 plugin using pidgin
  • Fits CONTRIBUTING.md.

omemo-1
ememo-2


@emmanuelrosa
Copy link
Contributor Author

To add plugin, add pidgin as follows:

environment.systemPackages = with pkgs; [
    (pidgin.override { plugins = [ purple-lurch]; })
];

Copy link
Member

@disassembler disassembler left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! Just some minor changes and I think this is ready.

{ stdenv, fetchFromGitHub, pidgin, minixml, libxml2, sqlite, libgcrypt, cmake }:

stdenv.mkDerivation rec {
name = "purple-lurch-0.6.7";
Copy link
Member

Choose a reason for hiding this comment

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

split version into it's separate variable

src = fetchFromGitHub {
owner = "gkdr";
repo = "lurch";
rev = "3156e14c2dcf9968438aceb7fda8f82eae6f552d";
Copy link
Member

Choose a reason for hiding this comment

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

Since is a release on upstream github repo, just do "v${version}" instead of specifying rev manually.

@orivej orivej changed the title purple-lurch: init 0.6.7 purple-lurch: init at 0.6.7 May 20, 2018
Emmanuel Rosa added 2 commits May 20, 2018 21:32
Adds XEP-0384: OMEMO Encryption for libpurple
@orivej orivej merged commit 0485ade into NixOS:master May 20, 2018
@emmanuelrosa emmanuelrosa deleted the lurch branch May 20, 2018 22:56
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

4 participants