Browse Source

Merge branch 'jc/gitlink' into next

* jc/gitlink:
  write-tree: --prefix=<path>
  read-tree: --prefix=<path>/ option.
maint
Junio C Hamano 19 years ago
parent
commit
b0121fb3f2
  1. 11
      Documentation/git-read-tree.txt
  2. 8
      Documentation/git-write-tree.txt
  3. 26
      cache-tree.c
  4. 2
      cache-tree.h
  5. 68
      read-tree.c
  6. 14
      t/t0000-basic.sh
  7. 23
      write-tree.c

11
Documentation/git-read-tree.txt

@ -8,7 +8,7 @@ git-read-tree - Reads tree information into the index


SYNOPSIS SYNOPSIS
-------- --------
'git-read-tree' (<tree-ish> | [[-m [--aggressive]| --reset] [-u | -i]] <tree-ish1> [<tree-ish2> [<tree-ish3>]]) 'git-read-tree' (<tree-ish> | [[-m [--aggressive] | --reset | --prefix=<prefix>] [-u | -i]] <tree-ish1> [<tree-ish2> [<tree-ish3>]])




DESCRIPTION DESCRIPTION
@ -63,6 +63,15 @@ OPTIONS
* when both sides adds a path identically. The resolution * when both sides adds a path identically. The resolution
is to add that path. is to add that path.


--prefix=<prefix>/::
Keep the current index contents, and read the contents
of named tree-ish under directory at `<prefix>`. The
original index file cannot have anything at the path
`<prefix>` itself, and have nothing in `<prefix>/`
directory. Note that the `<prefix>/` value must end
with a slash.


<tree-ish#>:: <tree-ish#>::
The id of the tree object(s) to be read/merged. The id of the tree object(s) to be read/merged.



8
Documentation/git-write-tree.txt

@ -8,7 +8,7 @@ git-write-tree - Creates a tree object from the current index


SYNOPSIS SYNOPSIS
-------- --------
'git-write-tree' [--missing-ok] 'git-write-tree' [--missing-ok] [--prefix=<prefix>/]


DESCRIPTION DESCRIPTION
----------- -----------
@ -30,6 +30,12 @@ OPTIONS
directory exist in the object database. This option disables this directory exist in the object database. This option disables this
check. check.


--prefix=<prefix>/::
Writes a tree object that represents a subdirectory
`<prefix>`. This can be used to write the tree object
for a subproject that is in the named subdirectory.


Author Author
------ ------
Written by Linus Torvalds <torvalds@osdl.org> Written by Linus Torvalds <torvalds@osdl.org>

26
cache-tree.c

@ -529,3 +529,29 @@ struct cache_tree *cache_tree_read(const char *buffer, unsigned long size)
return NULL; /* not the whole tree */ return NULL; /* not the whole tree */
return read_one(&buffer, &size); return read_one(&buffer, &size);
} }

struct cache_tree *cache_tree_find(struct cache_tree *it, const char *path)
{
while (*path) {
const char *slash;
struct cache_tree_sub *sub;

slash = strchr(path, '/');
if (!slash)
slash = path + strlen(path);
/* between path and slash is the name of the
* subtree to look for.
*/
sub = find_subtree(it, path, slash - path, 0);
if (!sub)
return NULL;
it = sub->cache_tree;
if (slash)
while (*slash && *slash == '/')
slash++;
if (!slash || !*slash)
return it; /* prefix ended with slashes */
path = slash;
}
return it;
}

2
cache-tree.h

@ -28,4 +28,6 @@ struct cache_tree *cache_tree_read(const char *buffer, unsigned long size);
int cache_tree_fully_valid(struct cache_tree *); int cache_tree_fully_valid(struct cache_tree *);
int cache_tree_update(struct cache_tree *, struct cache_entry **, int, int, int); int cache_tree_update(struct cache_tree *, struct cache_entry **, int, int, int);


struct cache_tree *cache_tree_find(struct cache_tree *, const char *);

#endif #endif

68
read-tree.c

@ -21,6 +21,7 @@ static int trivial_merges_only = 0;
static int aggressive = 0; static int aggressive = 0;
static int verbose_update = 0; static int verbose_update = 0;
static volatile int progress_update = 0; static volatile int progress_update = 0;
static const char *prefix = NULL;


static int head_idx = -1; static int head_idx = -1;
static int merge_size = 0; static int merge_size = 0;
@ -369,7 +370,8 @@ static int unpack_trees(merge_fn_t fn)
posns[i] = ((struct tree *) posn->item)->entries; posns[i] = ((struct tree *) posn->item)->entries;
posn = posn->next; posn = posn->next;
} }
if (unpack_trees_rec(posns, len, "", fn, &indpos)) if (unpack_trees_rec(posns, len, prefix ? prefix : "",
fn, &indpos))
return -1; return -1;
} }


@ -679,6 +681,28 @@ static int twoway_merge(struct cache_entry **src)
return deleted_entry(oldtree, current); return deleted_entry(oldtree, current);
} }


/*
* Bind merge.
*
* Keep the index entries at stage0, collapse stage1 but make sure
* stage0 does not have anything in prefix.
*/
static int bind_merge(struct cache_entry **src)
{
struct cache_entry *old = src[0];
struct cache_entry *a = src[1];

if (merge_size != 1)
return error("Cannot do a bind merge of %d trees\n",
merge_size);
if (!a)
return merged_entry(old, NULL);
if (old)
die("Entry '%s' overlaps. Cannot bind.", a->name);

return merged_entry(a, NULL);
}

