Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] Signal-Desktop: update to 7.23.0.
@ 2024-09-07 23:06 MouadCharradi
  2024-09-15 18:02 ` classabbyamp
                   ` (7 more replies)
  0 siblings, 8 replies; 9+ messages in thread
From: MouadCharradi @ 2024-09-07 23:06 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 914 bytes --]

There is a new pull request by MouadCharradi against master on the void-packages repository

https://github.com/MouadCharradi/void-packages Signal-Desktop
https://github.com/void-linux/void-packages/pull/52141

Signal-Desktop: update to 7.23.0.
#### Testing the changes
- I tested the changes in this PR: **briefly**

#### Local build testing
- I built this PR locally for my native architecture, x86_64-glibc


Using `yarn` to build the package resulted in many errors, so I decided to use `npm` instead (I copied the commands from the way Arch Linux does it). I also added a desktop that I copied from `/usr/share/applications`, since I got a build error saying that it was not found without it.

This still needs some testing,
People who might know about this more than me, any suggestions are welcome. Thanks

A patch file from https://github.com/void-linux/void-packages/pull/52141.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-Signal-Desktop-52141.patch --]
[-- Type: text/x-diff, Size: 3000 bytes --]

From 246a8fa8a39b2eb779d401e793306714c77f5a19 Mon Sep 17 00:00:00 2001
From: Mouad Charradi <charradimouad58@gmail.com>
Date: Sun, 8 Sep 2024 00:01:12 +0100
Subject: [PATCH] Signal-Desktop: update to 7.23.0.

---
 .../files/signal-desktop.desktop              | 10 ++++++++
 srcpkgs/Signal-Desktop/template               | 23 ++++++++++---------
 2 files changed, 22 insertions(+), 11 deletions(-)
 create mode 100644 srcpkgs/Signal-Desktop/files/signal-desktop.desktop

diff --git a/srcpkgs/Signal-Desktop/files/signal-desktop.desktop b/srcpkgs/Signal-Desktop/files/signal-desktop.desktop
new file mode 100644
index 00000000000000..037b39f212fb77
--- /dev/null
+++ b/srcpkgs/Signal-Desktop/files/signal-desktop.desktop
@@ -0,0 +1,10 @@
+[Desktop Entry]
+Name=Signal
+Exec=signal-desktop --no-sandbox %U
+Terminal=false
+Type=Application
+Icon=signal-desktop
+StartupWMClass=Signal
+Comment=Private messaging from your desktop
+MimeType=x-scheme-handler/sgnl;x-scheme-handler/signalcaptcha;
+Categories=Network;InstantMessaging;Chat;
diff --git a/srcpkgs/Signal-Desktop/template b/srcpkgs/Signal-Desktop/template
index be7236daea620a..535e000a8fe9e5 100644
--- a/srcpkgs/Signal-Desktop/template
+++ b/srcpkgs/Signal-Desktop/template
@@ -1,6 +1,6 @@
 # Template file for 'Signal-Desktop'
 pkgname=Signal-Desktop
-version=7.21.0
+version=7.23.0
 revision=1
 # Signal officially only supports x86_64
 # x86_64-musl could potentially work based on the Alpine port:
@@ -14,7 +14,7 @@ maintainer="Orphaned <orphan@voidlinux.org>"
 license="AGPL-3.0-only"
 homepage="https://github.com/signalapp/Signal-Desktop"
 distfiles="https://github.com/signalapp/Signal-Desktop/archive/v${version}.tar.gz"
-checksum=4506a3639685dd191f713d70ed3c6624a77cafe8fd6ff3f113b340d2cd337526
+checksum=00b3d03050545cc5f6d3fc9c729627ca7068d75476fc33c791c171530689ed5d
 nostrip_files="signal-desktop"
 
 post_extract() {
@@ -23,18 +23,19 @@ post_extract() {
 
 	vsed 's/"node": ".*"/"node": ">=20.0.0"/' -i package.json
 
-	npm install -g yarn
-	# Dependencies have to be installed before applying patch
-	yarn install --ignore-engines --frozen-lockfile
+	# Install dependencies for sticker-creator
+	npm --prefix ./sticker-creator/ install
+
+	# Install dependencies for signal-desktop
+	npm install --ignore-engines
 }
 
 do_build() {
-	yarn generate
-	yarn build-release
+	# Build the sticker creator
+	npm --prefix ./sticker-creator/ run build
 
-	bsdtar xOf release/signal-desktop_*.deb data.tar.xz | \
-		bsdtar xO ./usr/share/applications/signal-desktop.desktop > signal-desktop.desktop
-	vsed -i -e 's/\/opt\/Signal\///' signal-desktop.desktop
+	# Build signal-desktop
+	npm run build
 }
 
 do_install() {
@@ -51,7 +52,7 @@ do_install() {
 	vmkdir usr/bin
 	ln -s /usr/lib/signal-desktop/signal-desktop ${DESTDIR}/usr/bin/
 
-	vinstall signal-desktop.desktop 644 usr/share/applications
+	vinstall ${FILESDIR}/signal-desktop.desktop 644 usr/share/applications
 
 	vmkdir usr/share/icons/hicolor
 	for size in 16 32 256; do

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2024-09-18  2:10 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-09-07 23:06 [PR PATCH] Signal-Desktop: update to 7.23.0 MouadCharradi
2024-09-15 18:02 ` classabbyamp
2024-09-15 18:04 ` [PR REVIEW] " classabbyamp
2024-09-15 18:57 ` lawmanuk
2024-09-15 19:10 ` classabbyamp
2024-09-16  5:10 ` [PR PATCH] [Updated] " MouadCharradi
2024-09-16  5:12 ` [PR REVIEW] " MouadCharradi
2024-09-16  5:18 ` [PR PATCH] [Updated] " MouadCharradi
2024-09-18  2:10 ` [PR PATCH] [Merged]: Signal-Desktop: update to 7.24.1 classabbyamp

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).