[chirp_devel] [PATCH 4 of 4] [ft8x7] Add messages to help users remember clone procedure

Tom Hayward
Tue Oct 8 09:22:32 PDT 2013


On Tue, Oct 8, 2013 at 7:16 AM, Marco Filippi IZ3GME
<iz3gme.marco at gmail.com> wrote:
> @@ -304,8 +316,10 @@
>          data = ""
>          blocks = 0
>          status = chirp_common.Status()
> -        status.msg = "Cloning from radio"
> +        status.msg = self._START_CLONE_TX_MSG

I don't think this is the best place to print the instructions. There
should be an intermediary step so that the clone does not timeout
while they are reading instructions. It could be added just after the
experimental driver warning. In fact, the experimental driver warning
code is almost exactly what we need, but without the Yes/No prompt.

> +                    if blocks == 0:
> +                        raise Exception(_("Radio did not ack first block\n")
> +                                 + self._START_CLONE_RX_MSG)

This never prints because the first iteration of the loop is blocks ==
1. Again, it would be more useful to display the instructions prior to
starting the clone loop.

Tom KD7LXL



More information about the chirp_devel mailing list