{"thread":{"id":"34203","subject":"[PATCHv2] fix \"builtin-*\" references to be \"builtin/*\"","startedAt":"2013-06-18T17:44:58Z","lastAt":"2013-06-18T18:16:12Z","messageCount":2,"participants":["Phil Hord","Jonathan Nieder"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"221275","messageId":"1371577498-7928-1-git-send-email-hordp@cisco.com","threadId":"34203","inReplyTo":null,"subject":"[PATCHv2] fix \"builtin-*\" references to be \"builtin/*\"","fromName":"Phil Hord","fromEmail":"hordp@cisco.com","sentAt":"2013-06-18T17:44:58Z","receivedAt":"2013-06-18T17:44:58Z","isPatch":false,"sender":{"key":"phil.hord@gmail.com","avatar":"https://avatars.githubusercontent.com/u/123908?v=4"},"body":"Documentation and some comments still refer to files in builtin/\nas 'builtin-*.[cho]'. Update these to show the correct location.\n\nSigned-off-by: Phil Hord \nReviewed-by: Jonathan Nieder \nAssisted-by: Junio C Hamano \n---\n Documentation/git-log.txt | 4 ++--\n Documentation/technical/api-builtin.txt | 2 +-\n Documentation/technical/api-parse-options.txt | 12 ++++++------\n Documentation/user-manual.txt | 13 +++++++------\n builtin/help.c | 2 --\n builtin/notes.c | 2 +-\n builtin/replace.c | 2 +-\n transport.c | 2 +-\n transport.h | 2 +-\n 9 files changed, 20 insertions(+), 21 deletions(-)\n\ndiff --git a/Documentation/git-log.txt b/Documentation/git-log.txt\nindex 4687fe8..2ea79ba 100644\n--- a/Documentation/git-log.txt\n+++ b/Documentation/git-log.txt\n@@ -128,9 +128,9 @@ Examples\n \tin the \"release\" branch, along with the list of paths\n \teach commit modifies.\n \n-`git log --follow builtin-rev-list.c`::\n+`git log --follow builtin/rev-list.c`::\n \n-\tShows the commits that changed builtin-rev-list.c, including\n+\tShows the commits that changed builtin/rev-list.c, including\n \tthose commits that occurred before the file was given its\n \tpresent name.\n \ndiff --git a/Documentation/technical/api-builtin.txt b/Documentation/technical/api-builtin.txt\nindex 4a4228b..f3c1357 100644\n--- a/Documentation/technical/api-builtin.txt\n+++ b/Documentation/technical/api-builtin.txt\n@@ -39,7 +39,7 @@ where options is the bitwise-or of:\n \ton bare repositories.\n \tThis only makes sense when `RUN_SETUP` is also set.\n \n-. Add `builtin-foo.o` to `BUILTIN_OBJS` in `Makefile`.\n+. Add `builtin/foo.o` to `BUILTIN_OBJS` in `Makefile`.\n \n Additionally, if `foo` is a new command, there are 3 more things to do:\n \ndiff --git a/Documentation/technical/api-parse-options.txt b/Documentation/technical/api-parse-options.txt\nindex 1317db4..0be2b51 100644\n--- a/Documentation/technical/api-parse-options.txt\n+++ b/Documentation/technical/api-parse-options.txt\n@@ -275,10 +275,10 @@ Examples\n --------\n \n See `test-parse-options.c` and\n-`builtin-add.c`,\n-`builtin-clone.c`,\n-`builtin-commit.c`,\n-`builtin-fetch.c`,\n-`builtin-fsck.c`,\n-`builtin-rm.c`\n+`builtin/add.c`,\n+`builtin/clone.c`,\n+`builtin/commit.c`,\n+`builtin/fetch.c`,\n+`builtin/fsck.c`,\n+`builtin/rm.c`\n for real-world examples.\ndiff --git a/Documentation/user-manual.txt b/Documentation/user-manual.txt\nindex e831cc2..2436124 100644\n--- a/Documentation/user-manual.txt\n+++ b/Documentation/user-manual.txt\n@@ -4256,15 +4256,16 @@ no longer need to call `setup_pager()` directly).\n Nowadays, `git log` is a builtin, which means that it is _contained_ in the\n command `git`. The source side of a builtin is\n \n-- a function called `cmd_`, typically defined in `builtin-.c`,\n- and declared in `builtin.h`,\n+- a function called `cmd_`, typically defined in `builtin/`\n+ (note that older versions of Git used to have it in `builtin-.c`\n+ instead), and declared in `builtin.h`.\n \n - an entry in the `commands[]` array in `git.c`, and\n \n - an entry in `BUILTIN_OBJECTS` in the `Makefile`.\n \n Sometimes, more than one builtin is contained in one source file. For\n-example, `cmd_whatchanged()` and `cmd_log()` both reside in `builtin-log.c`,\n+example, `cmd_whatchanged()` and `cmd_log()` both reside in `builtin/log.c`,\n since they share quite a bit of code. In that case, the commands which are\n _not_ named like the `.c` file in which they live have to be listed in\n `BUILT_INS` in the `Makefile`.\n@@ -4287,10 +4288,10 @@ For the sake of clarity, let's stay with `git cat-file`, because it\n - is plumbing, and\n \n - was around even in the initial commit (it literally went only through\n- some 20 revisions as `cat-file.c`, was renamed to `builtin-cat-file.c`\n+ some 20 revisions as `cat-file.c`, was renamed to `builtin/cat-file.c`\n when made a builtin, and then saw less than 10 versions).\n \n-So, look into `builtin-cat-file.c`, search for `cmd_cat_file()` and look what\n+So, look into `builtin/cat-file.c`, search for `cmd_cat_file()` and look what\n it does.\n \n ------------------------------------------------------------------\n@@ -4366,7 +4367,7 @@ Another example: Find out what to do in order to make some script a\n builtin:\n \n -------------------------------------------------\n-$ git log --no-merges --diff-filter=A builtin-*.c\n+$ git log --no-merges --diff-filter=A builtin/*.c\n -------------------------------------------------\n \n You see, Git is actually the best tool to find out about the source of Git\ndiff --git a/builtin/help.c b/builtin/help.c\nindex 062957f..f1e236b 100644\n--- a/builtin/help.c\n+++ b/builtin/help.c\n@@ -1,6 +1,4 @@\n /*\n- * builtin-help.c\n- *\n * Builtin help command\n */\n #include \"cache.h\"\ndiff --git a/builtin/notes.c b/builtin/notes.c\nindex 57748a6..d9a67d9 100644\n--- a/builtin/notes.c\n+++ b/builtin/notes.c\n@@ -4,7 +4,7 @@\n * Copyright (c) 2010 Johan Herland \n *\n * Based on git-notes.sh by Johannes Schindelin,\n- * and builtin-tag.c by Kristian Høgsberg and Carlos Rica.\n+ * and builtin/tag.c by Kristian Høgsberg and Carlos Rica.\n */\n \n #include \"cache.h\"\ndiff --git a/builtin/replace.c b/builtin/replace.c\nindex 398ccd5..59d3115 100644\n--- a/builtin/replace.c\n+++ b/builtin/replace.c\n@@ -3,7 +3,7 @@\n *\n * Copyright (c) 2008 Christian Couder \n *\n- * Based on builtin-tag.c by Kristian Høgsberg \n+ * Based on builtin/tag.c by Kristian Høgsberg \n * and Carlos Rica that was itself based on\n * git-tag.sh and mktag.c by Linus Torvalds.\n */\ndiff --git a/transport.c b/transport.c\nindex 359a671..e15db98 100644\n--- a/transport.c\n+++ b/transport.c\n@@ -1232,7 +1232,7 @@ int transport_fetch_refs(struct transport *transport, struct ref *refs)\n \t\t * then local and remote refs are likely to still be equal.\n \t\t * Just feed them all to the fetch method in that case.\n \t\t * This condition shouldn't be met in a non-deepening fetch\n-\t\t * (see builtin-fetch.c:quickfetch()).\n+\t\t * (see builtin/fetch.c:quickfetch()).\n \t\t */\n \t\theads = xmalloc(nr_refs * sizeof(*heads));\n \t\tfor (rm = refs; rm; rm = rm->next)\ndiff --git a/transport.h b/transport.h\nindex 4edebc5..ea70ea7 100644\n--- a/transport.h\n+++ b/transport.h\n@@ -170,7 +170,7 @@ int transport_connect(struct transport *transport, const char *name,\n int transport_helper_init(struct transport *transport, const char *name);\n int bidirectional_transfer_loop(int input, int output);\n \n-/* common methods used by transport.c and builtin-send-pack.c */\n+/* common methods used by transport.c and builtin/send-pack.c */\n void transport_verify_remote_names(int nr_heads, const char **heads);\n \n void transport_update_tracking_ref(struct remote *remote, struct ref *ref, int verbose);\n-- \n1.8.3.1.560.gb7ca476.dirty\n"},{"id":"221278","messageId":"20130618181612.GC408@google.com","threadId":"34203","inReplyTo":"1371577498-7928-1-git-send-email-hordp@cisco.com","subject":"Re: [PATCHv2] fix \"builtin-*\" references to be \"builtin/*\"","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2013-06-18T18:16:12Z","receivedAt":"2013-06-18T18:16:12Z","isPatch":false,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Phil Hord wrote:\n\n> Reviewed-by: Jonathan Nieder \n\nYep, still looks good from a quick glance.\n\nThanks for the quick turnaround.\n"}]}