transport: clarify error message when transport is not selected

When no transport is selected, the error message dumps the available
transports, but not how to actually select one.

Change-Id: I63da2a4b59e3f6cc8d30bd631e41a82636a056ef
Signed-off-by: Karl Palsson <karlp@tweak.net.au>
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/2406
Tested-by: jenkins
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>

diff --git a/src/transport/transport.c b/src/transport/transport.c
index 3f5337e35a7c95464ffd4d9bed48c1047360cc68..c57064bbdde8b40d0e17c0155ea98769e7eb4367 100644 (file)
--- a/src/transport/transport.c
+++ b/src/transport/transport.c
@@ -239,12 +239,13 @@ COMMAND_HANDLER(handle_transport_init)
 {
        LOG_DEBUG("%s", __func__);
        if (!session) {
-               LOG_ERROR("session's transport is not selected.");
+               LOG_ERROR("session transport was not selected. Use 'transport select <transport>'");
 
                /* no session transport configured, print transports then fail */
+               LOG_ERROR("Transports available:");
                const char * const *vector = allowed_transports;
                while (*vector) {
-                       LOG_ERROR("allow transport '%s'", *vector);
+                       LOG_ERROR("%s", *vector);
                        vector++;
                }
                return ERROR_FAIL;
@@ -277,7 +278,7 @@ static int jim_transport_select(Jim_Interp *interp, int argc, Jim_Obj * const *a
        switch (argc) {
                case 1:         /* return/display */
                        if (!session) {
-                               LOG_ERROR("session's transport is not selected.");
+                               LOG_ERROR("session transport was not selected. Use 'transport select <transport>'");
                                return JIM_ERR;
                        } else {
                                Jim_SetResultString(interp, session->name, -1);