Arch Linux User Repository

Please, remove from PKGBUILD line make -j17 the option -j17. First of, it gonna eat up user's RAM because that takes a lot of memory. I just had a lock up because of this. Second, users usually set the -jX number they prefer inside MAKEFLAGS in their /etc/makepkg.conf.

(ideally one wouldn't use make at all in preference of modern ninja since cmake does support such generator. But tdesktop seems to have something broken in their cmake configuration, and -GNinja option to cmake makes cmake configuration fail)

Check https://github.com/telegramdesktop/tdesktop/issues/6219

It appears that tdesktop-ffmpeg-fix-convertFromARGB32PM.patch and tdesktop-rlottie-static-qt.patch are no longer available

Hunk #1 succeeded at 235 (offset -5 lines).
patching file Telegram/SourceFiles/ui/style/style_core_font.cpp
patching file Telegram/Resources/qrc/telegram.qrc
patching file Telegram/SourceFiles/ui/twidget.cpp
Hunk #1 succeeded at 71 (offset 1 line).
patching file Telegram/SourceFiles/lottie/lottie_animation.cpp
Hunk #1 FAILED at 15.
1 out of 1 hunk FAILED -- saving rejects to file Telegram/SourceFiles/lottie/lottie_animation.cpp.rej
==> ERROR: A failure occurred in prepare().
    Aborting...
:: Unable to build telegram-desktop-git - makepkg exited with code: 4
=>> Try again? [y/N]: 

https://gist.github.com/bakatrouble/31caf6d64785f6a634dca3d292139898

Current tdesktop.patch (just used it to build version 1.7.0.r7.g63fc55263)

Note: you should replace sha256 for this file in PKGBUILD with 'SKIP'

Also owned the package and fixed the checksum, should work without manual fixes now

I solved the problem by the following steps: 1. Run makepkg to apply existing successful patches. 2. Edit Telegram/SourceFiles/core/launcher.cpp according to the tdesktop.patch: add 3 lines in Launcher::exec() and change a macro from #ifndef OS_MAC_OLD to #if !defined(Q_OS_MAC) && QT_VERSION >= QT_VERSION_CHECK(5, 6, 0). 3. Generate the correct patch by git diff > tdesktop.patch and replace the old one with this, and editing the sha256sums accordingly.

@bakatrouble: that hunk also results in build failure, or I'm not editing tdesktop.patch correctly.

To fix the tdesktop.patch file to work with current source code replace the second chunk (for Telegram/SourceFiles/core/launcher.cpp) with following snippet:

--- a/Telegram/SourceFiles/core/launcher.cpp
+++ b/Telegram/SourceFiles/core/launcher.cpp
@@ -209,11 +209,12 @@
    QApplication::setApplicationName(qsl("TelegramDesktop"));

 #if defined(Q_OS_LINUX) && QT_VERSION >= QT_VERSION_CHECK(5, 7, 0)
    QApplication::setDesktopFileName(qsl("telegramdesktop.desktop"));
 #endif
-#ifndef OS_MAC_OLD
+#if !defined(Q_OS_MAC) && QT_VERSION >= QT_VERSION_CHECK(5, 6, 0)
+   // Retina display support is working fine, others are not.
    QApplication::setAttribute(Qt::AA_DisableHighDpiScaling, true);
-#endif // OS_MAC_OLD
+#endif // not defined Q_OS_MAC and QT_VERSION >= 5.6.0

    initHook();
 }

I've disowned the package because I don't have enough time to prepare and maintain proper patches. Further reliance on those from the official repository, in the long run, won't work.

If someone has enough time and determination, I encourage to play with the package ;)

==> Validating source files with sha256sums...
    tdesktop ... Skipped
...
    no-gtk2.patch ... Passed
    tdesktop.patch ... FAILED
    Use-system-wide-font.patch ... Passed
==> ERROR: One or more files did not pass the validity check!