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

dbus-daemon-proxy: add missing return statement #518

Conversation

leiflm
Copy link
Contributor

@leiflm leiflm commented Jan 18, 2022

The missing return statement leads to a SIGABRT.

Signed-off-by: Leif Middelschulte Leif.Middelschulte@klsmartin.com

The missing `return` statement leads to a `SIGABRT`.

Signed-off-by: Leif Middelschulte <Leif.Middelschulte@klsmartin.com>
@akuster
Copy link
Contributor

akuster commented Jan 29, 2022

thanks. change in my testing branch.

@kraj
Copy link
Contributor

kraj commented Feb 8, 2022

applied with b5a9b02

@kraj kraj closed this Feb 8, 2022
kraj pushed a commit to YoeDistro/meta-openembedded that referenced this pull request Jul 25, 2023
- The project uses poetry now, so inherit python_poetry_core
- Add python3-asyncio and python3-compression to RDEPENDS

Changelog (https://github.com/amoffat/sh/blob/develop/CHANGELOG.md):

2.0.4 - 5/13/22

- Allow ok_code to be used with fg openembedded#665
- Make sure new_group never creates a new session openembedded#675

2.0.2 / 2.0.3 (misversioned) - 2/13/22

- Performance regression when using a generator with _in openembedded#650
- Adding test support for python 3.11

2.0.0 - 2/9/22

- Executed commands now return a unicode string by default
- Removed magical module-like execution contexts openembedded#636
- Added basic asyncio support via _async
- Dropped support for Python < 3.8
- Bumped default tty size to more standard (24, 80)
- First argument being a RunningCommand no longer automatically passes it as stdin
- RunningCommand.__eq__ no longer has the side effect of executing the command openembedded#518
- _tee now supports both "err" and "out" openembedded#215
- Removed the builtin override cd link
- Altered process launching model to behave more expectedly openembedded#495
- Bugfix where _no_out isn't allowed with _iter="err" openembedded#638
- Allow keyword arguments to have a list of values openembedded#529

Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
Signed-off-by: Khem Raj <raj.khem@gmail.com>
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