Skip to content

Fixed arguments on appimage - #2818

Merged
daiyam merged 2 commits into
VSCodium:insiderfrom
redsigma:feat-allow-appimage-cli-commands
May 11, 2026
Merged

daiyam merged 2 commits into
VSCodium:insiderfrom
redsigma:feat-allow-appimage-cli-commands

Conversation

@redsigma

@redsigma redsigma commented May 1, 2026

Copy link
Copy Markdown
Contributor

When doing commands such as below, it opens an instance of codium instead of listing active extensions in cli or installing extensions.

vscodium.AppImage --list-extensions

vscodium.AppImage --install-extension

This PR fixes that by using a simpler approach rather than adjusting for each possible command. The following syntax will properly work and installing extensions are also properly working even with a custom --extensions-dir

vscodium.AppImage -- --list-extensions
vscodium.AppImage -- --install-extension

For running the app normally you can still use the following

vscodium.AppImage

vscodium.AppImage --user-data-dir /home/user/mycustomdata  --extensions-dir /home/user/mycustomextension

@redsigma redsigma changed the title fix(appimage): fixed arguments on appimage Fixed arguments on appimage May 1, 2026
Comment thread build/linux/appimage/recipe.yml Outdated
- export QT_PLUGIN_PATH="${HERE}"/usr/lib/qt4/plugins/:"${HERE}"/usr/lib/i386-linux-gnu/qt4/plugins/:"${HERE}"/usr/lib/x86_64-linux-gnu/qt4/plugins/:"${HERE}"/usr/lib32/qt4/plugins/:"${HERE}"/usr/lib64/qt4/plugins/:"${HERE}"/usr/lib/qt5/plugins/:"${HERE}"/usr/lib/i386-linux-gnu/qt5/plugins/:"${HERE}"/usr/lib/x86_64-linux-gnu/qt5/plugins/:"${HERE}"/usr/lib32/qt5/plugins/:"${HERE}"/usr/lib64/qt5/plugins/:"${QT_PLUGIN_PATH}"
- EXEC=$(grep -e '^Exec=.*' "${HERE}"/*.desktop | head -n 1 | cut -d "=" -f 2- | sed -e 's|%.||g')
- exec ${EXEC} "$@"
- EXEC="${HERE}/usr/share/@@APPNAME@@/@@APPNAME@@"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should be EXEC="${HERE}/usr/share/@@APPNAME@@/@@BINARY_NAME@@" (same thing for the one underneath).

BINARY_NAME: codium-insiders

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

i think build.sh has to be changed because @@BINARY_NAME@@ doesnt exist there

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

yep

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

should be good now

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Sorry about that, I didn't see that there was drift in the variable names... I will need to fix that.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

which variables need fixing ? I can take a look on that and include it in this PR

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

in

sed -i "s|@@NAME@@|${APP_NAME}|g" recipe.yml
sed -i "s|@@APPNAME@@|${BINARY_NAME}|g" recipe.yml
sed -i "s|@@BINARY_NAME@@|${BINARY_NAME}|g" recipe.yml

APP_NAME -> NAME
BINARY_NAME -> APPNAME

It should be

sed -i "s|@@APP_NAME@@|${APP_NAME}|g" recipe.yml
sed -i "s|@@BINARY_NAME@@|${BINARY_NAME}|g" recipe.yml

Thx

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

made the change. If you spot any issues let me know

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Sorry, you forgot to make the changes in the recipe.yml...

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

check again

@redsigma
redsigma force-pushed the feat-allow-appimage-cli-commands branch from 021f6b4 to 96ba126 Compare May 5, 2026 15:07
@daiyam daiyam added this to the N+1 milestone May 11, 2026
@redsigma
redsigma force-pushed the feat-allow-appimage-cli-commands branch from fbb2ea6 to 04b8e8a Compare May 11, 2026 14:01
@daiyam
daiyam merged commit 0c87817 into VSCodium:insider May 11, 2026
@daiyam

daiyam commented May 11, 2026

Copy link
Copy Markdown
Member

Thx

@redsigma
redsigma deleted the feat-allow-appimage-cli-commands branch June 8, 2026 09:09
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Sep 7, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants