From: Felipe Contreras <felipe.contreras@gmail.com>
To: git@vger.kernel.org
Cc: Junio C Hamano <gitster@pobox.com>,
Brandon Casey <drafnel@gmail.com>,
Jonathan Nieder <jrnieder@gmail.com>,
Ramkumar Ramachandra <artagnon@gmail.com>,
Felipe Contreras <felipe.contreras@gmail.com>
Subject: [PATCH 1/2] log-tree: remove dependency from sequencer
Date: Fri, 7 Jun 2013 17:16:48 -0500 [thread overview]
Message-ID: <1370643409-3431-2-git-send-email-felipe.contreras@gmail.com> (raw)
In-Reply-To: <1370643409-3431-1-git-send-email-felipe.contreras@gmail.com>
Move the relevant code from sequencer to log-tree. This code is not
specific to sequencer, and this allows the sequencer to move out of
libgit.
Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
log-tree.c | 161 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-
log-tree.h | 3 ++
sequencer.c | 160 ++---------------------------------------------------------
sequencer.h | 4 --
4 files changed, 166 insertions(+), 162 deletions(-)
diff --git a/log-tree.c b/log-tree.c
index 2eb69bc..654f5db 100644
--- a/log-tree.c
+++ b/log-tree.c
@@ -9,10 +9,13 @@
#include "string-list.h"
#include "color.h"
#include "gpg-interface.h"
-#include "sequencer.h"
#include "line-log.h"
+#define APPEND_SIGNOFF_DEDUP (1u << 0)
+
struct decoration name_decoration = { "object names" };
+const char sign_off_header[] = "Signed-off-by: ";
+static const char cherry_picked_prefix[] = "(cherry picked from commit ";
enum decoration_type {
DECORATION_NONE = 0,
@@ -472,6 +475,162 @@ static void show_mergetag(struct rev_info *opt, struct commit *commit)
free_commit_extra_headers(to_free);
}
+static int is_rfc2822_line(const char *buf, int len)
+{
+ int i;
+
+ for (i = 0; i < len; i++) {
+ int ch = buf[i];
+ if (ch == ':')
+ return 1;
+ if (!isalnum(ch) && ch != '-')
+ break;
+ }
+
+ return 0;
+}
+
+static int is_cherry_picked_from_line(const char *buf, int len)
+{
+ /*
+ * We only care that it looks roughly like (cherry picked from ...)
+ */
+ return len > strlen(cherry_picked_prefix) + 1 &&
+ !prefixcmp(buf, cherry_picked_prefix) && buf[len - 1] == ')';
+}
+
+/*
+ * Returns 0 for non-conforming footer
+ * Returns 1 for conforming footer
+ * Returns 2 when sob exists within conforming footer
+ * Returns 3 when sob exists within conforming footer as last entry
+ */
+int has_conforming_footer(struct strbuf *sb, struct strbuf *sob,
+ int ignore_footer)
+{
+ char prev;
+ int i, k;
+ int len = sb->len - ignore_footer;
+ const char *buf = sb->buf;
+ int found_sob = 0;
+
+ /* footer must end with newline */
+ if (!len || buf[len - 1] != '\n')
+ return 0;
+
+ prev = '\0';
+ for (i = len - 1; i > 0; i--) {
+ char ch = buf[i];
+ if (prev == '\n' && ch == '\n') /* paragraph break */
+ break;
+ prev = ch;
+ }
+
+ /* require at least one blank line */
+ if (prev != '\n' || buf[i] != '\n')
+ return 0;
+
+ /* advance to start of last paragraph */
+ while (i < len - 1 && buf[i] == '\n')
+ i++;
+
+ for (; i < len; i = k) {
+ int found_rfc2822;
+
+ for (k = i; k < len && buf[k] != '\n'; k++)
+ ; /* do nothing */
+ k++;
+
+ found_rfc2822 = is_rfc2822_line(buf + i, k - i - 1);
+ if (found_rfc2822 && sob &&
+ !strncmp(buf + i, sob->buf, sob->len))
+ found_sob = k;
+
+ if (!(found_rfc2822 ||
+ is_cherry_picked_from_line(buf + i, k - i - 1)))
+ return 0;
+ }
+ if (found_sob == i)
+ return 3;
+ if (found_sob)
+ return 2;
+ return 1;
+}
+
+void append_cherrypick(struct strbuf *msgbuf, struct object *obj)
+{
+ if (!has_conforming_footer(msgbuf, NULL, 0))
+ strbuf_addch(msgbuf, '\n');
+ strbuf_addstr(msgbuf, cherry_picked_prefix);
+ strbuf_addstr(msgbuf, sha1_to_hex(obj->sha1));
+ strbuf_addstr(msgbuf, ")\n");
+}
+
+void append_signoff(struct strbuf *msgbuf, int ignore_footer, unsigned flag)
+{
+ unsigned no_dup_sob = flag & APPEND_SIGNOFF_DEDUP;
+ struct strbuf sob = STRBUF_INIT;
+ int has_footer;
+
+ strbuf_addstr(&sob, sign_off_header);
+ strbuf_addstr(&sob, fmt_name(getenv("GIT_COMMITTER_NAME"),
+ getenv("GIT_COMMITTER_EMAIL")));
+ strbuf_addch(&sob, '\n');
+
+ /*
+ * If the whole message buffer is equal to the sob, pretend that we
+ * found a conforming footer with a matching sob
+ */
+ if (msgbuf->len - ignore_footer == sob.len &&
+ !strncmp(msgbuf->buf, sob.buf, sob.len))
+ has_footer = 3;
+ else
+ has_footer = has_conforming_footer(msgbuf, &sob, ignore_footer);
+
+ if (!has_footer) {
+ const char *append_newlines = NULL;
+ size_t len = msgbuf->len - ignore_footer;
+
+ if (!len) {
+ /*
+ * The buffer is completely empty. Leave foom for
+ * the title and body to be filled in by the user.
+ */
+ append_newlines = "\n\n";
+ } else if (msgbuf->buf[len - 1] != '\n') {
+ /*
+ * Incomplete line. Complete the line and add a
+ * blank one so that there is an empty line between
+ * the message body and the sob.
+ */
+ append_newlines = "\n\n";
+ } else if (len == 1) {
+ /*
+ * Buffer contains a single newline. Add another
+ * so that we leave room for the title and body.
+ */
+ append_newlines = "\n";
+ } else if (msgbuf->buf[len - 2] != '\n') {
+ /*
+ * Buffer ends with a single newline. Add another
+ * so that there is an empty line between the message
+ * body and the sob.
+ */
+ append_newlines = "\n";
+ } /* else, the buffer already ends with two newlines. */
+
+ if (append_newlines)
+ strbuf_splice(msgbuf, msgbuf->len - ignore_footer, 0,
+ append_newlines, strlen(append_newlines));
+ }
+
+ if (has_footer != 3 && (!no_dup_sob || has_footer != 2))
+ strbuf_splice(msgbuf, msgbuf->len - ignore_footer, 0,
+ sob.buf, sob.len);
+
+ strbuf_release(&sob);
+}
+
void show_log(struct rev_info *opt)
{
struct strbuf msgbuf = STRBUF_INIT;
diff --git a/log-tree.h b/log-tree.h
index d6ecd4d..1039e49 100644
--- a/log-tree.h
+++ b/log-tree.h
@@ -25,4 +25,7 @@ void load_ref_decorations(int flags);
void fmt_output_commit(struct strbuf *, struct commit *, struct rev_info *);
void fmt_output_subject(struct strbuf *, const char *subject, struct rev_info *);
+void append_cherrypick(struct strbuf *msgbuf, struct object *obj);
+void append_signoff(struct strbuf *msgbuf, int ignore_footer, unsigned flag);
+
#endif
diff --git a/sequencer.c b/sequencer.c
index ab6f8a7..e92e039 100644
--- a/sequencer.c
+++ b/sequencer.c
@@ -14,94 +14,10 @@
#include "merge-recursive.h"
#include "refs.h"
#include "argv-array.h"
+#include "log-tree.h"
#define GIT_REFLOG_ACTION "GIT_REFLOG_ACTION"
-const char sign_off_header[] = "Signed-off-by: ";
-static const char cherry_picked_prefix[] = "(cherry picked from commit ";
-
-static int is_rfc2822_line(const char *buf, int len)
-{
- int i;
-
- for (i = 0; i < len; i++) {
- int ch = buf[i];
- if (ch == ':')
- return 1;
- if (!isalnum(ch) && ch != '-')
- break;
- }
-
- return 0;
-}
-
-static int is_cherry_picked_from_line(const char *buf, int len)
-{
- /*
- * We only care that it looks roughly like (cherry picked from ...)
- */
- return len > strlen(cherry_picked_prefix) + 1 &&
- !prefixcmp(buf, cherry_picked_prefix) && buf[len - 1] == ')';
-}
-
-/*
- * Returns 0 for non-conforming footer
- * Returns 1 for conforming footer
- * Returns 2 when sob exists within conforming footer
- * Returns 3 when sob exists within conforming footer as last entry
- */
-static int has_conforming_footer(struct strbuf *sb, struct strbuf *sob,
- int ignore_footer)
-{
- char prev;
- int i, k;
- int len = sb->len - ignore_footer;
- const char *buf = sb->buf;
- int found_sob = 0;
-
- /* footer must end with newline */
- if (!len || buf[len - 1] != '\n')
- return 0;
-
- prev = '\0';
- for (i = len - 1; i > 0; i--) {
- char ch = buf[i];
- if (prev == '\n' && ch == '\n') /* paragraph break */
- break;
- prev = ch;
- }
-
- /* require at least one blank line */
- if (prev != '\n' || buf[i] != '\n')
- return 0;
-
- /* advance to start of last paragraph */
- while (i < len - 1 && buf[i] == '\n')
- i++;
-
- for (; i < len; i = k) {
- int found_rfc2822;
-
- for (k = i; k < len && buf[k] != '\n'; k++)
- ; /* do nothing */
- k++;
-
- found_rfc2822 = is_rfc2822_line(buf + i, k - i - 1);
- if (found_rfc2822 && sob &&
- !strncmp(buf + i, sob->buf, sob->len))
- found_sob = k;
-
- if (!(found_rfc2822 ||
- is_cherry_picked_from_line(buf + i, k - i - 1)))
- return 0;
- }
- if (found_sob == i)
- return 3;
- if (found_sob)
- return 2;
- return 1;
-}
-
static void remove_sequencer_state(void)
{
struct strbuf seq_dir = STRBUF_INIT;
@@ -578,13 +494,8 @@ static int do_pick_commit(struct commit *commit, struct replay_opts *opts)
strbuf_addstr(&msgbuf, p);
}
- if (opts->record_origin) {
- if (!has_conforming_footer(&msgbuf, NULL, 0))
- strbuf_addch(&msgbuf, '\n');
- strbuf_addstr(&msgbuf, cherry_picked_prefix);
- strbuf_addstr(&msgbuf, sha1_to_hex(commit->object.sha1));
- strbuf_addstr(&msgbuf, ")\n");
- }
+ if (opts->record_origin)
+ append_cherrypick(&msgbuf, &commit->object);
}
if (!opts->strategy || !strcmp(opts->strategy, "recursive") || opts->action == REPLAY_REVERT) {
@@ -1123,68 +1034,3 @@ int sequencer_pick_revisions(struct replay_opts *opts)
save_opts(opts);
return pick_commits(todo_list, opts);
}
-
-void append_signoff(struct strbuf *msgbuf, int ignore_footer, unsigned flag)
-{
- unsigned no_dup_sob = flag & APPEND_SIGNOFF_DEDUP;
- struct strbuf sob = STRBUF_INIT;
- int has_footer;
-
- strbuf_addstr(&sob, sign_off_header);
- strbuf_addstr(&sob, fmt_name(getenv("GIT_COMMITTER_NAME"),
- getenv("GIT_COMMITTER_EMAIL")));
- strbuf_addch(&sob, '\n');
-
- /*
- * If the whole message buffer is equal to the sob, pretend that we
- * found a conforming footer with a matching sob
- */
- if (msgbuf->len - ignore_footer == sob.len &&
- !strncmp(msgbuf->buf, sob.buf, sob.len))
- has_footer = 3;
- else
- has_footer = has_conforming_footer(msgbuf, &sob, ignore_footer);
-
- if (!has_footer) {
- const char *append_newlines = NULL;
- size_t len = msgbuf->len - ignore_footer;
-
- if (!len) {
- /*
- * The buffer is completely empty. Leave foom for
- * the title and body to be filled in by the user.
- */
- append_newlines = "\n\n";
- } else if (msgbuf->buf[len - 1] != '\n') {
- /*
- * Incomplete line. Complete the line and add a
- * blank one so that there is an empty line between
- * the message body and the sob.
- */
- append_newlines = "\n\n";
- } else if (len == 1) {
- /*
- * Buffer contains a single newline. Add another
- * so that we leave room for the title and body.
- */
- append_newlines = "\n";
- } else if (msgbuf->buf[len - 2] != '\n') {
- /*
- * Buffer ends with a single newline. Add another
- * so that there is an empty line between the message
- * body and the sob.
- */
- append_newlines = "\n";
- } /* else, the buffer already ends with two newlines. */
-
- if (append_newlines)
- strbuf_splice(msgbuf, msgbuf->len - ignore_footer, 0,
- append_newlines, strlen(append_newlines));
- }
-
- if (has_footer != 3 && (!no_dup_sob || has_footer != 2))
- strbuf_splice(msgbuf, msgbuf->len - ignore_footer, 0,
- sob.buf, sob.len);
-
- strbuf_release(&sob);
-}
diff --git a/sequencer.h b/sequencer.h
index 1fc22dc..c341918 100644
--- a/sequencer.h
+++ b/sequencer.h
@@ -6,8 +6,6 @@
#define SEQ_TODO_FILE "sequencer/todo"
#define SEQ_OPTS_FILE "sequencer/opts"
-#define APPEND_SIGNOFF_DEDUP (1u << 0)
-
enum replay_action {
REPLAY_REVERT,
REPLAY_PICK
@@ -50,6 +48,4 @@ int sequencer_pick_revisions(struct replay_opts *opts);
extern const char sign_off_header[];
-void append_signoff(struct strbuf *msgbuf, int ignore_footer, unsigned flag);
-
#endif
--
1.8.3.698.g079b096
next prev parent reply other threads:[~2013-06-07 22:18 UTC|newest]
Thread overview: 82+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-07 22:16 [PATCH 0/2] Move sequencer Felipe Contreras
2013-06-07 22:16 ` Felipe Contreras [this message]
2013-06-07 22:16 ` [PATCH 2/2] Move sequencer to builtin Felipe Contreras
2013-06-08 2:35 ` Duy Nguyen
2013-06-08 10:14 ` Felipe Contreras
2013-06-08 11:42 ` Duy Nguyen
2013-06-08 12:25 ` Felipe Contreras
2013-06-08 12:34 ` Duy Nguyen
2013-06-08 12:55 ` Ramkumar Ramachandra
2013-06-08 13:15 ` Duy Nguyen
2013-06-08 13:32 ` Felipe Contreras
2013-06-08 13:34 ` Ramkumar Ramachandra
2013-06-08 14:10 ` Felipe Contreras
2013-06-08 14:10 ` Duy Nguyen
2013-06-08 14:20 ` Felipe Contreras
2013-06-09 4:34 ` Jeff King
2013-06-09 9:58 ` Ramkumar Ramachandra
2013-06-09 17:55 ` Jeff King
2013-06-09 18:06 ` Ramkumar Ramachandra
2013-06-09 18:11 ` Felipe Contreras
2013-06-09 18:22 ` Jeff King
2013-06-09 18:29 ` Felipe Contreras
2013-06-09 18:44 ` Ramkumar Ramachandra
2013-06-09 18:49 ` Jeff King
2013-06-09 18:54 ` Felipe Contreras
2013-06-09 18:07 ` Felipe Contreras
2013-06-09 12:09 ` Felipe Contreras
2013-06-08 13:28 ` Felipe Contreras
2013-06-08 16:49 ` Jonathan Nieder
2013-06-08 17:06 ` Felipe Contreras
2013-06-08 17:34 ` Jonathan Nieder
2013-06-08 17:44 ` Felipe Contreras
2013-06-08 19:15 ` Felipe Contreras
2013-06-09 1:40 ` Jonathan Nieder
2013-06-09 2:17 ` Felipe Contreras
2013-06-09 3:21 ` Jonathan Nieder
2013-06-09 3:34 ` Felipe Contreras
2013-06-09 5:26 ` Jeff King
2013-06-09 12:15 ` Felipe Contreras
2013-06-09 17:40 ` Jeff King
2013-06-09 18:01 ` Felipe Contreras
2013-06-09 18:10 ` Jeff King
2013-06-09 18:16 ` Felipe Contreras
2013-06-09 19:11 ` Johan Herland
2013-06-09 19:29 ` Felipe Contreras
2013-06-09 21:42 ` Michael Haggerty
2013-06-09 23:40 ` Stefano Lattarini
2013-06-10 5:15 ` Felipe Contreras
2013-06-10 9:05 ` Bad attitudes and problems in the Git community (was: Re: [PATCH 2/2] Move sequencer to builtin) Stefano Lattarini
2013-06-10 16:58 ` Felipe Contreras
2013-06-10 18:11 ` Martin von Zweigbergk
2013-06-10 18:33 ` Martin Langhoff
2013-06-10 18:40 ` Martin von Zweigbergk
2013-06-10 21:34 ` Felipe Contreras
2013-06-10 5:12 ` [PATCH 2/2] Move sequencer to builtin Felipe Contreras
2013-06-11 9:18 ` Andres Freund
2013-06-11 9:29 ` Felipe Contreras
2013-06-20 21:11 ` Thiago Farina
2013-06-09 17:53 ` Thomas Rast
2013-06-09 18:03 ` Felipe Contreras
2013-06-09 12:48 ` Ramkumar Ramachandra
2013-06-09 13:08 ` Felipe Contreras
2013-06-09 18:04 ` Jeff King
2013-06-09 18:32 ` Ramkumar Ramachandra
2013-06-09 18:45 ` Jeff King
2013-06-09 19:57 ` Jonathan Nieder
2013-06-09 20:07 ` Felipe Contreras
2013-06-09 20:34 ` Ramkumar Ramachandra
2013-06-09 21:39 ` Junio C Hamano
2013-06-10 5:06 ` Felipe Contreras
2013-06-10 8:32 ` Junio C Hamano
2013-06-10 16:53 ` Felipe Contreras
2013-06-10 16:55 ` Felipe Contreras
2013-06-10 17:34 ` Matthieu Moy
2013-06-10 18:09 ` Ramkumar Ramachandra
2013-06-10 21:43 ` Felipe Contreras
2013-06-09 18:48 ` Felipe Contreras
2013-06-09 19:25 ` Thomas Rast
2013-06-09 19:54 ` Ramkumar Ramachandra
2013-06-09 20:02 ` Felipe Contreras
2013-06-08 3:35 ` [PATCH 0/2] Move sequencer Ramkumar Ramachandra
2013-06-08 10:26 ` Felipe Contreras
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1370643409-3431-2-git-send-email-felipe.contreras@gmail.com \
--to=felipe.contreras@gmail.com \
--cc=artagnon@gmail.com \
--cc=drafnel@gmail.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=jrnieder@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).