Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle broken pipe issue #17

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

pirat89
Copy link
Member

@pirat89 pirat89 commented May 23, 2023

One of common use-cases is piping the leapp inspector output to another utilis - such as less:
$ leapp-inspector messages | less

It's annoying to see the broken
pipe traceback after closing the application. Fix that for Py2 & Py3 to not see this error anymore.

Note

Currently it works just under Py3. Py2 is still broken.

One of common use-cases is piping the leapp inspector output to
another utilis - such as less:
  $ leapp-inspector messages | less

It's annoying to see the broken
pipe traceback after closing the application. Fix that for Py2 & Py3
to not see this error anymore.
@pirat89 pirat89 marked this pull request as ready for review November 11, 2024 16:26
@pirat89
Copy link
Member Author

pirat89 commented Nov 11, 2024

Considering that it does not make sense to fix it for Py2. As far as the solution has not impact on py2, I am considering it ok.

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.

1 participant