Re: git: 50119a0b4317 - main - net-im/telegram-desktop: Cleanup and set new maintainer

From: Baptiste Daroussin <bapt_at_FreeBSD.org>
Date: Tue, 23 Nov 2021 15:10:01 UTC
On Wed, Nov 17, 2021 at 05:08:01PM +0000, Olivier Cochard wrote:
> The branch main has been updated by olivier:
> 
> URL: https://cgit.FreeBSD.org/ports/commit/?id=50119a0b431718d00a22f424a550239246021ff9
> 
> commit 50119a0b431718d00a22f424a550239246021ff9
> Author:     Olivier Cochard <olivier@FreeBSD.org>
> AuthorDate: 2021-11-17 17:05:06 +0000
> Commit:     Olivier Cochard <olivier@FreeBSD.org>
> CommitDate: 2021-11-17 17:07:29 +0000
> 
>     net-im/telegram-desktop: Cleanup and set new maintainer
>     
>     Reported by:    glebius
> ---
>  net-im/telegram-desktop/Makefile                   | 17 +++++++-------
>  ...ceFiles_chat__helpers_stickers__emoji__pack.cpp | 11 ---------
>  ...Party_libtgvoip_os_posix_NetworkSocketPosix.cpp | 10 --------
>  ...ebrtc__dsp_rtc__base_platform__thread__types.cc | 27 ----------------------
>  ...Telegram_ThirdParty_tgcalls_tgcalls_Manager.cpp | 13 -----------
>  ...h-Telegram_ThirdParty_tgcalls_tgcalls_Manager.h | 10 --------
>  .../files/patch-Telegram_lib__base_base_bytes.h    | 20 ----------------
>  .../patch-Telegram_lib__base_base_zlib__help.h     | 13 -----------
>  ...ew_platform_linux_webview__linux__webkit__gtk.h | 10 --------
>  .../patch-cmake_external_rlottie_CMakeLists.txt    | 11 ---------
>  10 files changed, 9 insertions(+), 133 deletions(-)
> 
> diff --git a/net-im/telegram-desktop/Makefile b/net-im/telegram-desktop/Makefile
> index 40291a38b098..37f07d61f344 100644
> --- a/net-im/telegram-desktop/Makefile
> +++ b/net-im/telegram-desktop/Makefile
> @@ -6,7 +6,7 @@ MASTER_SITES=	https://github.com/${GH_ACCOUNT}/${GH_PROJECT}/releases/download/v
>  DISTNAME=	tdesktop-${DISTVERSION}-full
>  DISTFILES=	${DISTNAME}${EXTRACT_SUFX}
>  
> -MAINTAINER=	henry.hu.sh@gmail.com
> +MAINTAINER=	glebius@FreeBSD.org
>  COMMENT=	Telegram Desktop messaging app
>  
>  LICENSE=	GPLv3
> @@ -61,11 +61,8 @@ CMAKE_ARGS=	-DDESKTOP_APP_SPECIAL_TARGET="" -DTDESKTOP_LAUNCHER_BASENAME="telegr
>  		-DTDESKTOP_API_ID=${TELEGRAM_API_ID} -DTDESKTOP_API_HASH=${TELEGRAM_API_HASH} \
>  		-Ddisable_autoupdate=1
>  CMAKE_ON=	DESKTOP_APP_USE_PACKAGED DESKTOP_APP_DISABLE_CRASH_REPORTS \
> -		TDESKTOP_DISABLE_REGISTER_CUSTOM_SCHEME TDESKTOP_DISABLE_DESKTOP_FILE_GENERATION \
>  		DESKTOP_APP_DISABLE_WAYLAND_INTEGRATION
> -CMAKE_OFF=	DESKTOP_APP_USE_PACKAGED_RLOTTIE DESKTOP_APP_USE_PACKAGED_VARIANT \
> -		DESKTOP_APP_USE_GLIBC_WRAPS TDESKTOP_USE_PACKAGED_TGVOIP \
> -		DESKTOP_APP_USE_PACKAGED_FONTS
> +CMAKE_OFF=	DESKTOP_APP_USE_PACKAGED_FONTS
>  
>  # Disable DCHECKs, otherwise video calls will crash.
>  CXXFLAGS+=	-DNDEBUG
> @@ -73,6 +70,8 @@ CXXFLAGS+=	-DNDEBUG
>  # see Bugzilla #245452 for details
>  NO_CCACHE=	yes
>  
> +INSTALLS_ICONS=	yes

why bringing back INSTALLS_ICONS ? it is not needed anymore