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
linux-steam-integration: init at 0.7.2 #33051
Conversation
}; | ||
|
||
steamBinPath = "${stdenv.lib.makeBinPath (with pkgs; [ steam ])}/steam"; | ||
zenityBinPath = "${stdenv.lib.makeBinPath (with pkgs; [ zenity ])}/zenity"; |
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.
these two can go in a let expression since they're not actual arguments for mkDerivation
010c358
to
37fa40c
Compare
Since I got the 32bit parts to build I think this PR is ready. It at least works on my 64bit systems and don't complain about missing 32bit libraries when running lsi-steam. |
915e76d
to
d85c9f7
Compare
cc @jagajaga, any opinions on this? |
Cc @abbradar |
b3258a2
to
105f8f5
Compare
homepage = https://github.com/solus-project/linux-steam-integration; | ||
license = licenses.lgpl21; | ||
maintainers = [ maintainers.etu ]; | ||
platforms = platforms.linux; |
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.
Should be platforms = [ "x86_64-linux" ];
.
stdenv.lib.platforms.linux
includes mips and arm arches where steam does not run.
name = "linux-steam-integration-${version}"; | ||
|
||
nativeBuildInputs = [ meson ninja pkgconfig git gettext gcc_multi ]; | ||
buildInputs = [ pkgs.gtk3 libressl ]; |
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.
It is better to take gtk3
as an argument rather than using gtk3
from the pkgs
attribute set.
105f8f5
to
bfd4036
Compare
bfd4036
to
d732c53
Compare
Motivation for this change
Linux Steam Integration is the foundation of the work that the Solus Project have done to get steam to "work better" on their OS. It seems like they solve a lot of problems with their wrappers and things.
Things to solve:
Here's the script used by archlinux to see how they build it both as 64 and 32 bit. https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=linux-steam-integration
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)