[PATCH v4 0/2] Add in-place editing support to git interpret-trailers
- From
Tobias Klauser <tklauser@distanz.ch>
- Date
- Jan 14, 2016, 16:57 UTC
- Message-ID
- <1452790676-11937-1-git-send-email-tklauser@distanz.ch>
This patch series adds support for in-place editing to git interpret-trailers akin to sed -i, perl -i.
v3->v4: - Reword a test title, as suggested by Eric Sunshine. - Add a test to verify that the original file is not clobbered/deleted on error, as suggested by Eric Sunshine. - Move code specific to in-place editing from process_trailers() into a separate function to keep the overall flow clean. Suggested by Eric Sunshine. - Drop unnecessary braces, as pointed out by Eric Sunshine. - Use a more meaningful title for patch 1/2. Suggested by Junio Hamano.
v2->v3: - Rephrase two error messages according to the suggestions by Matthieu Moy.
v1->v2: - Split patch to make review easier, as suggested by Matthieu Moy. - Rename FILE * function parameters to a more readable name, as suggested by Matthieu Moy. - Write output to temporary file and rename after successfully written in full to avoid losing the original file in case of an error/interrupt. Pointed out by Eric Sunshine.
Tobias Klauser (2): trailer: allow to write to files other than stdout interpret-trailers: add option for in-place editing
Documentation/git-interpret-trailers.txt | 24 ++++++++++- builtin/interpret-trailers.c | 13 ++++-- t/t7513-interpret-trailers.sh | 40 ++++++++++++++++++ trailer.c | 69 +++++++++++++++++++++++++------- trailer.h | 3 +- 5 files changed, 129 insertions(+), 20 deletions(-)
-- 2.7.0.1.g5e091f5