blacklight/libsoup
1
0
Fork 0
Code Issues Pull requests 1 Projects Releases Packages Wiki Activity Actions

fix: Replace occurrences of g_module_* in soup2_is_loaded with dl* #1

Open
blacklight wants to merge 1 commit from test/comment-libsoup2-check-in-init into master
pull from: test/comment-libsoup2-check-in-init
merge into: blacklight:master
Conversation 0 Commits 1 Files changed 1 +16 -6
blacklight commented 2025-07-15 22:37:10 +02:00
Owner
Copy link

Calling g_module_open in the library constructor can cause deadlocks when libsoup is used with other libraries that also contend for GLib mutexes.

dlopen should be used instead.

Context: https://gitlab.gnome.org/GNOME/libsoup/-/issues/463#note_2497431
Docs: https://gitlab.gnome.org/GNOME/glib/-/merge_requests/4691

Calling `g_module_open` in the library constructor can cause deadlocks when libsoup is used with other libraries that also contend for GLib mutexes. `dlopen` should be used instead. Context: https://gitlab.gnome.org/GNOME/libsoup/-/issues/463#note_2497431 Docs: https://gitlab.gnome.org/GNOME/glib/-/merge_requests/4691
Calling `g_module_open` in the library constructor can cause deadlocks
when libsoup is used with other libraries that also contend for GLib
mutexes.

`dlopen` should be used instead.

Context: https://gitlab.gnome.org/GNOME/libsoup/-/issues/463#note_2497431
Docs: https://gitlab.gnome.org/GNOME/glib/-/merge_requests/4691
blacklight force-pushed test/comment-libsoup2-check-in-init from 6426df07ca to dccfc355d9 2025-07-15 22:40:57 +02:00 Compare
blacklight force-pushed test/comment-libsoup2-check-in-init from dccfc355d9 to b98bc20276 2025-07-15 22:51:35 +02:00 Compare
blacklight force-pushed test/comment-libsoup2-check-in-init from b98bc20276 to cc0d40b020 2025-07-15 22:54:10 +02:00 Compare
blacklight force-pushed test/comment-libsoup2-check-in-init from cc0d40b020 to 604f8cfa5e 2025-07-15 23:12:38 +02:00 Compare
blacklight force-pushed test/comment-libsoup2-check-in-init from 604f8cfa5e to 644c541425 2025-07-15 23:14:41 +02:00 Compare
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin test/comment-libsoup2-check-in-init:test/comment-libsoup2-check-in-init
git switch test/comment-libsoup2-check-in-init

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch master
git merge --no-ff test/comment-libsoup2-check-in-init
git switch test/comment-libsoup2-check-in-init
git rebase master
git switch master
git merge --ff-only test/comment-libsoup2-check-in-init
git switch test/comment-libsoup2-check-in-init
git rebase master
git switch master
git merge --no-ff test/comment-libsoup2-check-in-init
git switch master
git merge --squash test/comment-libsoup2-check-in-init
git switch master
git merge --ff-only test/comment-libsoup2-check-in-init
git switch master
git merge test/comment-libsoup2-check-in-init
git push origin master
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: blacklight/libsoup#1
No description provided.
Delete branch "test/comment-libsoup2-check-in-init"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?