Skip to content

Improve the reporting of D-Bus connection error#660

Merged
imobachgs merged 1 commit intoagama-project:masterfrom
imobachgs:connection-error
Jul 13, 2023
Merged

Improve the reporting of D-Bus connection error#660
imobachgs merged 1 commit intoagama-project:masterfrom
imobachgs:connection-error

Conversation

@imobachgs
Copy link
Copy Markdown
Contributor

@imobachgs imobachgs commented Jul 13, 2023

Improve the error reporting when a D-Bus connection cannot be established. By the way, it updates the changes file (I forgot to do it on #659).

Before:

Error: Connecting to Agama bus at unix:path=/run/agama/bus

Caused by:
    0: Connecting to Agama bus at unix:path=/run/agama/bus
    1: I/O error: Broken pipe (os error 32)
    2: Broken pipe (os error 32)

Now:

Could not connect to Agama bus at 'unix:path=/run/agama/bus'

Caused by:
    0: I/O error: Broken pipe (os error 32)
    1: Broken pipe (os error 32)

@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 72.398%. remained the same when pulling 67cfefc on imobachgs:connection-error into f33d147 on openSUSE:master.

Copy link
Copy Markdown
Contributor

@joseivanlopez joseivanlopez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@imobachgs imobachgs merged commit 083f2d1 into agama-project:master Jul 13, 2023
@imobachgs imobachgs deleted the connection-error branch July 13, 2023 12:14
@imobachgs imobachgs mentioned this pull request Jul 13, 2023
@imobachgs imobachgs mentioned this pull request Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants