#100: flashrom: Change start/end -s and -e to -S and -E, and change erase -E to -e. -------------------------------------+-------------------------------------- Reporter: stuge | Owner: somebody Type: enhancement | Status: new Priority: major | Milestone: flashrom v1.0 Component: flashrom | Version: Keywords: usability cli options | Dependencies: Patchstatus: patch needs review | -------------------------------------+--------------------------------------
#100: flashrom: Change start/end -s and -e to -S and -E, and change erase -E to -e. ----------------------------+----------------------------------------------- Reporter: stuge | Owner: somebody Type: enhancement | Status: new Priority: major | Milestone: flashrom v1.0 Component: flashrom | Version: Resolution: | Keywords: usability cli options Dependencies: | Patchstatus: patch needs review ----------------------------+-----------------------------------------------
Comment(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.
#100: flashrom: Change start/end -s and -e to -S and -E, and change erase -E to -e. ----------------------------+----------------------------------------------- Reporter: stuge | Owner: somebody Type: enhancement | Status: closed Priority: major | Milestone: flashrom v1.1 Component: flashrom | Version: Resolution: duplicate | Keywords: usability cli options Dependencies: | Patchstatus: there is no patch ----------------------------+----------------------------------------------- Changes (by stuge):
* status: new => closed * patchstatus: patch needs review => there is no patch * resolution: => duplicate * milestone: flashrom v1.0 => flashrom v1.1
Comment:
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.
#100: flashrom: Change start/end -s and -e to -S and -E, and change erase -E to -e. -------------------------+-------------------------------------------------- Reporter: stuge | Owner: somebody Type: defect | Status: reopened Priority: major | Milestone: flashrom v1.0 Component: flashrom | Version: Resolution: | Keywords: usability cli options Dependencies: | Patchstatus: there is no patch -------------------------+-------------------------------------------------- Changes (by stuge):
* status: closed => reopened * type: enhancement => defect * resolution: duplicate => * milestone: flashrom v1.1 => flashrom v1.0
Comment:
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.
#100: flashrom: Change start/end -s and -e to -S and -E, and change erase -E to -e. -------------------------+-------------------------------------------------- Reporter: stuge | Owner: somebody Type: defect | Status: closed Priority: major | Milestone: flashrom v1.0 Component: flashrom | Version: Resolution: fixed | Keywords: usability cli options Dependencies: | Patchstatus: there is no patch -------------------------+-------------------------------------------------- Changes (by hailfinger):
* status: reopened => closed * resolution: => fixed
Comment:
--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".
-
coreboot