Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#8571 - Rbutil - console interface

Attached to Project: Rockbox
Opened by Dominik Wenger (Domonoky) - Saturday, 09 February 2008, 20:07 GMT+2
Last edited by Dominik Riebeling (bluebrother) - Wednesday, 11 January 2012, 21:43 GMT+2
Task Type Patches
Category Rbutil
Status Closed
Assigned To Dominik Riebeling (bluebrother)
Dominik Wenger (Domonoky)
Player Type Another
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version RBUtil 1.1
Due Date Undecided
Percent Complete 100%
Private No

Details

A Console Interface for Rbutil.
Its still unfinished.

supported actions:
- all configurations
- Install Bootloader
- Install Rockbox
- Install Fonts
- Install Doom files
- Install voice file
- Create Voice file
- Create Talkfiles

Missing is:
- uninstallation
- Themes
- bugfixing :-)
- Code clean up

Extact the attached 7zip file int rbutil/rbutilqt so that there is a new folder rbutil/rbutilqt/cli. change into this folder and run qmake && make.
This task depends upon

Closed by  Dominik Riebeling (bluebrother)
Wednesday, 11 January 2012, 21:43 GMT+2
Reason for closing:  Out of Date
Additional comments about closing:  no activity since years.
Comment by Dominik Wenger (Domonoky) - Saturday, 09 February 2008, 20:14 GMT+2
the missing attachment.
   cli.7z (13.9 KiB)
Comment by Dominik Wenger (Domonoky) - Saturday, 09 February 2008, 21:04 GMT+2

