Opened 5 years ago
Closed 4 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 flashrom.org) | Keywords: | usability cli options |
| Cc: | Dependencies: | ||
| Patch Status: | there is no patch |
Description
Attachments (1)
Change History (5)
Changed 5 years ago by stuge
comment:1 Changed 5 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 4 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 4 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 4 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".

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