* [PR PATCH] utox: update to 0.17.2.
@ 2020-04-13 0:51 avoidr
2020-04-13 21:34 ` [PR PATCH] [Updated] " avoidr
` (3 more replies)
0 siblings, 4 replies; 6+ messages in thread
From: avoidr @ 2020-04-13 0:51 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 312 bytes --]
There is a new pull request by avoidr against master on the void-packages repository
https://github.com/avoidr/void-packages utox
https://github.com/void-linux/void-packages/pull/20942
utox: update to 0.17.2.
[ci skip]
A patch file from https://github.com/void-linux/void-packages/pull/20942.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-utox-20942.patch --]
[-- Type: text/x-diff, Size: 1829 bytes --]
From 20a21dae0f4f456c2b930aec858ee786c1cb8727 Mon Sep 17 00:00:00 2001
From: avoidr <avoidr@firemail.cc>
Date: Mon, 13 Apr 2020 02:48:51 +0200
Subject: [PATCH] utox: update to 0.17.2.
[ci skip]
---
srcpkgs/utox/patches/ftbfs.patch | 11 -----------
srcpkgs/utox/template | 10 +++++-----
2 files changed, 5 insertions(+), 16 deletions(-)
delete mode 100644 srcpkgs/utox/patches/ftbfs.patch
diff --git a/srcpkgs/utox/patches/ftbfs.patch b/srcpkgs/utox/patches/ftbfs.patch
deleted file mode 100644
index b17a97f13d4..00000000000
--- a/srcpkgs/utox/patches/ftbfs.patch
+++ /dev/null
@@ -1,11 +0,0 @@
---- src/av/audio.h
-+++ src/av/audio.h
-@@ -5,7 +5,7 @@
- #include <stdint.h>
- #include <stdbool.h>
-
--typedef struct ALCdevice_struct ALCdevice;
-+typedef struct ALCdevice ALCdevice;
-
- extern bool utox_audio_thread_init;
-
diff --git a/srcpkgs/utox/template b/srcpkgs/utox/template
index d04cc5c5748..add510fd45c 100644
--- a/srcpkgs/utox/template
+++ b/srcpkgs/utox/template
@@ -1,6 +1,6 @@
# Template file for 'utox'
pkgname=utox
-version=0.17.1
+version=0.17.2
revision=1
wrksrc=uTox
build_style=cmake
@@ -10,12 +10,12 @@ makedepends="check-devel dbus-devel filter_audio-devel fontconfig-devel
libXrender-devel libopenal-devel libsodium-devel libvpx-devel opus-devel
toxcore-devel v4l-utils-devel"
short_desc="Lightweight Tox instant messenger client written in C"
-maintainer="Spencer Hill <spencernh77@gmail.com>"
+maintainer="avoidr <avoidr@firemail.cc>"
license="MIT"
-homepage="http://utox.io/"
+homepage="https://github.com/uTox/uTox/"
distfiles="https://github.com/uTox/uTox/releases/download/v${version}/uTox-${version}-full.tar.gz"
-checksum=6a6d399827f31593f840274b08ed80039c99d5abf0bd6634faf22cc064930460
-nocross=yes
+checksum=
+#nocross=yes
post_install() {
vlicense LICENSE
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PR PATCH] [Updated] utox: update to 0.17.2.
2020-04-13 0:51 [PR PATCH] utox: update to 0.17.2 avoidr
@ 2020-04-13 21:34 ` avoidr
2020-04-13 21:35 ` avoidr
` (2 subsequent siblings)
3 siblings, 0 replies; 6+ messages in thread
From: avoidr @ 2020-04-13 21:34 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 317 bytes --]
There is an updated pull request by avoidr against master on the void-packages repository
https://github.com/avoidr/void-packages utox
https://github.com/void-linux/void-packages/pull/20942
utox: update to 0.17.2.
[ci skip]
A patch file from https://github.com/void-linux/void-packages/pull/20942.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-utox-20942.patch --]
[-- Type: text/x-diff, Size: 1868 bytes --]
From 0660f014f47e6e4fa776e9a853d4cb6119367f95 Mon Sep 17 00:00:00 2001
From: avoidr <avoidr@firemail.cc>
Date: Mon, 13 Apr 2020 02:48:51 +0200
Subject: [PATCH] utox: update to 0.17.2.
---
srcpkgs/utox/patches/ftbfs.patch | 11 -----------
srcpkgs/utox/template | 9 ++++-----
2 files changed, 4 insertions(+), 16 deletions(-)
delete mode 100644 srcpkgs/utox/patches/ftbfs.patch
diff --git a/srcpkgs/utox/patches/ftbfs.patch b/srcpkgs/utox/patches/ftbfs.patch
deleted file mode 100644
index b17a97f13d4..00000000000
--- a/srcpkgs/utox/patches/ftbfs.patch
+++ /dev/null
@@ -1,11 +0,0 @@
---- src/av/audio.h
-+++ src/av/audio.h
-@@ -5,7 +5,7 @@
- #include <stdint.h>
- #include <stdbool.h>
-
--typedef struct ALCdevice_struct ALCdevice;
-+typedef struct ALCdevice ALCdevice;
-
- extern bool utox_audio_thread_init;
-
diff --git a/srcpkgs/utox/template b/srcpkgs/utox/template
index d04cc5c5748..e943349d961 100644
--- a/srcpkgs/utox/template
+++ b/srcpkgs/utox/template
@@ -1,6 +1,6 @@
# Template file for 'utox'
pkgname=utox
-version=0.17.1
+version=0.17.2
revision=1
wrksrc=uTox
build_style=cmake
@@ -10,12 +10,11 @@ makedepends="check-devel dbus-devel filter_audio-devel fontconfig-devel
libXrender-devel libopenal-devel libsodium-devel libvpx-devel opus-devel
toxcore-devel v4l-utils-devel"
short_desc="Lightweight Tox instant messenger client written in C"
-maintainer="Spencer Hill <spencernh77@gmail.com>"
+maintainer="avoidr <avoidr@firemail.cc>"
license="MIT"
-homepage="http://utox.io/"
+homepage="https://github.com/uTox/uTox/"
distfiles="https://github.com/uTox/uTox/releases/download/v${version}/uTox-${version}-full.tar.gz"
-checksum=6a6d399827f31593f840274b08ed80039c99d5abf0bd6634faf22cc064930460
-nocross=yes
+checksum=8437827b899e4e358adaf0641680c214f4b6dfb52af6534f261d747a905f08b6
post_install() {
vlicense LICENSE
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PR PATCH] [Updated] utox: update to 0.17.2.
2020-04-13 0:51 [PR PATCH] utox: update to 0.17.2 avoidr
2020-04-13 21:34 ` [PR PATCH] [Updated] " avoidr
@ 2020-04-13 21:35 ` avoidr
2020-04-13 22:12 ` avoidr
2020-04-14 14:47 ` [PR PATCH] [Merged]: " Hoshpak
3 siblings, 0 replies; 6+ messages in thread
From: avoidr @ 2020-04-13 21:35 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 317 bytes --]
There is an updated pull request by avoidr against master on the void-packages repository
https://github.com/avoidr/void-packages utox
https://github.com/void-linux/void-packages/pull/20942
utox: update to 0.17.2.
[ci skip]
A patch file from https://github.com/void-linux/void-packages/pull/20942.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-utox-20942.patch --]
[-- Type: text/x-diff, Size: 1868 bytes --]
From b050b8ab4d8eb1d063413f0cbadb81366e7559df Mon Sep 17 00:00:00 2001
From: avoidr <avoidr@firemail.cc>
Date: Mon, 13 Apr 2020 02:48:51 +0200
Subject: [PATCH] utox: update to 0.17.2.
---
srcpkgs/utox/patches/ftbfs.patch | 11 -----------
srcpkgs/utox/template | 9 ++++-----
2 files changed, 4 insertions(+), 16 deletions(-)
delete mode 100644 srcpkgs/utox/patches/ftbfs.patch
diff --git a/srcpkgs/utox/patches/ftbfs.patch b/srcpkgs/utox/patches/ftbfs.patch
deleted file mode 100644
index b17a97f13d4..00000000000
--- a/srcpkgs/utox/patches/ftbfs.patch
+++ /dev/null
@@ -1,11 +0,0 @@
---- src/av/audio.h
-+++ src/av/audio.h
-@@ -5,7 +5,7 @@
- #include <stdint.h>
- #include <stdbool.h>
-
--typedef struct ALCdevice_struct ALCdevice;
-+typedef struct ALCdevice ALCdevice;
-
- extern bool utox_audio_thread_init;
-
diff --git a/srcpkgs/utox/template b/srcpkgs/utox/template
index d04cc5c5748..e943349d961 100644
--- a/srcpkgs/utox/template
+++ b/srcpkgs/utox/template
@@ -1,6 +1,6 @@
# Template file for 'utox'
pkgname=utox
-version=0.17.1
+version=0.17.2
revision=1
wrksrc=uTox
build_style=cmake
@@ -10,12 +10,11 @@ makedepends="check-devel dbus-devel filter_audio-devel fontconfig-devel
libXrender-devel libopenal-devel libsodium-devel libvpx-devel opus-devel
toxcore-devel v4l-utils-devel"
short_desc="Lightweight Tox instant messenger client written in C"
-maintainer="Spencer Hill <spencernh77@gmail.com>"
+maintainer="avoidr <avoidr@firemail.cc>"
license="MIT"
-homepage="http://utox.io/"
+homepage="https://github.com/uTox/uTox/"
distfiles="https://github.com/uTox/uTox/releases/download/v${version}/uTox-${version}-full.tar.gz"
-checksum=6a6d399827f31593f840274b08ed80039c99d5abf0bd6634faf22cc064930460
-nocross=yes
+checksum=8437827b899e4e358adaf0641680c214f4b6dfb52af6534f261d747a905f08b6
post_install() {
vlicense LICENSE
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PR PATCH] [Updated] utox: update to 0.17.2.
2020-04-13 0:51 [PR PATCH] utox: update to 0.17.2 avoidr
2020-04-13 21:34 ` [PR PATCH] [Updated] " avoidr
2020-04-13 21:35 ` avoidr
@ 2020-04-13 22:12 ` avoidr
2020-04-14 14:47 ` [PR PATCH] [Merged]: " Hoshpak
3 siblings, 0 replies; 6+ messages in thread
From: avoidr @ 2020-04-13 22:12 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 308 bytes --]
There is an updated pull request by avoidr against master on the void-packages repository
https://github.com/avoidr/void-packages utox
https://github.com/void-linux/void-packages/pull/20942
utox: update to 0.17.2.
A patch file from https://github.com/void-linux/void-packages/pull/20942.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-utox-20942.patch --]
[-- Type: text/x-diff, Size: 1848 bytes --]
From ab6f72cf32457e93aee7a2b8dbd55b83cc422c41 Mon Sep 17 00:00:00 2001
From: avoidr <avoidr@firemail.cc>
Date: Mon, 13 Apr 2020 02:48:51 +0200
Subject: [PATCH] utox: update to 0.17.2.
---
srcpkgs/utox/patches/ftbfs.patch | 11 -----------
srcpkgs/utox/template | 8 ++++----
2 files changed, 4 insertions(+), 15 deletions(-)
delete mode 100644 srcpkgs/utox/patches/ftbfs.patch
diff --git a/srcpkgs/utox/patches/ftbfs.patch b/srcpkgs/utox/patches/ftbfs.patch
deleted file mode 100644
index b17a97f13d4..00000000000
--- a/srcpkgs/utox/patches/ftbfs.patch
+++ /dev/null
@@ -1,11 +0,0 @@
---- src/av/audio.h
-+++ src/av/audio.h
-@@ -5,7 +5,7 @@
- #include <stdint.h>
- #include <stdbool.h>
-
--typedef struct ALCdevice_struct ALCdevice;
-+typedef struct ALCdevice ALCdevice;
-
- extern bool utox_audio_thread_init;
-
diff --git a/srcpkgs/utox/template b/srcpkgs/utox/template
index d04cc5c5748..10f4f1e9d88 100644
--- a/srcpkgs/utox/template
+++ b/srcpkgs/utox/template
@@ -1,6 +1,6 @@
# Template file for 'utox'
pkgname=utox
-version=0.17.1
+version=0.17.2
revision=1
wrksrc=uTox
build_style=cmake
@@ -10,11 +10,11 @@ makedepends="check-devel dbus-devel filter_audio-devel fontconfig-devel
libXrender-devel libopenal-devel libsodium-devel libvpx-devel opus-devel
toxcore-devel v4l-utils-devel"
short_desc="Lightweight Tox instant messenger client written in C"
-maintainer="Spencer Hill <spencernh77@gmail.com>"
+maintainer="avoidr <avoidr@firemail.cc>"
license="MIT"
-homepage="http://utox.io/"
+homepage="https://github.com/uTox/uTox/"
distfiles="https://github.com/uTox/uTox/releases/download/v${version}/uTox-${version}-full.tar.gz"
-checksum=6a6d399827f31593f840274b08ed80039c99d5abf0bd6634faf22cc064930460
+checksum=8437827b899e4e358adaf0641680c214f4b6dfb52af6534f261d747a905f08b6
nocross=yes
post_install() {
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PR PATCH] [Merged]: utox: update to 0.17.2.
2020-04-13 0:51 [PR PATCH] utox: update to 0.17.2 avoidr
` (2 preceding siblings ...)
2020-04-13 22:12 ` avoidr
@ 2020-04-14 14:47 ` Hoshpak
3 siblings, 0 replies; 6+ messages in thread
From: Hoshpak @ 2020-04-14 14:47 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 157 bytes --]
There's a merged pull request on the void-packages repository
utox: update to 0.17.2.
https://github.com/void-linux/void-packages/pull/20942
Description:
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PR PATCH] [Updated] utox: update to 0.17.2.
2020-08-11 21:51 [PR PATCH] " avoidr
@ 2020-08-11 23:15 ` avoidr
0 siblings, 0 replies; 6+ messages in thread
From: avoidr @ 2020-08-11 23:15 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 308 bytes --]
There is an updated pull request by avoidr against master on the void-packages repository
https://github.com/avoidr/void-packages utox
https://github.com/void-linux/void-packages/pull/24232
utox: update to 0.17.2.
A patch file from https://github.com/void-linux/void-packages/pull/24232.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-utox-24232.patch --]
[-- Type: text/x-diff, Size: 6540 bytes --]
From 67ca22b91375b91ab885647a4a31fbb87f1d5aa2 Mon Sep 17 00:00:00 2001
From: avoidr <avoidr@posteo.de>
Date: Tue, 11 Aug 2020 23:46:48 +0200
Subject: [PATCH] utox: add fix for settings crash
---
...en-changing-toxcore-related-settings.patch | 149 ++++++++++++++++++
srcpkgs/utox/template | 3 +-
2 files changed, 151 insertions(+), 1 deletion(-)
create mode 100644 srcpkgs/utox/patches/0001-Fix-crash-when-changing-toxcore-related-settings.patch
diff --git a/srcpkgs/utox/patches/0001-Fix-crash-when-changing-toxcore-related-settings.patch b/srcpkgs/utox/patches/0001-Fix-crash-when-changing-toxcore-related-settings.patch
new file mode 100644
index 00000000000..833b6e29dc3
--- /dev/null
+++ b/srcpkgs/utox/patches/0001-Fix-crash-when-changing-toxcore-related-settings.patch
@@ -0,0 +1,149 @@
+From c034ec9fffe79d46503dbb2686fec113bea2d08b Mon Sep 17 00:00:00 2001
+From: avoidr <avoidr@posteo.de>
+Date: Sat, 11 Jul 2020 05:18:22 +0200
+Subject: [PATCH] Fix crash when changing toxcore-related settings
+
+uTox killed toxcore without killing toxav first, which led to a crash.
+Resources should be deallocated in reverse order of their allocation.
+
+utox_av_ctrl_thread was started only once, when uTox was started, so
+when toxav was killed properly, the thread was never re-created for the
+next use of Tox. utox_av_ctrl_thread is now created whenever a new
+toxcore starts.
+
+Then, the Filter_Audio pointer had to be set to NULL after it's free'd,
+otherwise it wouldn't get re-initialised, and an invalid pointer would
+be passed to kill_filter_audio(), causing a crash.
+
+Finally, replaced magic number 0 with TOX_KILL.
+Deleted a redundant `toxav_thread_msg = false`.
+---
+ src/av/audio.c | 1 +
+ src/av/utox_av.c | 7 ++-----
+ src/main.c | 3 ---
+ src/tox.c | 20 +++++++++++---------
+ 4 files changed, 14 insertions(+), 17 deletions(-)
+
+diff --git a/src/av/audio.c b/src/av/audio.c
+index 324ceb22..d4384d8b 100644
+--- a/src/av/audio.c
++++ b/src/av/audio.c
+@@ -909,6 +909,7 @@ void utox_audio_thread(void *args) {
+ }
+
+ utox_filter_audio_kill(f_a);
++ f_a = NULL;
+
+ // missing some cleanup ?
+ alDeleteSources(1, &ringtone);
+diff --git a/src/av/utox_av.c b/src/av/utox_av.c
+index 90e04b36..d7adb9ce 100644
+--- a/src/av/utox_av.c
++++ b/src/av/utox_av.c
+@@ -275,7 +275,6 @@ void utox_av_ctrl_thread(void *UNUSED(args)) {
+ }
+ }
+ }
+-
+ toxav_thread_msg = false;
+
+ if (av) {
+@@ -286,19 +285,17 @@ void utox_av_ctrl_thread(void *UNUSED(args)) {
+ }
+ }
+
+-
+ postmessage_audio(UTOXAUDIO_KILL, 0, 0, NULL);
+ postmessage_video(UTOXVIDEO_KILL, 0, 0, NULL);
+-
+ // Wait for all a/v threads to return 0
+ while (utox_audio_thread_init || utox_video_thread_init) {
+ yieldcpu(1);
+ }
+
+- toxav_thread_msg = false;
++ toxav_kill(av);
++
+ utox_av_ctrl_init = false;
+
+- toxav_kill(av);
+ LOG_NOTE("UTOXAV", "Clean thread exit!");
+ return;
+ }
+diff --git a/src/main.c b/src/main.c
+index a3072edf..39218062 100644
+--- a/src/main.c
++++ b/src/main.c
+@@ -269,9 +269,6 @@ void utox_init(void) {
+ if (settings.curr_version != settings.last_version) {
+ settings.show_splash = true;
+ }
+-
+- // We likely want to start this on every system.
+- thread(utox_av_ctrl_thread, NULL);
+ }
+
+ void utox_raze(void) {
+diff --git a/src/tox.c b/src/tox.c
+index 8d93ee19..e9fe3ae8 100644
+--- a/src/tox.c
++++ b/src/tox.c
+@@ -234,10 +234,8 @@ void tox_settingschanged(void) {
+ dropdown_list_clear(&dropdown_audio_out);
+ dropdown_list_clear(&dropdown_video);
+
+- // send the reconfig message!
+- postmessage_toxcore(0, 1, 0, NULL);
+-
+ LOG_NOTE("Toxcore", "Restarting Toxcore");
++ postmessage_toxcore(TOX_KILL, 1, 0, NULL); // send the reconfig message!
+ while (!tox_thread_init) {
+ yieldcpu(1);
+ }
+@@ -476,8 +474,8 @@ void toxcore_thread(void *UNUSED(args)) {
+ // avoid trying the creation of thousands of tox instances before user changes the settings
+ if (tox_thread_msg) {
+ TOX_MSG *msg = &tox_msg;
+- // If msg->msg is 0, reconfig
+- if (!msg->msg) {
++
++ if (msg->msg == TOX_KILL) {
+ reconfig = (bool) msg->param1;
+ tox_thread_init = UTOX_TOX_THREAD_INIT_NONE;
+ }
+@@ -513,6 +511,7 @@ void toxcore_thread(void *UNUSED(args)) {
+ postmessage_utox(UPDATE_TRAY, 0, 0, NULL);
+ postmessage_utox(PROFILE_DID_LOAD, 0, 0, NULL);
+
++ thread(utox_av_ctrl_thread, NULL);
+ postmessage_utoxav(UTOXAV_NEW_TOX_INSTANCE, 0, 0, av);
+ }
+
+@@ -548,9 +547,9 @@ void toxcore_thread(void *UNUSED(args)) {
+ // If there's a message, load it, and send to the tox message thread
+ if (tox_thread_msg) {
+ TOX_MSG *msg = &tox_msg;
+- // If msg->msg is 0, reconfig if needed and break from tox_do
+- if (!msg->msg) {
+- reconfig = msg->param1;
++
++ if (msg->msg == TOX_KILL) {
++ reconfig = msg->param1; // reconfig if needed
+ tox_thread_msg = 0;
+ tox_thread_init = UTOX_TOX_THREAD_INIT_NONE;
+ break;
+@@ -574,7 +573,10 @@ void toxcore_thread(void *UNUSED(args)) {
+ write_save(tox);
+ edit_setstr(&edit_profile_password, (char *)"", 0);
+
+- // Stop toxcore.
++ postmessage_utoxav(UTOXAV_KILL, 0, 0, NULL);
++ while (utox_av_ctrl_init) {
++ yieldcpu(1);
++ }
+ LOG_TRACE("Toxcore", "tox thread ending");
+ tox_kill(tox);
+ }
+--
+2.26.2
+
diff --git a/srcpkgs/utox/template b/srcpkgs/utox/template
index 755c3203291..20f7f8c209d 100644
--- a/srcpkgs/utox/template
+++ b/srcpkgs/utox/template
@@ -1,7 +1,7 @@
# Template file for 'utox'
pkgname=utox
version=0.17.2
-revision=1
+revision=2
wrksrc=uTox
build_style=cmake
configure_args="-DENABLE_ASAN=OFF"
@@ -15,6 +15,7 @@ license="MIT"
homepage="https://github.com/uTox/uTox/"
distfiles="https://github.com/uTox/uTox/releases/download/v${version}/uTox-${version}-full.tar.gz"
checksum=8437827b899e4e358adaf0641680c214f4b6dfb52af6534f261d747a905f08b6
+patch_args="-Np1"
nocross=yes
post_install() {
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-08-11 23:15 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-13 0:51 [PR PATCH] utox: update to 0.17.2 avoidr
2020-04-13 21:34 ` [PR PATCH] [Updated] " avoidr
2020-04-13 21:35 ` avoidr
2020-04-13 22:12 ` avoidr
2020-04-14 14:47 ` [PR PATCH] [Merged]: " Hoshpak
2020-08-11 21:51 [PR PATCH] " avoidr
2020-08-11 23:15 ` [PR PATCH] [Updated] " avoidr
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).