Opened 7 years ago

Closed 6 years ago

#100 closed defect (fixed)

flashrom: Change start/end -s and -e to -S and -E, and change erase -E to -e.

Reported by: stuge Owned by: somebody
Priority: major Milestone: flashrom v1.0
Component: flashrom (please use trac on Keywords: usability cli options
Cc: Dependencies:
Patch Status: there is no patch


Attachments (1)

fr.5.swapopts.patch (5.6 KB) - added by stuge 7 years ago.
applies to r3276, will gladly rebase if this is agreed on

Download all attachments as: .zip

Change History (5)

Changed 7 years ago by stuge

applies to r3276, will gladly rebase if this is agreed on

comment:1 Changed 7 years ago by hailfinger

If you can make sure nobody erases his chip by accident with this patch, why not? That would reauire additional command line argument verification, though.

comment:2 Changed 7 years ago by stuge

  • Milestone changed from flashrom v1.0 to flashrom v1.1
  • Patch Status changed from patch needs review to there is no patch
  • Resolution set to duplicate
  • Status changed from new to closed

Moving to 1.1 where -s -e should be replaced anyway, with something like -s -n which would specify an inclusive byte range to operate on, AKA partial operation. Depends/duplicate #104.

comment:3 Changed 7 years ago by stuge

  • Milestone changed from flashrom v1.1 to flashrom v1.0
  • Resolution duplicate deleted
  • Status changed from closed to reopened
  • Type changed from enhancement to defect

Carl-Daniel Hailfinger wrote:

  • The short command lines switches have no decided upon capitalization.


We can either say "screw users, they should read the man page every time they update" or we decide user interface questions now.

This is tricky. There are already users which know and use the existing user interface. There are good reasons for changing option capitalization. And there are good reasons both for doing it before 1.0 and for waiting until after 1.0.

We should try to pick the lesser evil of course, and also make sure to provide prominent notices in release announcement, README and man page when we change any of the existing options. Reopening for more discussion.

comment:4 Changed 6 years ago by hailfinger

  • Resolution set to fixed
  • Status changed from reopened to closed

--exclude-start and --exclude-end have been killed in r608, so this ticket has been obsoleted.

Unfortunately, there is no resolution "obsoleted", so I'm closing it as "fixed".

Add Comment

Modify Ticket

as closed The owner will remain somebody.
The resolution will be deleted. Next status will be 'reopened'.

E-mail address and user name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.