Skip to content

Conversation

@demperor-music
Copy link

@demperor-music demperor-music commented Aug 8, 2025

Risolve #11.
Lo script compila il programma, scarica tutte le dipendenze e le impacchetta in un singolo file AppImage.
Lo script va eseguito nella cartella "Installer/SpaghettiCh2".

L'AppImage è stato testato su Arch, Fedora ed Alpine.

Per modificare il modo in cui l'app appare nel menù delle app se l'utente decide di installarla, modificare la sezione che inizia con "Creazione file .desktop..."

image image

Questo dovrebbe servire per prevenire problemi di compatibilità futuri con le versioni di Linux.
Lo script compila il programma, scarica tutte le dipendenze e le impacchetta in un singolo file AppImage.
Il file risultante è stato testato su Arch, Fedora ed Alpine.
@jcb936
Copy link
Member

jcb936 commented Aug 8, 2025

Nice - gli diamo un'occhiata e la testiamo!

Added quotes around variable names for safety
Something broke in the last commit, so i fixed it.
(a variable name somehow got cut out)
@jcb936 jcb936 self-requested a review August 9, 2025 14:39
cd $current_dir

echo -n "Compilazione del patcher..."
dotnet build -c Release -o $_pkgdir/opt/spaghettiproject
Copy link
Member

Choose a reason for hiding this comment

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

Qui andrebbe usato il comando di .net per deploy il progetto properly:

dotnet restore USPInstaller.csproj -r linux-x64
dotnet publish USPInstaller.csproj -r linux-x64 --self-contained

echo -n "Creazione script di avvio..."
echo "#!/bin/bash

\$APPDIR/usr/bin/dotnet \$APPDIR/opt/spaghettiproject/USPInstaller.dll" > "$_pkgdir/usr/bin/USPInstaller.sh"
Copy link
Member

Choose a reason for hiding this comment

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

Senza dover creare uno script di avvio - se cambi la generazione dell'eseguibile a dotnet publish --self-contained come da commento sopra hai tutto self contained per far partire tutto, senza dover anche avere il runtime e puoi runnare ./USPInstaller direttamente

@jcb936 jcb936 self-assigned this Aug 9, 2025
@jcb936 jcb936 requested review from CRefice and removed request for CRefice August 9, 2025 14:42
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