/* /*
* One-way merge. * One-way merge.
* *
@ -760,7 +784,7 @@ static void prime_cache_tree(void)


} }


static const char read_tree_usage[] = "git-read-tree (<sha> | -m [--aggressive] [-u | -i] <sha1> [<sha2> [<sha3>]])"; static const char read_tree_usage[] = "git-read-tree (<sha> | [[-m [--aggressive] | --reset | --prefix=<prefix>] [-u | -i]] <sha1> [<sha2> [<sha3>]])";


static struct cache_file cache_file; static struct cache_file cache_file;


@ -805,9 +829,24 @@ int main(int argc, char **argv)
continue; continue;
} }


/* "--prefix=<subdirectory>/" means keep the current index
* entries and put the entries from the tree under the
* given subdirectory.
*/
if (!strncmp(arg, "--prefix=", 9)) {
if (stage || merge || prefix)
usage(read_tree_usage);
prefix = arg + 9;
merge = 1;
stage = 1;
if (read_cache_unmerged())
die("you need to resolve your current index first");
continue;
}

/* This differs from "-m" in that we'll silently ignore unmerged entries */ /* This differs from "-m" in that we'll silently ignore unmerged entries */
if (!strcmp(arg, "--reset")) { if (!strcmp(arg, "--reset")) {
if (stage || merge) if (stage || merge || prefix)
usage(read_tree_usage); usage(read_tree_usage);
reset = 1; reset = 1;
merge = 1; merge = 1;
@ -828,7 +867,7 @@ int main(int argc, char **argv)


/* "-m" stands for "merge", meaning we start in stage 1 */ /* "-m" stands for "merge", meaning we start in stage 1 */
if (!strcmp(arg, "-m")) { if (!strcmp(arg, "-m")) {
if (stage || merge) if (stage || merge || prefix)
usage(read_tree_usage); usage(read_tree_usage);
if (read_cache_unmerged()) if (read_cache_unmerged())
die("you need to resolve your current index first"); die("you need to resolve your current index first");
@ -850,12 +889,31 @@ int main(int argc, char **argv)
if ((update||index_only) && !merge) if ((update||index_only) && !merge)
usage(read_tree_usage); usage(read_tree_usage);


if (prefix) {
int pfxlen = strlen(prefix);
int pos;
if (prefix[pfxlen-1] != '/')
die("prefix must end with /");
if (stage != 2)
die("binding merge takes only one tree");
pos = cache_name_pos(prefix, pfxlen);
if (0 <= pos)
die("corrupt index file");
pos = -pos-1;
if (pos < active_nr &&
!strncmp(active_cache[pos]->name, prefix, pfxlen))
die("subdirectory '%s' already exists.", prefix);
pos = cache_name_pos(prefix, pfxlen-1);
if (0 <= pos)
die("file '%.*s' already exists.", pfxlen-1, prefix);
}

if (merge) { if (merge) {
if (stage < 2) if (stage < 2)
die("just how do you expect me to merge %d trees?", stage-1); die("just how do you expect me to merge %d trees?", stage-1);
switch (stage - 1) { switch (stage - 1) {
case 1: case 1:
fn = oneway_merge; fn = prefix ? bind_merge : oneway_merge;
break; break;
case 2: case 2:
fn = twoway_merge; fn = twoway_merge;

14
t/t0000-basic.sh

@ -195,6 +195,20 @@ test_expect_success \
'git-ls-tree -r output for a known tree.' \ 'git-ls-tree -r output for a known tree.' \
'diff current expected' 'diff current expected'


test_expect_success \
'writing partial tree out with git-write-tree --prefix.' \
'ptree=$(git-write-tree --prefix=path3)'
test_expect_success \
'validate object ID for a known tree.' \
'test "$ptree" = 21ae8269cacbe57ae09138dcc3a2887f904d02b3'

test_expect_success \
'writing partial tree out with git-write-tree --prefix.' \
'ptree=$(git-write-tree --prefix=path3/subp3)'
test_expect_success \
'validate object ID for a known tree.' \
'test "$ptree" = 3c5e5399f3a333eddecce7a9b9465b63f65f51e2'

################################################################ ################################################################
rm .git/index rm .git/index
test_expect_success \ test_expect_success \

23
write-tree.c

@ -8,8 +8,10 @@
#include "cache-tree.h" #include "cache-tree.h"


static int missing_ok = 0; static int missing_ok = 0;
static char *prefix = NULL;


static const char write_tree_usage[] = "git-write-tree [--missing-ok]"; static const char write_tree_usage[] =
"git-write-tree [--missing-ok] [--prefix=<prefix>/]";


static struct cache_file cache_file; static struct cache_file cache_file;


@ -21,13 +23,18 @@ int main(int argc, char **argv)


newfd = hold_index_file_for_update(&cache_file, get_index_file()); newfd = hold_index_file_for_update(&cache_file, get_index_file());
entries = read_cache(); entries = read_cache();
if (argc == 2) {
if (!strcmp(argv[1], "--missing-ok")) while (1 < argc) {
char *arg = argv[1];
if (!strcmp(arg, "--missing-ok"))
missing_ok = 1; missing_ok = 1;
else if (!strncmp(arg, "--prefix=", 9))
prefix = arg + 9;
else else
die(write_tree_usage); die(write_tree_usage);
argc--; argv++;
} }

if (argc > 2) if (argc > 2)
die("too many options"); die("too many options");


@ -54,6 +61,12 @@ int main(int argc, char **argv)
* performance penalty and not a big deal. * performance penalty and not a big deal.
*/ */
} }
printf("%s\n", sha1_to_hex(active_cache_tree->sha1)); if (prefix) {
struct cache_tree *subtree =
cache_tree_find(active_cache_tree, prefix);
printf("%s\n", sha1_to_hex(subtree->sha1));
}
else
printf("%s\n", sha1_to_hex(active_cache_tree->sha1));
return 0; return 0;
} }

Loading…
Cancel
Save