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
pygtkglext: init at 1.1.0 #42555
pygtkglext: init at 1.1.0 #42555
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks mostly OK, some small changes are needed.
However, I'm not so sure we should add this package. It is indeed old. You mention you saw warnings, but if everything works fine without this package, why add it?
{ stdenv, fetchurl, buildPythonPackage, pkgconfig, gtk2, pygtk, gtkglext, pygobject2 | ||
, libglade ? null, libGLU, xorg, isPy3k }: | ||
|
||
stdenv.mkDerivation rec { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
buildPythonPackage
and pass format = "other";
stdenv.mkDerivation rec { | ||
pname = "pygtkglext"; | ||
version = "1.1.0"; | ||
name = pname + "-" + version; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The name
attribute is added by buildPython*
and should therefore be removed.
pygtk pygobject2 gtk2 gtkglext | ||
libGLU xorg.libXmu | ||
]; | ||
configureScript = "./configure"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this would not be needed.
Here are the logs from attaching to a locally running xpra session:
I'm not sure what it falls back to without OpenGL support, but that message seemed scary enough that it was worth looking into what provides |
This is a rather old library that is a dependency of the xpra client. If it's not provided, the xpra client warns you.
It looks like there is new xpra (2.3.1) which should support GTK 3 and Python 3. This is therefore unnecessary if we update the package. |
D'oh, I was testing with the version of xpra from 18.03. Thanks for the heads up. |
Note that our package still uses GTK 2, because it was auto-updated. It should be changed to Python 3 and Gtk 3 and then |
This is a rather old library that is a dependency of the xpra client.
If it's not provided, the xpra client warns you.
Motivation for this change
This library was missing and xpra was warning me.
This is my first new derivation, let me know if anything looks unnecessary. This package's setup.py doesn't even work, so I had to use the standard build process. Is there a way that the site-packages folder can automatically be added to Python's module search directories? I'm not sure it's totally functional at this point.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)