Github messages for voidlinux
 help / color / mirror / Atom feed
From: reedts <reedts@users.noreply.github.com>
To: ml@inbox.vuxu.org
Subject: Re: [PR PATCH] [Updated] Signal-Desktop: update to 5.4.1.
Date: Mon, 28 Jun 2021 09:50:09 +0200	[thread overview]
Message-ID: <20210628075009.jHyqaTJXPuHcyda49SY7B6D92ZZU6CuY-RMYhjDm7jA@z> (raw)
In-Reply-To: <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-31521@inbox.vuxu.org>

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

There is an updated pull request by reedts against master on the void-packages repository

https://github.com/reedts/void-packages signal-v5.4.1
https://github.com/void-linux/void-packages/pull/31521

Signal-Desktop: update to 5.4.1.
<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [x] I built this PR locally for my native architecture, (x86_64-glibc)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-signal-v5.4.1-31521.patch --]
[-- Type: text/x-diff, Size: 4274 bytes --]

From 6480c209274ab2dc60892963a4aa325ea4834caf Mon Sep 17 00:00:00 2001
From: reedts <j.reedts@gmail.com>
Date: Wed, 16 Jun 2021 09:56:32 +0200
Subject: [PATCH] Signal-Desktop: update to 5.6.2.

---
 .../patches/build-from-tarball.patch          |  9 -----
 .../expire-from-source-date-epoch.patch       | 13 +++++++
 srcpkgs/Signal-Desktop/template               | 35 +++++++++++++------
 3 files changed, 38 insertions(+), 19 deletions(-)
 delete mode 100644 srcpkgs/Signal-Desktop/patches/build-from-tarball.patch
 create mode 100644 srcpkgs/Signal-Desktop/patches/expire-from-source-date-epoch.patch

diff --git a/srcpkgs/Signal-Desktop/patches/build-from-tarball.patch b/srcpkgs/Signal-Desktop/patches/build-from-tarball.patch
deleted file mode 100644
index cab451f7f35c..000000000000
--- a/srcpkgs/Signal-Desktop/patches/build-from-tarball.patch
+++ /dev/null
@@ -1,9 +0,0 @@
---- a/Gruntfile.js	2020-04-26 23:32:09.861789510 +0200
-+++ b/Gruntfile.js	2020-04-26 23:32:21.424734031 +0200
-@@ -452,6 +452,5 @@
-     'concat',
-     'copy:deps',
-     'sass',
--    'date',
-   ]);
- };
diff --git a/srcpkgs/Signal-Desktop/patches/expire-from-source-date-epoch.patch b/srcpkgs/Signal-Desktop/patches/expire-from-source-date-epoch.patch
new file mode 100644
index 000000000000..d29c79835179
--- /dev/null
+++ b/srcpkgs/Signal-Desktop/patches/expire-from-source-date-epoch.patch
@@ -0,0 +1,13 @@
+--- a/Gruntfile.js
++++ b/Gruntfile.js
+@@ -203,9 +203,7 @@ module.exports = grunt => {
+   });
+ 
+   grunt.registerTask('getExpireTime', () => {
+-    grunt.task.requires('gitinfo');
+-    const gitinfo = grunt.config.get('gitinfo');
+-    const committed = gitinfo.local.branch.current.lastCommitTime;
++    const committed = parseInt(process.env.SOURCE_DATE_EPOCH, 10) * 1000;
+     const time = Date.parse(committed) + 1000 * 60 * 60 * 24 * 90;
+     grunt.file.write(
+       'config/local-production.json',
diff --git a/srcpkgs/Signal-Desktop/template b/srcpkgs/Signal-Desktop/template
index 8999b6e483ff..03b925a60e53 100644
--- a/srcpkgs/Signal-Desktop/template
+++ b/srcpkgs/Signal-Desktop/template
@@ -1,33 +1,46 @@
 # Template file for 'Signal-Desktop'
 pkgname=Signal-Desktop
-version=1.39.5
+version=5.6.2
 revision=1
-# Due to electron
+# Signal officially only supports x86_64 (also due to Electron)
+# discontinued Electron 32-bit support: https://www.electronjs.org/blog/linux-32bit-support
 # 32-bit is not supported https://github.com/signalapp/Signal-Desktop/issues/1661
+# armv7hf/arm64: https://github.com/signalapp/Signal-Desktop/issues/3410
 archs="x86_64"
-hostmakedepends="git nodejs python tar yarn"
-depends="libnotify libappindicator"
+hostmakedepends="git git-lfs nodejs python3 tar yarn"
+depends="cairo gtk+3 libvips pango"
 short_desc="Signal Private Messenger for Linux"
 maintainer="Julio Galvan <juliogalvan@protonmail.com>"
-license="GPL-3.0-only"
+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=f9f55c0195579b43712bf370b753337432eda4bcf89809d1c8f3deb0b4ee4400
+checksum=9bafd0673ce518379dd3f872beb3f3075bc29a979ea5328150c4c842ef910e87
 nostrip_files="signal-desktop"
 
-pre_build() {
-	vsed -e 's/"node": "/&>=/' -i package.json
+post_extract() {
+	# git-lfs hook needs to be installed for one of the dependencies
+	git lfs install
+
+	vsed 's/"node": "/&>=/' -i package.json
+
+	# Dependencies have to be installed before applying patch
+	yarn install --ignore-engines --frozen-lockfile
 }
 
 do_build() {
-	yarn install --ignore-engines
 	yarn grunt
-	yarn build-release --dir
+	yarn generate
+	yarn build-release
 }
 
 do_install() {
 	vmkdir usr/lib/signal-desktop
 
+	# Remove prebuilt binaries for foreign architectures (to not confuse strip)
+	for prebin in darwin-x64 linux-arm64 win32-ia32 win32-x64; do
+		rm -rf release/linux-unpacked/resources/app.asar.unpacked/node_modules/{ffi-napi,ref-napi}/prebuilds/$prebin
+	done
+
 	vcopy release/linux-unpacked/* usr/lib/signal-desktop
 
 	vmkdir usr/bin
@@ -40,4 +53,6 @@ do_install() {
 	for size in 16 32 48 128 256 1024; do
 		vinstall images/icon_${size}.png 644 usr/share/icons/hicolor/${size}x${size}/apps/ signal.png
 	done
+
+	vlicense LICENSE
 }

  parent reply	other threads:[~2021-06-28  7:50 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-16  7:57 [PR PATCH] " reedts
2021-06-22  8:43 ` [PR PATCH] [Updated] " reedts
2021-06-24 22:24 ` ScrelliCopter
2021-06-25  6:50 ` [PR PATCH] [Updated] " reedts
2021-06-25  6:51 ` reedts
2021-06-27 12:33 ` Veganizer
2021-06-28  1:59 ` ericonr
2021-06-28  7:50 ` reedts [this message]
2021-06-28  7:50 ` [PR PATCH] [Updated] " reedts
2021-06-28  7:51 ` reedts
2021-06-28  8:14 ` ScrelliCopter
2021-06-29 22:15 ` [PR PATCH] [Merged]: Signal-Desktop: update to 5.6.2 Piraty

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210628075009.jHyqaTJXPuHcyda49SY7B6D92ZZU6CuY-RMYhjDm7jA@z \
    --to=reedts@users.noreply.github.com \
    --cc=ml@inbox.vuxu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).