[bug #50087] option parse error: after -U flash:w:program_to_upload.hex, subsequent -V ignored

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[bug #50087] option parse error: after -U flash:w:program_to_upload.hex, subsequent -V ignored

Joerg Wunsch-6
URL:
  <http://savannah.nongnu.org/bugs/?50087>

                 Summary: option parse error: after -U
flash:w:program_to_upload.hex, subsequent -V ignored
                 Project: AVR Downloader/UploaDEr
            Submitted by: None
            Submitted on: Thu 19 Jan 2017 09:08:55 AM UTC
                Category: None
                Severity: 3 - Normal
                Priority: 5 - Normal
              Item Group: None
                  Status: None
                 Privacy: Public
             Assigned to: None
         Originator Name: Britton Kerin
        Originator Email: [hidden email]
             Open/Closed: Open
                 Release: 6.3
         Discussion Lock: Any
     Programmer hardware: arduino
             Device type: atmega328P

    _______________________________________________________

Details:

Moving the -V in front of -U causes the read-back verification to be skipped
as expected.

I guess it's probably all options and not just -V.

I wildly guess that it's somehow related to the not-present optional 4th
sub-argument to the -U option (the format argument).




    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?50087>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/


_______________________________________________
avrdude-dev mailing list
[hidden email]
https://lists.nongnu.org/mailman/listinfo/avrdude-dev
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[bug #50087] option parse error: after -U flash:w:program_to_upload.hex, subsequent -V ignored

Joerg Wunsch-6
Update of bug #50087 (project avrdude):

                Severity:              3 - Normal => 2 - Minor              
                  Status:                    None => Confirmed              

    _______________________________________________________

Follow-up Comment #1:

The problem is that the -U option parser already evaluates
the "verify" flag (see the "HERE!" comment):


      case 'U':
        upd = parse_op(optarg);
        if (upd == NULL) {
          avrdude_message(MSG_INFO, "%s: error parsing update operation
'%s'\n",
                  progname, optarg);
          exit(1);
        }
        ladd(updates, upd);

        if (/* HERE! */ verify && upd->op == DEVICE_WRITE) {
          upd = dup_update(upd);
          upd->op = DEVICE_VERIFY;
          ladd(updates, upd);
        }
        break;


Thus, if the -V option has not been present at this point,
it's being ignored ("verify" defaults to 1).

Unfortunately, there's no easy fix for that.

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?50087>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/


_______________________________________________
avrdude-dev mailing list
[hidden email]
https://lists.nongnu.org/mailman/listinfo/avrdude-dev
Loading...