Skip to content
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

Specified architecture for Debian / Ubuntu #88

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pasabanov
Copy link

It seems that the repository https://download.vscodium.com/debs vscodium main does not have any other architecture other than amd64.
Specifying the architecture removes the warning about the missing architecture when executing apt update if the user has the i386 architecture on his machine (for example, if he has steam installed).

@daiyam
Copy link
Member

daiyam commented Oct 29, 2024

There are amd64 and arm64.

@pasabanov
Copy link
Author

pasabanov commented Oct 29, 2024

There are amd64 and arm64.

So, maybe it's worth adding a note about these two options on the download page?
While it might seem redundant, it can be done concisely.
For example, below the Add the repository: command, it could be mentioned that the architecture can also be specified.

Because I think the warning that apt update gives:

N: Skipping acquire of configured file 'main/binary-i386/Packages' as repository 'https://download.vscodium.com/debs vscodium InRelease' doesn't support architecture 'i386'

is quite unpleasant.

@pasabanov
Copy link
Author

One solution could be to do what Microsoft does with VS Code: https://code.visualstudio.com/docs/setup/linux.

Quoting: echo "deb [arch=amd64,arm64,armhf signed-by=/etc/apt/keyrings/packages.microsoft.gpg] https://packages.microsoft.com/repos/code stable main" |sudo tee /etc/apt/sources.list.d/vscode.list > /dev/null.

Maybe we could change the command to deb [arch=amd64,arch=arm64 signed-by=/usr/share/keyrings/vscodium-archive-keyring.gpg] https://download.vscodium.com/debs vscodium main?

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.

2 participants