Re: cvs commit: src/sbin/hammer Makefile cmd_snapshot.c hammer.h hammer.c

[Michael Neumann <mneumann@xxxxxxxx>]

Michael Neumann wrote:
> mneumann    2008/06/25 06:10:06 PDT
>
> DragonFly src repository
>
>   Modified files:
>     sbin/hammer          Makefile hammer.h hammer.c 
>   Added files:
>     sbin/hammer          cmd_snapshot.c 
>   Log:
>   Implement: hammer snapshot <softlink-dir> [<filesystem>]

How about to change the command line syntax into:

  hammer snapshot <softlink-dir> [<snapshot-name>] [<filesystem>]

Alternatively, it could be possible (slightly "harder" to implement), so
achive the same with:

  hammer snapshot <softlink-dir+snapshot-name> [<filesystem>]

So that the hammer utility would determine whether the path given for
softlink-dir actually exists, if not, it is assumed to be the snapshot
name. I don't like it that much, because there are some implicit
assumption, so if nobody objects, I'd go with the first approach.

Right now, the snapshots generated are named after the transaction id,
something like: 0xffffffffffffffffff, which clearly is not that
beautiful!

Regards,

  Michael