Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • G GLib
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 851
    • Issues 851
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 50
    • Merge requests 50
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • GNOMEGNOME
  • GLib
  • Merge requests
  • !2354

gtestdbus: Print the dbus address on a specific FD intead of stdout

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Marco Trevisan requested to merge 3v1n0/glib:dbus-daemon-print-address-fd into main Nov 23, 2021
  • Overview 5
  • Commits 1
  • Pipelines 6
  • Changes 1

We used to use a pipe for the dbus daemon stdout to read the defined address, but that was already requiring a workaround to ensure that dbus daemon children were then able to write to stdout. However the current implementation is still causing troubles in some cases in which the daemon is very verbose, leading to hangs when writing to stdout.

As per this, just don't handle stdout ourself, but use instead a specific pipe to get the address address. That can now be safely closed once we've received the data we need.

This reverts commit d80adeaa.

Fixes: #2537 (closed)

/cc @bberg

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: dbus-daemon-print-address-fd