Github messages for voidlinux
 help / color / mirror / Atom feed
* [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).