Skip to content

Conversation

RadxaPanda
Copy link
Contributor

No description provided.

@RadxaPanda RadxaPanda requested a review from RadxaYuntian as a code owner June 21, 2024 07:55
@RadxaPanda RadxaPanda marked this pull request as draft June 21, 2024 07:56
@RadxaYuntian
Copy link
Member

There is now a common function to get user account, then you can use it to get home.

@RadxaYuntian RadxaYuntian force-pushed the main branch 2 times, most recently from 7ca3725 to 4b92694 Compare July 11, 2024 10:08
Copy link
Member

@RadxaYuntian RadxaYuntian left a comment

Choose a reason for hiding this comment

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

Currently I can install steam with following commands:

sudo --preserve-env=https_proxy apt update
sudo --preserve-env=https_proxy apt install -y jq

cat << EOF >steam.sh
set -e
$(curl -Ls "https://github.com/raw/RadxaPanda/rsetup/main/src/usr/lib/rsetup/cli/steam.sh")
__parameter_count_check() {
    :
}
set -x
install_steam
EOF

sudo --preserve-env=https_proxy bash ./steam.sh

Running winecfg resulted in following error:

radxa@rock-5-itx:~$ wine winecfg
Box86 with Dynarec v0.3.7 fcbebfa8 built on Jul  7 2024 01:13:25
Error: file is not found (check BOX86_PATH)

steam can be launched from terminal.

@RadxaYuntian
Copy link
Member

After steam installer unpacked, there is a missing dependency of zenity, and seems crashed during loading of steamui.so.

@RadxaPanda RadxaPanda marked this pull request as ready for review July 15, 2024 02:40
@RadxaPanda
Copy link
Contributor Author

RadxaPanda commented Jul 15, 2024

Currently I can install steam with following commands:

sudo --preserve-env=https_proxy apt update
sudo --preserve-env=https_proxy apt install -y jq

cat << EOF >steam.sh
set -e
$(curl -Ls "https://github.com/raw/RadxaPanda/rsetup/main/src/usr/lib/rsetup/cli/steam.sh")
__parameter_count_check() {
    :
}
set -x
install_steam
EOF

sudo --preserve-env=https_proxy bash ./steam.sh

Running winecfg resulted in following error:

radxa@rock-5-itx:~$ wine winecfg
Box86 with Dynarec v0.3.7 fcbebfa8 built on Jul  7 2024 01:13:25
Error: file is not found (check BOX86_PATH)

steam can be launched from terminal.

Now with my latest commit wine winecfg does "nothing" but manually invoking ~/wine/bin/winecfg opens the config
And also steam works fine, i was able to play stardew valley

@RadxaPanda RadxaPanda requested a review from RadxaYuntian July 15, 2024 02:50
@RadxaYuntian
Copy link
Member

Also what is the complete installation process? Run the script then manually with ~/wine/bin/winecfg? If there is mandatory manual steps then the script should let the user know.

@RadxaPanda
Copy link
Contributor Author

Also what is the complete installation process? Run the script then manually with ~/wine/bin/winecfg? If there is mandatory manual steps then the script should let the user know.

Issue was caused by not properly escaping $@ in the script (was being set to nothing)

@RadxaYuntian RadxaYuntian requested a review from CodeChenL July 17, 2024 05:59
@RadxaYuntian
Copy link
Member

@CodeChenL please check if you can install and launch Steam & some Windows games successfully.

No need to test uninstall.

Copy link
Contributor

@CodeChenL CodeChenL left a comment

Choose a reason for hiding this comment

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

Native and proton translations work fine, but performance is low, only some 2D games with low resolution can be played.Like terraria.

@RadxaPanda
Copy link
Contributor Author

Native and proton translations work fine, but performance is low, only some 2D games with low resolution can be played.Like terraria.

When i tried this on panfork i was getting better performance than on panthor

@RadxaPanda RadxaPanda requested a review from RadxaYuntian July 25, 2024 02:31
@RadxaPanda
Copy link
Contributor Author

This should be able to be merged

@RadxaYuntian
Copy link
Member

Right now the functionality is tested, but the code still needs more clean up before it can be merged.

@RadxaPanda
Copy link
Contributor Author

My commit message is bad doesn't is spelled incorrectly

install_box64() {
curl -Ls https://ryanfortner.github.io/box64-debs/box64.list > /etc/apt/sources.list.d/box64.list
curl -Ls https://ryanfortner.github.io/box64-debs/KEY.gpg | gpg --dearmor --batch --yes -o /etc/apt/trusted.gpg.d/box64-debs-archive-keyring.gpg
apt-get update -y && apt-get install -y box64-rk3588
Copy link
Member

Choose a reason for hiding this comment

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

This can be listed in Depends of the repackaged steam.

install_box86() {
curl -Ls https://itai-nelken.github.io/weekly-box86-debs/debian/box86.list > /etc/apt/sources.list.d/box86.list
curl -Ls https://itai-nelken.github.io/weekly-box86-debs/debian/KEY.gpg | gpg --dearmor --batch --yes -o /etc/apt/trusted.gpg.d/box86-debs-archive-keyring.gpg
apt-get update -y && apt-get install -y box86
Copy link
Member

Choose a reason for hiding this comment

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

This can be listed in Depends of the repackaged steam.

Comment on lines +125 to +132
wget https://cdn.cloudflare.steamstatic.com/client/installer/steam.deb
ar x steam.deb
tar xf data.tar.xz

# move deb contents to steam folder
mv ./usr/* "$steam_dir"
popd || return 1
rm -rf "$temp_dir"
Copy link
Member

Choose a reason for hiding this comment

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

Those can be the content of the repackaged steam. Can we still install steam under say /usr/lib/x86_64-linux-gnu?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Definitely can but then ull need to rebuild once in a while I think but it should be fine

wine_pkg="$(sudo -u "$(logname)" mktemp -d)"
pushd "$wine_pkg" || return 1
latest_version="$(basename "$(curl -ILs -o /dev/null -w "%{url_effective}" https://github.com/Kron4ek/Wine-Builds/releases/latest)")"
curl -Lso "wine-$latest_version-x86.tar.xz" "https://github.com/Kron4ek/Wine-Builds/releases/download/$latest_version/wine-$latest_version-x86.tar.xz"
Copy link
Member

Choose a reason for hiding this comment

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

Make a wine package and install under /usr/lib/x86_64-linux-gnu as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants