The changes are still made in a way that there's never a commit where something is broken that wasn't before, hence the function split happening first, before we enable and use pre_run. Let me know if you want me to rejig this, I think that the end result is fine, but you might have a different opinion about the way we get to that result. Cheers Changes from v3: - dropped glib upgrade - use pre_run menu hook to setup dbus if needed Changes from v2: - rebased against latest upstream Bastien Nocera (9): client: Split installing submenu and doing I/O client: Install submenus before contacting bluez daemon shared/shell: Add function to handle early help calls client: Fix --help hanging if bluetoothd is not running shared/shell: Add pre_run menu callback client: Port "admin" menu to pre_run client: Port "player" menu to pre_run client: Port "mgmt" menu to pre_run client: Port "assistant" menu to pre_run client/admin.c | 13 +++++++++++++ client/assistant.c | 12 +++++++++++- client/main.c | 13 +++++++------ client/mgmt.c | 23 +++++++++++++++++------ client/mgmt.h | 2 +- client/player.c | 11 ++++++++++- src/shared/shell.c | 19 +++++++++++++++++++ src/shared/shell.h | 4 ++++ tools/btmgmt.c | 6 +----- 9 files changed, 83 insertions(+), 20 deletions(-) -- 2.49.0