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

Telegram snap for armhf is an old version #25217

Closed
archisman-panigrahi opened this issue Oct 22, 2022 · 12 comments
Closed

Telegram snap for armhf is an old version #25217

archisman-panigrahi opened this issue Oct 22, 2022 · 12 comments

Comments

@archisman-panigrahi
Copy link

Steps to reproduce

Telegram for snap for armhf is still at version 4.0.2, whereas other architectures host the latest version 4.2.4.

Please update the armhf version of snap to the latest version.

Expected behaviour

The armhf snap should offer the latest telegram. Except snap, there is no other way to get the latest telegram on Raspberry Pi OS (armhf), since flatpak has stopped offering armhf builds. However, the snap hosts an older version of Telegram.

Actual behaviour

The armhf snap installs an older version of telegram.

Operating system

Raspberry Pi OS (bullseye)

Version of Telegram Desktop

4.0.2

Installation source

Snap

Logs

No response

@ilya-fedin
Copy link
Contributor

ilya-fedin commented Oct 22, 2022

Here's the build log of failing build attempt on Launchpad: https://launchpad.net/~telegram.desktop/+snap/telegram-desktop-master/+build/1896244/+files/buildlog_snap_ubuntu_jammy_armhf_telegram-desktop-master_BUILDING.txt.gz

Feel free to fix it if you know how to, personally I don't see a way.

@archisman-panigrahi
Copy link
Author

I asked a question in Launchpad, and someone pointed out some compiler flag which might be causing the issue. I was also asked to contact the people in the team https://launchpad.net/~telegram.desktop, but I don't know if they are the same people as you.

https://answers.launchpad.net/launchpad/+question/703579

@ilya-fedin
Copy link
Contributor

ilya-fedin commented Oct 23, 2022

Yeah, launchpad repository is owned by @john-preston, but I don't think he even looks there. Unfortunately, we have no idea what to do with this flag, we're just trying to build openssl 1.1 (as we're not compatible with openssl 3 provided by core22) and it fails on armhf. We unfortunately don't have enough experience in ARM-specific compiler flags (nor in openssl build system) to fix this on our own.

@xvitaly
Copy link
Contributor

xvitaly commented Oct 27, 2022

We unfortunately don't have enough experience in ARM-specific compiler flags (nor in openssl build system) to fix this on our own.

Can you try this patch?

@ilya-fedin
Copy link
Contributor

That patch seem to touch only armv8 (arm64) assembler code while the problem is with armv7 (armhf). arm64 builds just fine.

@ilya-fedin
Copy link
Contributor

ilya-fedin commented Oct 27, 2022

And I don't understand how patching assembler code would help with building C code (where the error triggers)

@xvitaly
Copy link
Contributor

xvitaly commented Oct 27, 2022

@ilya-fedin On Fedora it builds fine for armv7hl: https://koji.fedoraproject.org/koji/buildinfo?buildID=1997398

You can check all patch set here: https://src.fedoraproject.org/rpms/openssl/tree/f35

@ilya-fedin
Copy link
Contributor

ilya-fedin commented Oct 27, 2022

I believe it's related to Ubuntu gcc's embedded compiler flags as the flag it complains at is not present in command line

@github-actions
Copy link

Hey there!

This issue was inactive for a long time and will be automatically closed in 30 days if there isn't any further activity. We therefore assume that the user has lost interest or resolved the problem on their own.

Don't worry though; if this is an error, let us know with a comment and we'll be happy to reopen the issue.

Thanks!

@github-actions github-actions bot added the stale label Apr 26, 2023
@archisman-panigrahi
Copy link
Author

archisman-panigrahi commented Apr 26, 2023

I am commenting to keep the issue open. There are a lot of armv7 Raspberry Pi's out there, and there is no way to run latest Telegram natively on Debian/Ubuntu armhf.

@github-actions github-actions bot removed the stale label Apr 27, 2023
@ilya-fedin
Copy link
Contributor

I don't think this is fixable... Even though snap no more builds openssl, it now fails at building libyuv. It seem to fail in assembler code with the following creepy errors. It looks like the dependencies tdesktop relies on just don't care about armv7 anymore, so providing armv7 builds is no more feasible.
изображение

@ilya-fedin
Copy link
Contributor

ilya-fedin commented Sep 22, 2023

Looks like it got fixed after the last dependency update round
изображение

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants