fix "builtin-*" references to be "builtin/*"
Documentation and some comments still refer to files in builtin/ as 'builtin-*.[cho]'. Update these to show the correct location. Signed-off-by: Phil Hord <hordp@cisco.com> Reviewed-by: Jonathan Nieder <jrnieder@gmail.com> Assisted-by: Junio C Hamano <gitster@pobox.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>maint
							parent
							
								
									04a74b6cfa
								
							
						
					
					
						commit
						09b7e2204a
					
				|  | @ -108,9 +108,9 @@ Examples | |||
| 	in the "release" branch, along with the list of paths | ||||
| 	each commit modifies. | ||||
|  | ||||
| `git log --follow builtin-rev-list.c`:: | ||||
| `git log --follow builtin/rev-list.c`:: | ||||
|  | ||||
| 	Shows the commits that changed builtin-rev-list.c, including | ||||
| 	Shows the commits that changed builtin/rev-list.c, including | ||||
| 	those commits that occurred before the file was given its | ||||
| 	present name. | ||||
|  | ||||
|  |  | |||
|  | @ -39,7 +39,7 @@ where options is the bitwise-or of: | |||
| 	on bare repositories. | ||||
| 	This only makes sense when `RUN_SETUP` is also set. | ||||
|  | ||||
| . Add `builtin-foo.o` to `BUILTIN_OBJS` in `Makefile`. | ||||
| . Add `builtin/foo.o` to `BUILTIN_OBJS` in `Makefile`. | ||||
|  | ||||
| Additionally, if `foo` is a new command, there are 3 more things to do: | ||||
|  | ||||
|  |  | |||
|  | @ -269,10 +269,10 @@ Examples | |||
| -------- | ||||
|  | ||||
| See `test-parse-options.c` and | ||||
| `builtin-add.c`, | ||||
| `builtin-clone.c`, | ||||
| `builtin-commit.c`, | ||||
| `builtin-fetch.c`, | ||||
| `builtin-fsck.c`, | ||||
| `builtin-rm.c` | ||||
| `builtin/add.c`, | ||||
| `builtin/clone.c`, | ||||
| `builtin/commit.c`, | ||||
| `builtin/fetch.c`, | ||||
| `builtin/fsck.c`, | ||||
| `builtin/rm.c` | ||||
| for real-world examples. | ||||
|  |  | |||
|  | @ -4254,15 +4254,16 @@ no longer need to call `setup_pager()` directly). | |||
| Nowadays, `git log` is a builtin, which means that it is _contained_ in the | ||||
| command `git`.  The source side of a builtin is | ||||
|  | ||||
| - a function called `cmd_<bla>`, typically defined in `builtin-<bla>.c`, | ||||
|   and declared in `builtin.h`, | ||||
| - a function called `cmd_<bla>`, typically defined in `builtin/<bla.c>` | ||||
|   (note that older versions of Git used to have it in `builtin-<bla>.c` | ||||
|   instead), and declared in `builtin.h`. | ||||
|  | ||||
| - an entry in the `commands[]` array in `git.c`, and | ||||
|  | ||||
| - an entry in `BUILTIN_OBJECTS` in the `Makefile`. | ||||
|  | ||||
| Sometimes, more than one builtin is contained in one source file.  For | ||||
| example, `cmd_whatchanged()` and `cmd_log()` both reside in `builtin-log.c`, | ||||
| example, `cmd_whatchanged()` and `cmd_log()` both reside in `builtin/log.c`, | ||||
| since they share quite a bit of code.  In that case, the commands which are | ||||
| _not_ named like the `.c` file in which they live have to be listed in | ||||
| `BUILT_INS` in the `Makefile`. | ||||
|  | @ -4285,10 +4286,10 @@ For the sake of clarity, let's stay with `git cat-file`, because it | |||
| - is plumbing, and | ||||
|  | ||||
| - was around even in the initial commit (it literally went only through | ||||
|   some 20 revisions as `cat-file.c`, was renamed to `builtin-cat-file.c` | ||||
|   some 20 revisions as `cat-file.c`, was renamed to `builtin/cat-file.c` | ||||
|   when made a builtin, and then saw less than 10 versions). | ||||
|  | ||||
| So, look into `builtin-cat-file.c`, search for `cmd_cat_file()` and look what | ||||
| So, look into `builtin/cat-file.c`, search for `cmd_cat_file()` and look what | ||||
| it does. | ||||
|  | ||||
| ------------------------------------------------------------------ | ||||
|  | @ -4364,7 +4365,7 @@ Another example: Find out what to do in order to make some script a | |||
| builtin: | ||||
|  | ||||
| ------------------------------------------------- | ||||
| $ git log --no-merges --diff-filter=A builtin-*.c | ||||
| $ git log --no-merges --diff-filter=A builtin/*.c | ||||
| ------------------------------------------------- | ||||
|  | ||||
| You see, Git is actually the best tool to find out about the source of Git | ||||
|  |  | |||
|  | @ -1,6 +1,4 @@ | |||
| /* | ||||
|  * builtin-help.c | ||||
|  * | ||||
|  * Builtin help command | ||||
|  */ | ||||
| #include "cache.h" | ||||
|  |  | |||
|  | @ -4,7 +4,7 @@ | |||
|  * Copyright (c) 2010 Johan Herland <johan@herland.net> | ||||
|  * | ||||
|  * Based on git-notes.sh by Johannes Schindelin, | ||||
|  * and builtin-tag.c by Kristian Høgsberg and Carlos Rica. | ||||
|  * and builtin/tag.c by Kristian Høgsberg and Carlos Rica. | ||||
|  */ | ||||
|  | ||||
| #include "cache.h" | ||||
|  |  | |||
|  | @ -3,7 +3,7 @@ | |||
|  * | ||||
|  * Copyright (c) 2008 Christian Couder <chriscool@tuxfamily.org> | ||||
|  * | ||||
|  * Based on builtin-tag.c by Kristian Høgsberg <krh@redhat.com> | ||||
|  * Based on builtin/tag.c by Kristian Høgsberg <krh@redhat.com> | ||||
|  * and Carlos Rica <jasampler@gmail.com> that was itself based on | ||||
|  * git-tag.sh and mktag.c by Linus Torvalds. | ||||
|  */ | ||||
|  |  | |||
|  | @ -1153,7 +1153,7 @@ int transport_fetch_refs(struct transport *transport, struct ref *refs) | |||
| 		 * then local and remote refs are likely to still be equal. | ||||
| 		 * Just feed them all to the fetch method in that case. | ||||
| 		 * This condition shouldn't be met in a non-deepening fetch | ||||
| 		 * (see builtin-fetch.c:quickfetch()). | ||||
| 		 * (see builtin/fetch.c:quickfetch()). | ||||
| 		 */ | ||||
| 		heads = xmalloc(nr_refs * sizeof(*heads)); | ||||
| 		for (rm = refs; rm; rm = rm->next) | ||||
|  |  | |||
|  | @ -162,7 +162,7 @@ int transport_connect(struct transport *transport, const char *name, | |||
| int transport_helper_init(struct transport *transport, const char *name); | ||||
| int bidirectional_transfer_loop(int input, int output); | ||||
|  | ||||
| /* common methods used by transport.c and builtin-send-pack.c */ | ||||
| /* common methods used by transport.c and builtin/send-pack.c */ | ||||
| void transport_verify_remote_names(int nr_heads, const char **heads); | ||||
|  | ||||
| void transport_update_tracking_ref(struct remote *remote, struct ref *ref, int verbose); | ||||
|  |  | |||
		Loading…
	
		Reference in New Issue
	
	 Phil Hord
						Phil Hord