[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#71897] [PATCH 4/5] gnu: Add libprisma-for-telegram-desktop.
From: |
Maxim Cournoyer |
Subject: |
[bug#71897] [PATCH 4/5] gnu: Add libprisma-for-telegram-desktop. |
Date: |
Tue, 09 Jul 2024 07:54:36 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
Hi Dan,
dan <i@dan.games> writes:
> Hi Maxim,
>
> On 7/9/2024 10:29 AM, Maxim Cournoyer wrote:
>> Hi,
>> dan <i@dan.games> writes:
>>
>>> * gnu/packages/telegram.scm (libprisma-for-telegram-desktop): New variable.
>>>
>>> Change-Id: I855b463993654a8cad491c7b56c9581b3083f5e4
>>> ---
>>> gnu/packages/telegram.scm | 12 ++++++++++++
>>> 1 file changed, 12 insertions(+)
>>>
>>> diff --git a/gnu/packages/telegram.scm b/gnu/packages/telegram.scm
>>> index 26ab27293f..0a48361eb4 100644
>>> --- a/gnu/packages/telegram.scm
>>> +++ b/gnu/packages/telegram.scm
>>> @@ -400,6 +400,18 @@ (define cld3-for-telegram-desktop
>>> (base32
>>> "0ayrrhfdwrf4260h9fsirkhhfrcvc3qqnh6h9wj3ixij2lq0wwqb"))))
>>> +(define libprisma-for-telegram-desktop
>>> + (origin
>>> + (method git-fetch)
>>> + (uri (git-reference
>>> + (url "https://github.com/desktop-app/libprisma")
>>> + (commit "23b0d70f9709da9b38561d5706891a134d18df76")))
>>> + (file-name
>>> + (git-file-name "libprisma-for-telegram-desktop" "5.2.3"))
>>> + (sha256
>>> + (base32
>>> + "0fg4x4ikj7f3706bmfvkwq4smxc98qr3cgpm25w48n4ys6wfgadg"))))
>>> +
>> So this is built as part of telegram? If this doesn't have any
>> proper
>> build system to be built separately and linked, or if telegram lacks
>> such a feature in its own build system (I like to report such feature
>> request for that to upstream myself), another perhaps more maintainable
>> strategy would be to use recursive? #t on the git-reference to clone
>> these, and delete in a source snippet the submodules we don't want to
>> have.
>
> Most of the dependencies of telegram-desktop are packaged this way, so
> here I'm more or less following the convention. de If we decide to use
> recursive? #t, I think it's better to fully revamp the packaging of
> telegram-desktop. libprisma itself doesn't have a build system, and I
> think it's only used in telegram-desktop.
OK, fair enough. Thanks for the explanation (I also noticed the tens of
git submodules as source variables in a later commit).
Reviewed-by: Maxim Cournoyer <maxim.cournoyer@gmail>
--
Thanks,
Maxim
[bug#71897] [PATCH 2/5] gnu: xdg-desktop-portal: Update input style., dan, 2024/07/07
[bug#71897] [PATCH 1/5] gnu: xdg-desktop-portal: Update to 1.18.4., Maxim Cournoyer, 2024/07/08
[bug#71897] [PATCH v5 1/6] gnu: xdg-desktop-portal: Update to 1.18.4., dan, 2024/07/14
- [bug#71897] [PATCH v5 2/6] gnu: xdg-desktop-portal: Update input style., dan, 2024/07/14
- [bug#71897] [PATCH v5 5/6] gnu: Add libprisma-for-telegram-desktop., dan, 2024/07/14
- [bug#71897] [PATCH v5 3/6] gnu: Add cppgir., dan, 2024/07/14
- [bug#71897] [PATCH v5 6/6] gnu: telegram-desktop: Update to 5.2.3., dan, 2024/07/14
- [bug#71897] [PATCH v5 4/6] gnu: Add cppgir-for-telegram-desktop., dan, 2024/07/14