[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 1/6] add documenation for new backup framewor
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v4 1/6] add documenation for new backup framework |
Date: |
Thu, 21 Feb 2013 13:41:46 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Thu, Feb 21, 2013 at 05:14:04AM +0000, Dietmar Maurer wrote:
> > Thanks for writing this up. I don't think docs/backup.txt should be
> > committed
> > as-is though because it refers to you proposing this patch series. Once
> > merged
> > some of this document will no longer be relevant.
>
> Why will it be no longer relevant? It explain the basic idea.
>
> > You could include this in the patch series cover letter instead.
>
> I use 'git' to have a versioning system. using cover-lettter instead defeats
> that purpose.
It's a statement about the patch series at a given point in time. It
will quickly become outdated on the code is extended/modified.
I guess the cover letter is the wrong place, but commit messages would
be the right place. Then it is versioned by git but refers to a
specific tree so it doesn't become outdated like a file in the source
tree would.
Also, "Please let us know if you know an existing format which
provides the same functionality" sounds like you are addressing patch
reviewers.
Stefan
- Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver, (continued)
- Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver, Dietmar Maurer, 2013/02/26
- Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver, Stefan Hajnoczi, 2013/02/27
- Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver, Dietmar Maurer, 2013/02/27
- Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver, Kevin Wolf, 2013/02/28
- Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver, Stefan Hajnoczi, 2013/02/26
- Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver, Dietmar Maurer, 2013/02/26
- [Qemu-devel] [PATCH v4 6/6] add vm state to backups, Dietmar Maurer, 2013/02/20
- [Qemu-devel] [PATCH v4 1/6] add documenation for new backup framework, Dietmar Maurer, 2013/02/20
- [Qemu-devel] [PATCH v4 3/6] add backup related monitor commands, Dietmar Maurer, 2013/02/20
- Re: [Qemu-devel] [PATCH v4 3/6] add backup related monitor commands, Stefan Hajnoczi, 2013/02/20
- Re: [Qemu-devel] [PATCH v4 3/6] add backup related monitor commands, Dietmar Maurer, 2013/02/21
- Re: [Qemu-devel] [PATCH v4 3/6] add backup related monitor commands, Stefan Hajnoczi, 2013/02/21
- Re: [Qemu-devel] [PATCH v4 3/6] add backup related monitor commands, Dietmar Maurer, 2013/02/21
- Re: [Qemu-devel] [PATCH v4 3/6] add backup related monitor commands, Stefan Hajnoczi, 2013/02/22
- Re: [Qemu-devel] [PATCH v4 3/6] add backup related monitor commands, Dietmar Maurer, 2013/02/22
- Re: [Qemu-devel] [PATCH v4 3/6] add backup related monitor commands, Stefan Hajnoczi, 2013/02/25
- Re: [Qemu-devel] [PATCH v4 3/6] add backup related monitor commands, Dietmar Maurer, 2013/02/27
- Re: [Qemu-devel] [PATCH v4 3/6] add backup related monitor commands, Stefan Hajnoczi, 2013/02/28