Moved the Cli Folder up.
Put this new rbutilcli folder aside to the rbutilqt folder.
You can the update the rbutil.pro file to include both dirs, and build both version in one go.
Comment by Dominik Riebeling (bluebrother) - Sunday, 10 February 2008, 10:01 GMT+2
some ideas:
- make "esc" to always return to the previous menu
- in the configuratin menu, present the old values when entering the menu
- maybe (as long as we don't get more then 9 entries) accept menu selection input without enter confirmation?
- add a default action which is toggled by 0? That way we could provide a "usual installation path" for the user.
Comment by Dominik Riebeling (bluebrother) - Sunday, 10 February 2008, 12:41 GMT+2
current version doesn't build (at least on linux) due to the project file trying to build (and link) a GUI element. To fix it, remove browsedirtree.{cpp,h} from rbutilcli.pro. Ideally the project file would remove the need for QtGui (QT -= gui) for building, but that doesn't work yet.

In the long run it might be good to completely separate the logic of rbutil and the gui, so that we can have a new folder structure like this:
/rbutil/cli
/rbutil/gui
/rbutil/rbutilqt

But that's quite a way to go there and nothing that should happen before the cli is completed and in svn.
Comment by Dominik Wenger (Domonoky) - Sunday, 10 February 2008, 14:16 GMT+2

Planned improvments:
- present the old value (default value) for config option. allow to abort the config option.
- Improve Talkfile creation options.
- Better separation for gui / cli and rbutil base code. (after commit of this patch).

Update:
- The Return option is now always on 0.
- Autodetect on startup
- Fix a few bugs
- clean the code a bit.

Comment by Dominik Riebeling (bluebrother) - Sunday, 10 February 2008, 19:39 GMT+2
The attached diff removes the need to use QtGui with the currently attached cli version. It relies on the changes I recently committed to untie rbutil from QtGui (r16274).
   rbutilcli-noqtgui.diff (7.6 KiB)
 rbutilcli/autodetection.o                 |only
 rbutilcli/configurecli.cpp                |    4 ++--
 rbutilcli/configurecli.h                  |    2 +-
 rbutilcli/configurecli.o                  |only
 rbutilcli/encodersguicli.h                |    2 +-
 rbutilcli/encodersguicli.o                |only
 rbutilcli/encoders.o                      |only
 rbutilcli/httpget.o                       |only
 rbutilcli/installbootloader.o             |only
 rbutilcli/installzip.o                    |only
 rbutilcli/ipodio-posix.o                  |only
 rbutilcli/ipodpatcher.o                   |only
 rbutilcli/irivertools.o                   |only
 rbutilcli/main.cpp                        |    6 +++---
 rbutilcli/main.o                          |only
 rbutilcli/Makefile                        |only
 rbutilcli/md5sum.o                        |only
 rbutilcli/moc_autodetection.cpp           |only
 rbutilcli/moc_autodetection.o             |only
 rbutilcli/moc_configurecli.cpp            |only
 rbutilcli/moc_configurecli.o              |only
 rbutilcli/moc_encoders.cpp                |only
 rbutilcli/moc_encodersguicli.cpp          |only
 rbutilcli/moc_encodersguicli.o            |only
 rbutilcli/moc_encoders.o                  |only
 rbutilcli/moc_httpget.cpp                 |only
 rbutilcli/moc_httpget.o                   |only
 rbutilcli/moc_installbootloader.cpp       |only
 rbutilcli/moc_installbootloader.o         |only
 rbutilcli/moc_installzip.cpp              |only
 rbutilcli/moc_installzip.o                |only
 rbutilcli/moc_progressloggercli.cpp       |only
 rbutilcli/moc_progressloggercli.o         |only
 rbutilcli/moc_progressloggerinterface.cpp |only
 rbutilcli/moc_progressloggerinterface.o   |only
 rbutilcli/moc_rbsettings.cpp              |only
 rbutilcli/moc_rbsettings.o                |only
 rbutilcli/moc_rbutilcli.cpp               |only
 rbutilcli/moc_rbutilcli.o                 |only
 rbutilcli/moc_talkfile.cpp                |only
 rbutilcli/moc_talkfile.o                  |only
 rbutilcli/moc_tts.cpp                     |only
 rbutilcli/moc_ttsguicli.cpp               |only
 rbutilcli/moc_ttsguicli.o                 |only
 rbutilcli/moc_tts.o                       |only
 rbutilcli/moc_utilscli.cpp                |only
 rbutilcli/moc_utilscli.o                  |only
 rbutilcli/moc_voicefile.cpp               |only
 rbutilcli/moc_voicefile.o                 |only
 rbutilcli/progressloggercli.h             |    2 +-
 rbutilcli/progressloggercli.o             |only
 rbutilcli/qrc_rbutilqt.cpp                |only
 rbutilcli/qrc_rbutilqt.o                  |only
 rbutilcli/rbsettings.o                    |only
 rbutilcli/rbutilcli                       |only
 rbutilcli/rbutilcli.bz2                   |only
 rbutilcli/rbutilcli.cpp                   |    8 ++++----
 rbutilcli/rbutilcli.o                     |only
 rbutilcli/rbutilcli.pro                   |    1 +
 rbutilcli/sansaio-posix.o                 |only
 rbutilcli/sansapatcher.o                  |only
 rbutilcli/talkfile.o                      |only
 rbutilcli/ttsguicli.h                     |    2 +-
 rbutilcli/ttsguicli.o                     |only
 rbutilcli/tts.o                           |only
 rbutilcli/unzip.o                         |only
 rbutilcli/utilscli.cpp                    |    2 +-
 rbutilcli/utilscli.h                      |    2 +-
 rbutilcli/utilscli.o                      |only
 rbutilcli/utils.o                         |only
 rbutilcli/voicefile.o                     |only
 rbutilcli/voicefont.o                     |only
 rbutilcli/wavtrim.o                       |only
 rbutilcli/zip.o                           |only
 74 files changed, 16 insertions(+), 15 deletions(-)

Comment by Dominik Wenger (Domonoky) - Monday, 11 February 2008, 18:40 GMT+2

The Diff from Bluebrother was in a strange format.

The Diff plus my latest changes (sapi4 support, and bugfix in the sapi settings) as the rbutilcli folder attached.

Comment by Dominik Riebeling (bluebrother) - Friday, 15 February 2008, 21:10 GMT+2
Updated:
- update to build with current svn changes
- fix some strings to follow typographic rules

Some source files should get renamed IMO to use more sensible names (like encodersguicli.cpp -- is that gui or cli?) but I haven't changed that right now.
Comment by Dominik Riebeling (bluebrother) - Monday, 31 March 2008, 00:29 GMT+2
small update:
- make build with current svn again
- make empty input behave as return (i.e. selecting option 0)
Comment by Dominik Riebeling (bluebrother) - Wednesday, 16 April 2008, 23:24 GMT+2
updated again. Fixes a segfault when building against current svn. Untested, just wanted to give people the chance to try this out (or even picking it up ;-)

Loading...