tdesktop icon indicating copy to clipboard operation
tdesktop copied to clipboard

Telegram snap for armhf is an old version

Open archisman-panigrahi opened this issue 2 years ago • 8 comments

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

archisman-panigrahi avatar Oct 22 '22 06:10 archisman-panigrahi

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.

ilya-fedin avatar Oct 22 '22 06:10 ilya-fedin

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

archisman-panigrahi avatar Oct 23 '22 14:10 archisman-panigrahi

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.

ilya-fedin avatar Oct 23 '22 15:10 ilya-fedin

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?

xvitaly avatar Oct 27 '22 07:10 xvitaly

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

ilya-fedin avatar Oct 27 '22 07:10 ilya-fedin

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

ilya-fedin avatar Oct 27 '22 07:10 ilya-fedin

@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

xvitaly avatar Oct 27 '22 11:10 xvitaly

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

ilya-fedin avatar Oct 27 '22 14:10 ilya-fedin

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[bot] avatar Apr 26 '23 01:04 github-actions[bot]

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.

archisman-panigrahi avatar Apr 26 '23 03:04 archisman-panigrahi

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 avatar May 01 '23 19:05 ilya-fedin

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

ilya-fedin avatar Sep 22 '23 18:09 ilya-fedin