Skip to content

Commit

Permalink
Clarify uses of stand-alone patch.
Browse files Browse the repository at this point in the history
  • Loading branch information
@mitar
mitar committed Nov 1, 2019
1 parent 053df86 commit 606f2f1
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions draft-xx-httpbis-range-patch-00.txt
View file
Original file line number Diff line number Diff line change
Expand Up @@ -172,9 +172,11 @@ Table of Contents

When range patches are transmitted outside of HTTP session, a
stand-alone range patch format can be used. For example, in this
format it can be stored in a file. The format reuses structure from
HTTP and consists of headers separated from the patch body by an
empty line. Only Content-Range header is required. Example:
format a patch can be stored in a file, send to a mailing list, or a
code version control system can display the patch in the range patch
format. The format reuses structure from HTTP and consists of
headers separated from the patch body by an empty line. Only
Content-Range header is required. Example:

This comment has been minimized.

Sign in to view

Copy link
@toomim

toomim Nov 1, 2019

Member

I'm still not sold on this part, but that's ok.

One issue is that you can't replace multiple ranges in a single file with this format. That makes it a poor substitute for a universal diff, because you need a new file for each edit you made to a file.

This comment has been minimized.

Sign in to view

Copy link
@mitar

mitar Nov 1, 2019

Author Member

Why you cannot? You use multiple range patches in the file.

Maybe I can clarify that this is also possible if it is not clear.

This comment has been minimized.

Sign in to view

Copy link
@toomim

toomim Nov 1, 2019

Member

Ok, yeah it was not clear when I read it. It looked like a single patch per file. Otherwise, it needs to specify how you delimit multiple patches.

This comment has been minimized.

Sign in to view

Copy link
@mitar

mitar Nov 1, 2019

Author Member

Same as in the protocol.

Please open an issue for this as well.

This comment has been minimized.

Sign in to view

Copy link
@toomim

toomim Feb 13, 2021

Member

@josephg You've been interested in the question of whether to allow multiple custom patches per version. There is some discussion on this ^^^.
Content-Range: json /foo/bar/3/baz

Expand Down

0 comments on commit 606f2f1

Please sign in to comment.