Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] Signal-Desktop: add support for x86_64
@ 2022-09-01 16:44 akierig
  2022-09-01 16:44 ` [PR PATCH] [Updated] Signal-Desktop: add support for x86_64-musl akierig
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: akierig @ 2022-09-01 16:44 UTC (permalink / raw)
  To: ml

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

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

https://github.com/akierig/void-packages signal-desktop-musl
https://github.com/void-linux/void-packages/pull/39038

Signal-Desktop: add support for x86_64
#### 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
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - x86_64-musl (crossbuild)


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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-signal-desktop-musl-39038.patch --]
[-- Type: text/x-diff, Size: 942 bytes --]

From 579f3811934a4eb9cd38736347cc6586a48e4b87 Mon Sep 17 00:00:00 2001
From: akierig <anelki@fastmail.de>
Date: Thu, 1 Sep 2022 11:40:35 -0500
Subject: [PATCH] Signal-Desktop now builds on x86_musl

---
 srcpkgs/Signal-Desktop/template | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/srcpkgs/Signal-Desktop/template b/srcpkgs/Signal-Desktop/template
index a69f86dd28f9..7e58cc2d3773 100644
--- a/srcpkgs/Signal-Desktop/template
+++ b/srcpkgs/Signal-Desktop/template
@@ -5,7 +5,7 @@ revision=1
 # Signal officially only supports x86_64 (also due to Electron)
 # x86_64-musl fails because of its dependency on 'node-gyp' which depends on a glibc specific extension
 # armv7hf/arm64: https://github.com/signalapp/Signal-Desktop/issues/3410
-archs="x86_64"
+archs="x86_64 x86_64-musl"
 hostmakedepends="git git-lfs nodejs python3 tar yarn"
 depends="cairo gtk+3 libvips pango"
 short_desc="Signal Private Messenger for Linux"

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

* Re: [PR PATCH] [Updated] Signal-Desktop: add support for x86_64-musl
  2022-09-01 16:44 [PR PATCH] Signal-Desktop: add support for x86_64 akierig
@ 2022-09-01 16:44 ` akierig
  2022-09-01 17:00 ` [PR PATCH] [Closed]: " akierig
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: akierig @ 2022-09-01 16:44 UTC (permalink / raw)
  To: ml

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

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

https://github.com/akierig/void-packages signal-desktop-musl
https://github.com/void-linux/void-packages/pull/39038

Signal-Desktop: add support for x86_64-musl
#### 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
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - x86_64-musl (crossbuild)


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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-signal-desktop-musl-39038.patch --]
[-- Type: text/x-diff, Size: 948 bytes --]

From 08090c7b981710b4958ca0b6ed449fea72c6c91c Mon Sep 17 00:00:00 2001
From: akierig <anelki@fastmail.de>
Date: Thu, 1 Sep 2022 11:40:35 -0500
Subject: [PATCH] Signal-Desktop: add support for x86_64-musl

---
 srcpkgs/Signal-Desktop/template | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/srcpkgs/Signal-Desktop/template b/srcpkgs/Signal-Desktop/template
index a69f86dd28f9..7e58cc2d3773 100644
--- a/srcpkgs/Signal-Desktop/template
+++ b/srcpkgs/Signal-Desktop/template
@@ -5,7 +5,7 @@ revision=1
 # Signal officially only supports x86_64 (also due to Electron)
 # x86_64-musl fails because of its dependency on 'node-gyp' which depends on a glibc specific extension
 # armv7hf/arm64: https://github.com/signalapp/Signal-Desktop/issues/3410
-archs="x86_64"
+archs="x86_64 x86_64-musl"
 hostmakedepends="git git-lfs nodejs python3 tar yarn"
 depends="cairo gtk+3 libvips pango"
 short_desc="Signal Private Messenger for Linux"

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

* Re: Signal-Desktop: add support for x86_64-musl
  2022-09-01 16:44 [PR PATCH] Signal-Desktop: add support for x86_64 akierig
  2022-09-01 16:44 ` [PR PATCH] [Updated] Signal-Desktop: add support for x86_64-musl akierig
  2022-09-01 17:00 ` [PR PATCH] [Closed]: " akierig
@ 2022-09-01 17:00 ` akierig
  2022-09-05 15:08 ` selfisekai
  2022-09-05 16:15 ` akierig
  4 siblings, 0 replies; 6+ messages in thread
From: akierig @ 2022-09-01 17:00 UTC (permalink / raw)
  To: ml

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

New comment by akierig on void-packages repository

https://github.com/void-linux/void-packages/pull/39038#issuecomment-1234545501

Comment:
wait, I built it improperly locally. Apologies.

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

* Re: [PR PATCH] [Closed]: Signal-Desktop: add support for x86_64-musl
  2022-09-01 16:44 [PR PATCH] Signal-Desktop: add support for x86_64 akierig
  2022-09-01 16:44 ` [PR PATCH] [Updated] Signal-Desktop: add support for x86_64-musl akierig
@ 2022-09-01 17:00 ` akierig
  2022-09-01 17:00 ` akierig
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: akierig @ 2022-09-01 17:00 UTC (permalink / raw)
  To: ml

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

There's a closed pull request on the void-packages repository

Signal-Desktop: add support for x86_64-musl
https://github.com/void-linux/void-packages/pull/39038

Description:
#### 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
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - x86_64-musl (crossbuild)


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

* Re: Signal-Desktop: add support for x86_64-musl
  2022-09-01 16:44 [PR PATCH] Signal-Desktop: add support for x86_64 akierig
                   ` (2 preceding siblings ...)
  2022-09-01 17:00 ` akierig
@ 2022-09-05 15:08 ` selfisekai
  2022-09-05 16:15 ` akierig
  4 siblings, 0 replies; 6+ messages in thread
From: selfisekai @ 2022-09-05 15:08 UTC (permalink / raw)
  To: ml

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

New comment by selfisekai on void-packages repository

https://github.com/void-linux/void-packages/pull/39038#issuecomment-1237183158

Comment:
hey, FYI I've done this on Alpine repositories, the musl problems are:
- building ringrtc manually, which in turn requires building a Signal's fork of webrtc,
- building libsignal from separately downloaded source, with a sed on binding.gyp to compile for musl.

https://gitlab.alpinelinux.org/alpine/aports/-/blob/master/testing/signal-desktop/APKBUILD

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

* Re: Signal-Desktop: add support for x86_64-musl
  2022-09-01 16:44 [PR PATCH] Signal-Desktop: add support for x86_64 akierig
                   ` (3 preceding siblings ...)
  2022-09-05 15:08 ` selfisekai
@ 2022-09-05 16:15 ` akierig
  4 siblings, 0 replies; 6+ messages in thread
From: akierig @ 2022-09-05 16:15 UTC (permalink / raw)
  To: ml

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

New comment by akierig on void-packages repository

https://github.com/void-linux/void-packages/pull/39038#issuecomment-1237267823

Comment:
cool, thank you! I've not got the capacity right now to pick this back up but a patch from you or anyone on Musl (or a PR re: #38609) would be most welcome.

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

end of thread, other threads:[~2022-09-05 16:15 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-01 16:44 [PR PATCH] Signal-Desktop: add support for x86_64 akierig
2022-09-01 16:44 ` [PR PATCH] [Updated] Signal-Desktop: add support for x86_64-musl akierig
2022-09-01 17:00 ` [PR PATCH] [Closed]: " akierig
2022-09-01 17:00 ` akierig
2022-09-05 15:08 ` selfisekai
2022-09-05 16:15 ` akierig

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).