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

gajim: 1.0.3 -> 1.1.2 #54817

Merged
merged 3 commits into from Jan 31, 2019
Merged

gajim: 1.0.3 -> 1.1.2 #54817

merged 3 commits into from Jan 31, 2019

Conversation

aszlig
Copy link
Member

@aszlig aszlig commented Jan 28, 2019

The list of upstream changes is huge, so I'm not pasting it here in the commit message, but here is the upstream URL:

https://dev.gajim.org/gajim/gajim/blob/gajim-1.1.2/ChangeLog

One of the most visible updates are the design changes for various dialogs and the Emoji overhauls.

On our end, we now need three more dependencies, namely cssutils, precis-i18n and keyring, which I added accordingly.

In addition, the test runner is now integrated into setup.py, which we now use.

I also cleaned up the package expression a bit, eg. it's no longer wrapped in a big with lib;, so that nix-instantiate --parse is able to detect attribute errors (which is very useful if you have editor integration).

@etu
Copy link
Contributor

etu commented Jan 28, 2019

@GrahamcOfBorg build python37Packages.precis-i18n python37Packages.nbxmpp python27Packages.nbxmpp gajim python27Packages.precis-i18n

@@ -12,16 +21,14 @@
, extraPythonPackages ? pkgs: [], pythonPackages
}:

with lib;

buildPythonApplication rec {
name = "gajim-${version}";
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
name = "gajim-${version}";
pname = "gajim";


# Python dependencies
, nbxmpp, pyasn1, pygobject3, gnome3, dbus-python, pillow, cssutils
, precis-i18n, keyring
Copy link
Member

Choose a reason for hiding this comment

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

Would you mind getting rid of python3Packages.callPackage in all-packages.nix and specifying python3 as argument instead?

@@ -12,16 +21,14 @@
, extraPythonPackages ? pkgs: [], pythonPackages
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
, extraPythonPackages ? pkgs: [], pythonPackages
, extraPythonPackages ? pkgs: []

@@ -1,7 +1,16 @@
{ buildPythonApplication, lib, fetchurl, gettext, wrapGAppsHook

# Native dependencies
, python, gtk3, gobject-introspection
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
, python, gtk3, gobject-introspection
, python3, gtk3, gobject-introspection

] ++ lib.optionals enableJingle [ farstream gstreamer gst-plugins-base gst-libav gst-plugins-ugly ]
++ lib.optional enableSecrets libsecret
++ lib.optional enableSpelling gspell
++ lib.optional enableUPnP gupnp-igd;

nativeBuildInputs = [
gettext wrapGAppsHook
];

propagatedBuildInputs = [
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
propagatedBuildInputs = [
propagatedBuildInputs = with python3.pkgs; [

nbxmpp pyasn1 pygobject3 dbus-python pillow cssutils precis-i18n keyring
] ++ lib.optionals enableE2E [ pycrypto python-gnupg ]
++ lib.optional enableRST docutils
++ lib.optionals enableOmemoPluginDependencies [ python-axolotl qrcode ]
++ extraPythonPackages pythonPackages;
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
++ extraPythonPackages pythonPackages;
++ extraPythonPackages python3.pkgs;

@dotlambda
Copy link
Member

A disabled = !isPy3k is missing.

@aszlig
Copy link
Member Author

aszlig commented Jan 28, 2019

A disabled = !isPy3k is missing.

I think that's only really useful if Gajim would be part of pythonPackages, which it is not and wouldn't actually make a lot of sense. So if I'm not missing something, it would only matter if you use gajim.override with say. Python 2.x.

Nevertheless, the module needs a bit of cleanup anyway, for example right now it uses both Python 2.x and Python 3.x dependencies (eg. due to that Jingle support has been broken since ages). I'm time-constrained right now however, so I only did minor cleanups with this PR.

@dotlambda
Copy link
Member

@dotlambda
Copy link
Member

Also, you should add the pythonPackages. prefix to two of the commit messages.

Just a small bugfix release, but required for Gajim 1.1.2.

Upstream fixes:

  * Always bind after SM failed
  * Dont try and guess system language

Signed-off-by: aszlig <aszlig@nix.build>
Closes: NixOS#54081
This package is required since Gajim version 1.1.0 and I intentionally
didn't set meta.maintainers because I'm not going to be able to maintain
this package, except if Gajim requires a newer version.

Signed-off-by: aszlig <aszlig@nix.build>
The list of upstream changes is huge, so I'm not pasting it here in the
commit message, but here is the upstream URL:

  https://dev.gajim.org/gajim/gajim/blob/gajim-1.1.2/ChangeLog

One of the most visible updates are the design changes for various
dialogs and the Emoji overhauls.

On our end, we now need three more dependencies, namely cssutils,
precis-i18n and keyring, which I added accordingly.

In addition, the test runner is now integrated into setup.py, which we
now use.

I also cleaned up the package expression a bit, eg. it's no longer
wrapped in a big "with lib;", so that "nix-instantiate --parse" is able
to detect attribute errors (which is very useful if you have editor
integration).

Signed-off-by: aszlig <aszlig@nix.build>
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