* [PR PATCH] Signal-Desktop: update to 5.0.0
@ 2021-04-26 3:21 ScrelliCopter
2021-04-26 19:20 ` [PR REVIEW] " ericonr
` (8 more replies)
0 siblings, 9 replies; 19+ messages in thread
From: ScrelliCopter @ 2021-04-26 3:21 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1604 bytes --]
There is a new pull request by ScrelliCopter against master on the void-packages repository
https://github.com/ScrelliCopter/void-packages signal
https://github.com/void-linux/void-packages/pull/30505
Signal-Desktop: update to 5.0.0
<!-- 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, (ARCH-LIBC)
- [ ] 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/30505.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-signal-30505.patch --]
[-- Type: text/x-diff, Size: 2177 bytes --]
From 58df6e1d3055583e3af24efd677a3f55af4c1a7f Mon Sep 17 00:00:00 2001
From: a dinosaur <nick@a-dinosaur.com>
Date: Mon, 26 Apr 2021 13:03:23 +1000
Subject: [PATCH] Signal-Desktop: update to 5.0.0
---
.../Signal-Desktop/patches/build-from-tarball.patch | 9 ---------
srcpkgs/Signal-Desktop/template | 12 +++++++++---
2 files changed, 9 insertions(+), 12 deletions(-)
delete mode 100644 srcpkgs/Signal-Desktop/patches/build-from-tarball.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 e8c3db0e087e..000000000000
--- a/srcpkgs/Signal-Desktop/patches/build-from-tarball.patch
+++ /dev/null
@@ -1,9 +0,0 @@
---- Gruntfile.js.orig 2020-04-26 23:32:09.861789510 +0200
-+++ 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/template b/srcpkgs/Signal-Desktop/template
index 8999b6e483ff..e5784104c9f2 100644
--- a/srcpkgs/Signal-Desktop/template
+++ b/srcpkgs/Signal-Desktop/template
@@ -1,22 +1,28 @@
# Template file for 'Signal-Desktop'
pkgname=Signal-Desktop
-version=1.39.5
+version=5.0.0
revision=1
# Due to electron
# 32-bit is not supported https://github.com/signalapp/Signal-Desktop/issues/1661
archs="x86_64"
-hostmakedepends="git nodejs python tar yarn"
+hostmakedepends="git git-lfs nodejs python tar yarn"
depends="libnotify libappindicator"
short_desc="Signal Private Messenger for Linux"
maintainer="Julio Galvan <juliogalvan@protonmail.com>"
license="GPL-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=4ff58f5ab5f4de6477dd92069d8c2e9a830513acca527b51e0ee1c7cbe7d8627
nostrip_files="signal-desktop"
pre_build() {
+ git lfs install
+
+ # allow higher Node version
vsed -e 's/"node": "/&>=/' -i package.json
+
+ # allow tarball build
+ vsed "/'date',\$/d" -i Gruntfile.js
}
do_build() {
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PR REVIEW] Signal-Desktop: update to 5.0.0
2021-04-26 3:21 [PR PATCH] Signal-Desktop: update to 5.0.0 ScrelliCopter
@ 2021-04-26 19:20 ` ericonr
2021-04-26 23:56 ` ScrelliCopter
` (7 subsequent siblings)
8 siblings, 0 replies; 19+ messages in thread
From: ericonr @ 2021-04-26 19:20 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 220 bytes --]
New review comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/30505#discussion_r620581843
Comment:
I don't know if this is actually ok to do here. What files does it touch?
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PR REVIEW] Signal-Desktop: update to 5.0.0
2021-04-26 3:21 [PR PATCH] Signal-Desktop: update to 5.0.0 ScrelliCopter
2021-04-26 19:20 ` [PR REVIEW] " ericonr
@ 2021-04-26 23:56 ` ScrelliCopter
2021-04-27 2:23 ` ericonr
` (6 subsequent siblings)
8 siblings, 0 replies; 19+ messages in thread
From: ScrelliCopter @ 2021-04-26 23:56 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 256 bytes --]
New review comment by ScrelliCopter on void-packages repository
https://github.com/void-linux/void-packages/pull/30505#discussion_r620730210
Comment:
@ericonr I don't know, but it's what I needed to get any version past what's in the repos to build 🤷
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PR REVIEW] Signal-Desktop: update to 5.0.0
2021-04-26 3:21 [PR PATCH] Signal-Desktop: update to 5.0.0 ScrelliCopter
2021-04-26 19:20 ` [PR REVIEW] " ericonr
2021-04-26 23:56 ` ScrelliCopter
@ 2021-04-27 2:23 ` ericonr
2021-04-27 2:23 ` ericonr
` (5 subsequent siblings)
8 siblings, 0 replies; 19+ messages in thread
From: ericonr @ 2021-04-27 2:23 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 777 bytes --]
New review comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/30505#discussion_r620800327
Comment:
Man page says
```
DESCRIPTION
Perform the following actions to ensure that Git LFS is setup properly:
○
Set up the clean and smudge filters under the name "lfs" in the global Git config.
○
Install a pre-push hook to run git-lfs-pre-push(1) for the current repository, if run from inside one. If "core.hooksPath" is configured in any Git configuration (and supported, i.e., the installed Git version is at least 2.9.0), then the pre-push hook will be installed to that directory instead.
```
@void-linux/pkg-committers is this ok or should we change the position of the git config directory or something?
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PR REVIEW] Signal-Desktop: update to 5.0.0
2021-04-26 3:21 [PR PATCH] Signal-Desktop: update to 5.0.0 ScrelliCopter
` (2 preceding siblings ...)
2021-04-27 2:23 ` ericonr
@ 2021-04-27 2:23 ` ericonr
2021-04-27 2:41 ` q66
` (4 subsequent siblings)
8 siblings, 0 replies; 19+ messages in thread
From: ericonr @ 2021-04-27 2:23 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 214 bytes --]
New review comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/30505#discussion_r620800702
Comment:
Anyway, IMO this specific command should be done in `pre_configure`
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PR REVIEW] Signal-Desktop: update to 5.0.0
2021-04-26 3:21 [PR PATCH] Signal-Desktop: update to 5.0.0 ScrelliCopter
` (3 preceding siblings ...)
2021-04-27 2:23 ` ericonr
@ 2021-04-27 2:41 ` q66
2021-04-27 5:28 ` [PR PATCH] [Updated] " ScrelliCopter
` (3 subsequent siblings)
8 siblings, 0 replies; 19+ messages in thread
From: q66 @ 2021-04-27 2:41 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 197 bytes --]
New review comment by q66 on void-packages repository
https://github.com/void-linux/void-packages/pull/30505#discussion_r620806861
Comment:
if it touches anything outside builddir it's not okay
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PR PATCH] [Updated] Signal-Desktop: update to 5.0.0
2021-04-26 3:21 [PR PATCH] Signal-Desktop: update to 5.0.0 ScrelliCopter
` (4 preceding siblings ...)
2021-04-27 2:41 ` q66
@ 2021-04-27 5:28 ` ScrelliCopter
2021-05-02 23:13 ` kwshi
` (2 subsequent siblings)
8 siblings, 0 replies; 19+ messages in thread
From: ScrelliCopter @ 2021-04-27 5:28 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1609 bytes --]
There is an updated pull request by ScrelliCopter against master on the void-packages repository
https://github.com/ScrelliCopter/void-packages signal
https://github.com/void-linux/void-packages/pull/30505
Signal-Desktop: update to 5.0.0
<!-- 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, (ARCH-LIBC)
- [ ] 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/30505.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-signal-30505.patch --]
[-- Type: text/x-diff, Size: 2198 bytes --]
From 2354c5978d042880f4cad52f6685fa76cecc7332 Mon Sep 17 00:00:00 2001
From: a dinosaur <nick@a-dinosaur.com>
Date: Mon, 26 Apr 2021 13:03:23 +1000
Subject: [PATCH] Signal-Desktop: update to 5.0.0
---
.../patches/build-from-tarball.patch | 9 ---------
srcpkgs/Signal-Desktop/template | 14 +++++++++++---
2 files changed, 11 insertions(+), 12 deletions(-)
delete mode 100644 srcpkgs/Signal-Desktop/patches/build-from-tarball.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 e8c3db0e087e..000000000000
--- a/srcpkgs/Signal-Desktop/patches/build-from-tarball.patch
+++ /dev/null
@@ -1,9 +0,0 @@
---- Gruntfile.js.orig 2020-04-26 23:32:09.861789510 +0200
-+++ 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/template b/srcpkgs/Signal-Desktop/template
index 8999b6e483ff..bf6ee4b654a6 100644
--- a/srcpkgs/Signal-Desktop/template
+++ b/srcpkgs/Signal-Desktop/template
@@ -1,22 +1,30 @@
# Template file for 'Signal-Desktop'
pkgname=Signal-Desktop
-version=1.39.5
+version=5.0.0
revision=1
# Due to electron
# 32-bit is not supported https://github.com/signalapp/Signal-Desktop/issues/1661
archs="x86_64"
-hostmakedepends="git nodejs python tar yarn"
+hostmakedepends="git git-lfs nodejs python tar yarn"
depends="libnotify libappindicator"
short_desc="Signal Private Messenger for Linux"
maintainer="Julio Galvan <juliogalvan@protonmail.com>"
license="GPL-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=4ff58f5ab5f4de6477dd92069d8c2e9a830513acca527b51e0ee1c7cbe7d8627
nostrip_files="signal-desktop"
+pre_configure() {
+ git lfs install
+}
+
pre_build() {
+ # allow higher Node version
vsed -e 's/"node": "/&>=/' -i package.json
+
+ # allow tarball build
+ vsed "/'date',\$/d" -i Gruntfile.js
}
do_build() {
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: Signal-Desktop: update to 5.0.0
2021-04-26 3:21 [PR PATCH] Signal-Desktop: update to 5.0.0 ScrelliCopter
` (5 preceding siblings ...)
2021-04-27 5:28 ` [PR PATCH] [Updated] " ScrelliCopter
@ 2021-05-02 23:13 ` kwshi
2021-06-24 22:34 ` ScrelliCopter
2021-06-24 22:34 ` [PR PATCH] [Closed]: " ScrelliCopter
8 siblings, 0 replies; 19+ messages in thread
From: kwshi @ 2021-05-02 23:13 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 249 bytes --]
New comment by kwshi on void-packages repository
https://github.com/void-linux/void-packages/pull/30505#issuecomment-830923699
Comment:
FYI, this does not build successfully on my machine right now w/ the following error log:
http://ix.io/3lEb
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: Signal-Desktop: update to 5.0.0
2021-04-26 3:21 [PR PATCH] Signal-Desktop: update to 5.0.0 ScrelliCopter
` (6 preceding siblings ...)
2021-05-02 23:13 ` kwshi
@ 2021-06-24 22:34 ` ScrelliCopter
2021-06-24 22:34 ` [PR PATCH] [Closed]: " ScrelliCopter
8 siblings, 0 replies; 19+ messages in thread
From: ScrelliCopter @ 2021-06-24 22:34 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 175 bytes --]
New comment by ScrelliCopter on void-packages repository
https://github.com/void-linux/void-packages/pull/30505#issuecomment-867998040
Comment:
Closing in favour of #31521.
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PR PATCH] [Closed]: Signal-Desktop: update to 5.0.0
2021-04-26 3:21 [PR PATCH] Signal-Desktop: update to 5.0.0 ScrelliCopter
` (7 preceding siblings ...)
2021-06-24 22:34 ` ScrelliCopter
@ 2021-06-24 22:34 ` ScrelliCopter
8 siblings, 0 replies; 19+ messages in thread
From: ScrelliCopter @ 2021-06-24 22:34 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1442 bytes --]
There's a closed pull request on the void-packages repository
Signal-Desktop: update to 5.0.0
https://github.com/void-linux/void-packages/pull/30505
Description:
<!-- 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, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
- [ ] aarch64-musl
- [ ] armv7l
- [ ] armv6l-musl
-->
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: Signal-Desktop: update to 5.0.0.
2021-05-05 17:52 [PR PATCH] " lane-brain
` (7 preceding siblings ...)
2021-08-01 13:03 ` notthewave
@ 2021-08-27 17:33 ` ericonr
8 siblings, 0 replies; 19+ messages in thread
From: ericonr @ 2021-08-27 17:33 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 205 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/30696#issuecomment-907362146
Comment:
Merged in https://github.com/void-linux/void-packages/pull/31521
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: Signal-Desktop: update to 5.0.0.
2021-05-05 17:52 [PR PATCH] " lane-brain
` (6 preceding siblings ...)
2021-05-05 20:14 ` lane-brain
@ 2021-08-01 13:03 ` notthewave
2021-08-27 17:33 ` ericonr
8 siblings, 0 replies; 19+ messages in thread
From: notthewave @ 2021-08-01 13:03 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 162 bytes --]
New comment by notthewave on void-packages repository
https://github.com/void-linux/void-packages/pull/30696#issuecomment-890516881
Comment:
this can be closed
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: Signal-Desktop: update to 5.0.0.
2021-05-05 17:52 [PR PATCH] " lane-brain
` (5 preceding siblings ...)
2021-05-05 19:58 ` lane-brain
@ 2021-05-05 20:14 ` lane-brain
2021-08-01 13:03 ` notthewave
2021-08-27 17:33 ` ericonr
8 siblings, 0 replies; 19+ messages in thread
From: lane-brain @ 2021-05-05 20:14 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 3854 bytes --]
New comment by lane-brain on void-packages repository
https://github.com/void-linux/void-packages/pull/30696#issuecomment-832976856
Comment:
> `chmod -R +wX $HOME` works, but that's starting to sound like they build Go stuff in the middle of the build process??? Where do they even get the Go executable?
No idea here. But unfortunately no this mktemp method runs into another issue:
```
=> Signal-Desktop-5.0.0_1: skipping check (XBPS_CHECK_PKGS is disabled) ...
=> Signal-Desktop-5.0.0_1: running pre-install hook: 00-libdir ...
=> Signal-Desktop-5.0.0_1: running pre-install hook: 02-script-wrapper ...
=> Signal-Desktop-5.0.0_1: running pre-install hook: 98-fixup-gir-path ...
=> Signal-Desktop-5.0.0_1: running do_install ...
=> Signal-Desktop-5.0.0_1: running post_install ...
=> Signal-Desktop-5.0.0_1: running post-install hook: 00-compress-info-files ...
=> Signal-Desktop-5.0.0_1: running post-install hook: 00-fixup-gir-path ...
=> Signal-Desktop-5.0.0_1: running post-install hook: 00-libdir ...
=> Signal-Desktop-5.0.0_1: running post-install hook: 00-uncompress-manpages ...
=> Signal-Desktop-5.0.0_1: running post-install hook: 01-remove-misc ...
=> Signal-Desktop-5.0.0_1: running post-install hook: 02-remove-libtool-archives ...
=> Signal-Desktop-5.0.0_1: running post-install hook: 02-remove-perl-files ...
=> Signal-Desktop-5.0.0_1: running post-install hook: 02-remove-python-bytecode-files ...
=> Signal-Desktop-5.0.0_1: running post-install hook: 03-remove-empty-dirs ...
=> Signal-Desktop-5.0.0_1: running post-install hook: 04-create-xbps-metadata-scripts ...
mktemp: failed to create file via template ‘/tmp/tmp.XXXXXXXXXX’: No such file or directory
/void-packages/common/hooks/post-install/04-create-xbps-metadata-scripts.sh: line 27: $tmpf: ambiguous redirect
/void-packages/common/hooks/post-install/04-create-xbps-metadata-scripts.sh: line 203: $tmpf: ambiguous redirect
/void-packages/common/hooks/post-install/04-create-xbps-metadata-scripts.sh: line 307: $tmpf: ambiguous redirect
/void-packages/common/hooks/post-install/04-create-xbps-metadata-scripts.sh: line 314: $tmpf: ambiguous redirect
/void-packages/common/hooks/post-install/04-create-xbps-metadata-scripts.sh: line 315: $tmpf: ambiguous redirect
Added trigger 'gtk-icon-cache' for the 'INSTALL' script.
Added trigger 'update-desktopdb' for the 'INSTALL' script.
/void-packages/common/hooks/post-install/04-create-xbps-metadata-scripts.sh: line 333: $tmpf: ambiguous redirect
/void-packages/common/hooks/post-install/04-create-xbps-metadata-scripts.sh: line 334: $tmpf: ambiguous redirect
/void-packages/common/hooks/post-install/04-create-xbps-metadata-scripts.sh: line 341: $tmpf: ambiguous redirect
/void-packages/common/hooks/post-install/04-create-xbps-metadata-scripts.sh: line 342: $tmpf: ambiguous redirect
/void-packages/common/hooks/post-install/04-create-xbps-metadata-scripts.sh: line 341: $tmpf: ambiguous redirect
/void-packages/common/hooks/post-install/04-create-xbps-metadata-scripts.sh: line 342: $tmpf: ambiguous redirect
/void-packages/common/hooks/post-install/04-create-xbps-metadata-scripts.sh: line 345: $tmpf: ambiguous redirect
/void-packages/common/hooks/post-install/04-create-xbps-metadata-scripts.sh: line 346: $tmpf: ambiguous redirect
/void-packages/common/hooks/post-install/04-create-xbps-metadata-scripts.sh: line 347: $tmpf: ambiguous redirect
/void-packages/common/hooks/post-install/04-create-xbps-metadata-scripts.sh: line 361: $tmpf: ambiguous redirect
/void-packages/common/hooks/post-install/04-create-xbps-metadata-scripts.sh: line 362: $tmpf: ambiguous redirect
mv: missing destination file operand after '/destdir//Signal-Desktop-5.0.0/INSTALL'
Try 'mv --help' for more information.
=> ERROR: Signal-Desktop-5.0.0_1: failed to write INSTALL metadata file!
```
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: Signal-Desktop: update to 5.0.0.
2021-05-05 17:52 [PR PATCH] " lane-brain
` (4 preceding siblings ...)
2021-05-05 19:14 ` ericonr
@ 2021-05-05 19:58 ` lane-brain
2021-05-05 20:14 ` lane-brain
` (2 subsequent siblings)
8 siblings, 0 replies; 19+ messages in thread
From: lane-brain @ 2021-05-05 19:58 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 5649 bytes --]
New comment by lane-brain on void-packages repository
https://github.com/void-linux/void-packages/pull/30696#issuecomment-832968054
Comment:
> There's also #30505
>
> We need to figure out how to work with `git-lfs` here.
by the way, here is the build error if we don't install git-lfs
```
gyp info spawn args [ 'BUILDTYPE=Release', '-C', 'build' ]
make: Entering directory '/builddir/Signal-Desktop-5.0.0/node_modules/@journeyapps/sqlcipher/build'
CC(target) Release/obj.target/nothing/../../node-addon-api/nothing.o
AR(target) Release/obj.target/../../node-addon-api/nothing.a
COPY Release/nothing.a
ACTION deps_sqlite3_gyp_action_before_build_target_unpack_sqlite_dep Release/obj/gen/sqlcipher-amalgamation-3033000/sqlite3.c
Traceback (most recent call last):
File "/usr/lib/python3.9/tarfile.py", line 1682, in gzopen
t = cls.taropen(name, mode, fileobj, **kwargs)
File "/usr/lib/python3.9/tarfile.py", line 1659, in taropen
return cls(name, mode, fileobj, **kwargs)
File "/usr/lib/python3.9/tarfile.py", line 1522, in __init__
self.firstmember = self.next()
File "/usr/lib/python3.9/tarfile.py", line 2327, in next
tarinfo = self.tarinfo.fromtarfile(self)
File "/usr/lib/python3.9/tarfile.py", line 1112, in fromtarfile
buf = tarfile.fileobj.read(BLOCKSIZE)
File "/usr/lib/python3.9/gzip.py", line 300, in read
return self._buffer.read(size)
File "/usr/lib/python3.9/_compression.py", line 68, in readinto
data = self.read(len(byte_view))
File "/usr/lib/python3.9/gzip.py", line 487, in read
if not self._read_gzip_header():
File "/usr/lib/python3.9/gzip.py", line 435, in _read_gzip_header
raise BadGzipFile('Not a gzipped file (%r)' % magic)
gzip.BadGzipFile: Not a gzipped file (b've')
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/builddir/Signal-Desktop-5.0.0/node_modules/@journeyapps/sqlcipher/deps/./extract.py", line 7, in <module>
tfile = tarfile.open(tarball,'r:gz');
File "/usr/lib/python3.9/tarfile.py", line 1629, in open
return func(name, filemode, fileobj, **kwargs)
File "/usr/lib/python3.9/tarfile.py", line 1686, in gzopen
raise ReadError("not a gzip file")
tarfile.ReadError: not a gzip file
make: *** [deps/action_before_build.target.mk:13: Release/obj/gen/sqlcipher-amalgamation-3033000/sqlite3.c] Error 1
make: Leaving directory '/builddir/Signal-Desktop-5.0.0/node_modules/@journeyapps/sqlcipher/build'
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack at ChildProcess.onExit (/builddir/Signal-Desktop-5.0.0/node_modules/node-gyp/lib/build.js:194:23)
gyp ERR! stack at ChildProcess.emit (events.js:315:20)
gyp ERR! stack at Process.ChildProcess._handle.onexit (internal/child_process.js:277:12)
gyp ERR! System Linux 5.11.18_1
gyp ERR! command "/usr/bin/node" "/builddir/Signal-Desktop-5.0.0/node_modules/node-gyp/bin/node-gyp.js" "build" "--build-from-source" "--module=/builddir/Signal-Desktop-5.0.0/node_modules/@journeyapps/sqlcipher/lib/binding/napi-v6-linux-x64/node_sqlite3.node" "--module_name=node_sqlite3" "--module_path=/builddir/Signal-Desktop-5.0.0/node_modules/@journeyapps/sqlcipher/lib/binding/napi-v6-linux-x64" "--napi_version=7" "--node_abi_napi=napi" "--napi_build_version=6" "--node_napi_label=napi-v6"
gyp ERR! cwd /builddir/Signal-Desktop-5.0.0/node_modules/@journeyapps/sqlcipher
gyp ERR! node -v v14.16.0
gyp ERR! node-gyp -v v6.1.0
gyp ERR! not ok
node-pre-gyp ERR! build error
node-pre-gyp ERR! stack Error: Failed to execute '/usr/bin/node /builddir/Signal-Desktop-5.0.0/node_modules/node-gyp/bin/node-gyp.js build --build-from-source --module=/builddir/Signal-Desktop-5.0.0/node_modules/@journeyapps/sqlcipher/lib/binding/napi-v6-linux-x64/node_sqlite3.node --module_name=node_sqlite3 --module_path=/builddir/Signal-Desktop-5.0.0/node_modules/@journeyapps/sqlcipher/lib/binding/napi-v6-linux-x64 --napi_version=7 --node_abi_napi=napi --napi_build_version=6 --node_napi_label=napi-v6' (1)
node-pre-gyp ERR! stack at ChildProcess.<anonymous> (/builddir/Signal-Desktop-5.0.0/node_modules/node-pre-gyp/lib/util/compile.js:83:29)
node-pre-gyp ERR! stack at ChildProcess.emit (events.js:315:20)
node-pre-gyp ERR! stack at maybeClose (internal/child_process.js:1048:16)
node-pre-gyp ERR! stack at Process.ChildProcess._handle.onexit (internal/child_process.js:288:5)
node-pre-gyp ERR! System Linux 5.11.18_1
node-pre-gyp ERR! command "/usr/bin/node" "/builddir/Signal-Desktop-5.0.0/node_modules/@journeyapps/sqlcipher/node_modules/.bin/node-pre-gyp" "install" "--build-from-source"
node-pre-gyp ERR! cwd /builddir/Signal-Desktop-5.0.0/node_modules/@journeyapps/sqlcipher
node-pre-gyp ERR! node -v v14.16.0
node-pre-gyp ERR! node-pre-gyp -v v0.15.0
node-pre-gyp ERR! not ok
Failed to execute '/usr/bin/node /builddir/Signal-Desktop-5.0.0/node_modules/node-gyp/bin/node-gyp.js build --build-from-source --module=/builddir/Signal-Desktop-5.0.0/node_modules/@journeyapps/sqlcipher/lib/binding/napi-v6-linux-x64/node_sqlite3.node --module_name=node_sqlite3 --module_path=/builddir/Signal-Desktop-5.0.0/node_modules/@journeyapps/sqlcipher/lib/binding/napi-v6-linux-x64 --napi_version=7 --node_abi_napi=napi --napi_build_version=6 --node_napi_label=napi-v6' (1)
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.
=> ERROR: Signal-Desktop-5.0.0_1: pre_build: 'yarn install --ignore-engines' exited with 1
=> ERROR: in pre_build() at srcpkgs/Signal-Desktop/template:31
```
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: Signal-Desktop: update to 5.0.0.
2021-05-05 17:52 [PR PATCH] " lane-brain
` (3 preceding siblings ...)
2021-05-05 18:45 ` lane-brain
@ 2021-05-05 19:14 ` ericonr
2021-05-05 19:58 ` lane-brain
` (3 subsequent siblings)
8 siblings, 0 replies; 19+ messages in thread
From: ericonr @ 2021-05-05 19:14 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 302 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/30696#issuecomment-832942851
Comment:
`chmod -R +wX $HOME` works, but that's starting to sound like they build Go stuff in the middle of the build process??? Where do they even get the Go executable?
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: Signal-Desktop: update to 5.0.0.
2021-05-05 17:52 [PR PATCH] " lane-brain
` (2 preceding siblings ...)
2021-05-05 18:03 ` ericonr
@ 2021-05-05 18:45 ` lane-brain
2021-05-05 19:14 ` ericonr
` (4 subsequent siblings)
8 siblings, 0 replies; 19+ messages in thread
From: lane-brain @ 2021-05-05 18:45 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1158 bytes --]
New comment by lane-brain on void-packages repository
https://github.com/void-linux/void-packages/pull/30696#issuecomment-832924054
Comment:
The openssl linking and epoch source date patch yes. The former seemed justified because now we use openssl, the other one seemed necessitated by using tarballs instead of git. As far as I know about git-lfs, the documentation from Signal's github indicate that it's necessary for one of the yarn dependencies.
As an aside, I fixed the linting error (vlicense).
And now the build has concluded but I ran into an error at post_install:
```
...
rm: cannot remove '/tmp/go/pkg/mod/golang.org/x/exp@v0.0.0-20200513190911-00229845015e/utf8string/string_test.go': Permission denied
rm: cannot remove '/tmp/go/pkg/mod/golang.org/x/exp@v0.0.0-20200513190911-00229845015e/winfsnotify/winfsnotify.go': Permission denied
rm: cannot remove '/tmp/go/pkg/mod/golang.org/x/exp@v0.0.0-20200513190911-00229845015e/winfsnotify/winfsnotify_test.go': Permission denied
=> ERROR: Signal-Desktop-5.0.0_1: post_install: 'rm -rf $HOME' exited with 1
=> ERROR: in post_install() at srcpkgs/Signal-Desktop/template:65
```
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: Signal-Desktop: update to 5.0.0.
2021-05-05 17:52 [PR PATCH] " lane-brain
2021-05-05 17:54 ` lane-brain
2021-05-05 17:55 ` ericonr
@ 2021-05-05 18:03 ` ericonr
2021-05-05 18:45 ` lane-brain
` (5 subsequent siblings)
8 siblings, 0 replies; 19+ messages in thread
From: ericonr @ 2021-05-05 18:03 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 470 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/30696#issuecomment-832896410
Comment:
Are these patches imported from arch's packaging? https://github.com/archlinux/svntogit-community/tree/packages/signal-desktop/trunk
I think changing git config would be bad, but could you do `export HOME=$(mktemp -d)` in `pre_configure` and `rm -rf $HOME` in `post_install` ? I think it would work around the current hang up.
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: Signal-Desktop: update to 5.0.0.
2021-05-05 17:52 [PR PATCH] " lane-brain
2021-05-05 17:54 ` lane-brain
@ 2021-05-05 17:55 ` ericonr
2021-05-05 18:03 ` ericonr
` (6 subsequent siblings)
8 siblings, 0 replies; 19+ messages in thread
From: ericonr @ 2021-05-05 17:55 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 219 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/30696#issuecomment-832891205
Comment:
There's also #30505
We need to figure out how to work with `git-lfs` here.
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: Signal-Desktop: update to 5.0.0.
2021-05-05 17:52 [PR PATCH] " lane-brain
@ 2021-05-05 17:54 ` lane-brain
2021-05-05 17:55 ` ericonr
` (7 subsequent siblings)
8 siblings, 0 replies; 19+ messages in thread
From: lane-brain @ 2021-05-05 17:54 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 231 bytes --]
New comment by lane-brain on void-packages repository
https://github.com/void-linux/void-packages/pull/30696#issuecomment-832890547
Comment:
Checked in with the maintainer @julio641742, they gave the go ahead to submit update PR
^ permalink raw reply [flat|nested] 19+ messages in thread
end of thread, other threads:[~2021-08-27 17:33 UTC | newest]
Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-26 3:21 [PR PATCH] Signal-Desktop: update to 5.0.0 ScrelliCopter
2021-04-26 19:20 ` [PR REVIEW] " ericonr
2021-04-26 23:56 ` ScrelliCopter
2021-04-27 2:23 ` ericonr
2021-04-27 2:23 ` ericonr
2021-04-27 2:41 ` q66
2021-04-27 5:28 ` [PR PATCH] [Updated] " ScrelliCopter
2021-05-02 23:13 ` kwshi
2021-06-24 22:34 ` ScrelliCopter
2021-06-24 22:34 ` [PR PATCH] [Closed]: " ScrelliCopter
2021-05-05 17:52 [PR PATCH] " lane-brain
2021-05-05 17:54 ` lane-brain
2021-05-05 17:55 ` ericonr
2021-05-05 18:03 ` ericonr
2021-05-05 18:45 ` lane-brain
2021-05-05 19:14 ` ericonr
2021-05-05 19:58 ` lane-brain
2021-05-05 20:14 ` lane-brain
2021-08-01 13:03 ` notthewave
2021-08-27 17:33 ` ericonr
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).