Opened 14 months ago

Closed 14 months ago

#514 closed defect (fixed)

Unify the syntax used by various qvm-tools...

Reported by: joanna Owned by: joanna
Priority: major Milestone: Release 1
Component: core Keywords:
Cc: marmarek

Description

...specifically the place where the DstVM argument is given. Some tools (e.g. qvm-copy-to-vm) take the DstVM as the first argument ("Intel syntax"), while others (e.g. qvm-block -a) take it as last argument ("AT&T syntax"). Other tools to consider: qvm-pci (currently Intel syntax), qvm-clone ("AT&T syntax"), and probably a few others I've missed.

Change History (3)

comment:1 Changed 14 months ago by marmarek

qvm-clone is just like cp - IMHO now it is good order.

comment:2 Changed 14 months ago by aga

And remember to update the docs.

comment:3 Changed 14 months ago by joanna

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

​http://git.qubes-os.org/?p=marmarek/core.git;a=commit;h=e5a2209cab096e4b5877c5f73f87bd357486b199

So, generally we agreed that the "Intel syntax" is better, and now all tools seem to follow. With the exception of qvm-clone, which uses the "AT&T syntax", as it is really more intuitive in this case.

Note: See TracTickets for help on using tickets.