Skip to content

Commit

Permalink
gajim: 0.16.8 -> 0.16.9
Browse files Browse the repository at this point in the history
Upstream changes:

  * Improve Zeroconf behavior
  * Fix showing normal message event
  * remove usage of OpenSSL.rand
  * a few minor bugfixes

The really important part here is the third point about OpenSSL.rand,
because the rand attribute no longer exists in pyopenssl and thus Gajim
doesn't even start.

Also the fix-tests.patch has been fixed upstream as well, so we don't
need it anymore.

Another change in 0.16.9 that's not included in the changelog is that
there is a test_nogui target, which is also run by the CI upstream is
using, so let's use that and remove xvfb_run.

Signed-off-by: aszlig <aszlig@nix.build>
Cc: @7c6f434c, @Mic92
  • Loading branch information
aszlig committed Feb 9, 2018
1 parent 669e03e commit 855e668
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 24 deletions.
15 changes: 4 additions & 11 deletions pkgs/applications/networking/instant-messengers/gajim/default.nix
@@ -1,9 +1,6 @@
{ stdenv, fetchurl, autoreconfHook, python, intltool, pkgconfig, libX11
, ldns, pythonPackages

# Test requirements
, xvfb_run

, enableJingle ? true, farstream ? null, gst-plugins-bad ? null
, libnice ? null
, enableE2E ? true
Expand All @@ -25,13 +22,13 @@ with stdenv.lib;

stdenv.mkDerivation rec {
name = "gajim-${version}";
version = "0.16.8";
version = "0.16.9";

src = fetchurl {
name = "${name}.tar.bz2";
url = "https://dev.gajim.org/gajim/gajim/repository/archive.tar.bz2?"
+ "ref=${name}";
sha256 = "009cpzqh4zy7hc9pq3r5m4lgagwawhjab13rjzavb0n9ggijcscb";
sha256 = "121dh906zya9n7npyk7b5xama0z3ycy9jl7l5jm39pc86h1winh3";
};

patches = let
Expand All @@ -46,8 +43,7 @@ stdenv.mkDerivation rec {
name = "gajim-${name}.patch";
url = "https://dev.gajim.org/gajim/gajim/commit/${rev}.diff";
inherit sha256;
}) cherries)
++ [./fix-tests.patch]; # https://dev.gajim.org/gajim/gajim/issues/8660
}) cherries);

postPatch = ''
sed -i -e '0,/^[^#]/ {
Expand All @@ -74,8 +70,6 @@ stdenv.mkDerivation rec {

nativeBuildInputs = [
autoreconfHook pythonPackages.wrapPython intltool pkgconfig
# Test dependencies
xvfb_run
];

autoreconfPhase = ''
Expand Down Expand Up @@ -114,9 +108,8 @@ stdenv.mkDerivation rec {

doInstallCheck = true;
installCheckPhase = ''
XDG_DATA_DIRS="$out/share/gajim''${XDG_DATA_DIRS:+:}$XDG_DATA_DIRS" \
PYTHONPATH="test:$out/share/gajim/src:''${PYTHONPATH:+:}$PYTHONPATH" \
xvfb-run make test
make test_nogui

This comment has been minimized.

Copy link
@Mic92

Mic92 Feb 10, 2018

Member

makes sense.

'';

enableParallelBuilding = true;
Expand Down

This file was deleted.

0 comments on commit 855e668

Please sign in to comment.