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
Do not depend on pangox_compat #75909
Conversation
Switched to HEAD to not depend on broken pangox-compat.
* Long overdue update * Clean up the expression * Drop unneeded pangox_compat – it was probably needed before gtkglext propagated it.
It was probably needed before gtkglext propagated it.
Drop unneeded pangox_compat – it was probably needed before gtkglext propagated it. Also stop using aliases.
It was probably needed before gtkglext propagated it.
1f7d4d9
to
c3bc037
Compare
Hmmph, 😄 as usual I wish nothing needed |
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.
Looks great. I did not runtime test k3d, but everything else that was touched here.
I think even trying to remove |
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.
diff LGTM
[242 built (2 failed), 351 copied (3026.1 MiB), 673.8 MiB DL]
error: build of '/nix/store/203mf6w1mwgdvvdlp2cv1zksvcxn4wsk-env.drv' failed
https://github.com/NixOS/nixpkgs/pull/75909
1 package are marked as broken and were skipped:
openmodelica
2 package failed to build:
anydesk gnash
8 package were built:
celestia desmume factor-lang gnome2.gtkglext k3d lutris lutris-free pcb
Do not depend on pangox_compat (cherry picked from commit 76c526a)
Most of them are: * separate packages for different openmodelica components, * qt4 -> qt5, * patches to instruct the OMEdit wrapper with stdenv executables location, * adoption of NixOS#89731 and NixOS#109595, * openblas -> blas, lapack according to NixOS#83888, * parallel building, * getting rid of spurious build phases, * correct the license, * cross-compilation, * forcing compiler to clang++ according to OM build recommendations, * drop of pangox_compat according to NixOS#75909 and NixOS#76412, * better dependencies, and more.
Co-authored-by: Jaakko Luttinen <jaakko.luttinen@iki.fi> Most of changes are: * separate packages for different openmodelica components, * qt4 -> qt5, * patches to instruct the OMEdit wrapper with stdenv executables location, * adoption of NixOS#89731 and NixOS#109595, * openblas -> blas, lapack according to NixOS#83888, * parallel building, * getting rid of spurious build phases, * correct the license, * cross-compilation, * forcing compiler to clang++ according to OM build recommendations, * drop of pangox_compat according to NixOS#75909 and NixOS#76412, * better dependencies, and more.
pango
recently brokepangox_compat
package but most packages do not actually need it – they were probably only adding it becausegtkglext
required it. We switched to master branch of gtkglext which fixes lot of issues and also no longer requires pangox.gnome2.gtkglext
pcb
k3d
lutris
desmume
anydesk
openmodelica
anydesk
seems to be an exception:but what can we expect from unfree software 😜️
openmodelica
seems to fail for unrelated reasons:Fixes: #75790