This changes the way how cat_file selects the source of the file. It
accepts an optional parameter which is either -i or -w and will react on this
instead of the branch name. tg-patch is updated accordingly and can now
accepts the current branch name as argument with -i or -w given.
cat_file was also broken for the worktree case when we are not in the top level.
Also, tg-patch allowed to be on the top-base branch, but -i and -w doesn't
make sense there too.
Signed-off-by: Bert Wesarg <bert.wesarg@googlemail.com>
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
# check for repetitions of deps
depdir="$(get_temp tg-depdir -d)" ||
die "Can't check for multiple occurrences of deps"
# check for repetitions of deps
depdir="$(get_temp tg-depdir -d)" ||
die "Can't check for multiple occurrences of deps"
-cat_file "(i):.topdeps" |
+cat_file "$head_:.topdeps" -i |
while read dep; do
[ ! -d "$depdir/$dep" ] ||
die "Multiple occurrences of the same dep: $dep"
while read dep; do
[ ! -d "$depdir/$dep" ] ||
die "Multiple occurrences of the same dep: $dep"
diff_opts=
diff_committed_only=yes # will be unset for index/worktree
diff_opts=
diff_committed_only=yes # will be unset for index/worktree
arg="$1"; shift
case "$arg" in
-i)
arg="$1"; shift
case "$arg" in
-i)
+ [ -z "$head_from" ] || die "-i and -w are mutually exclusive"
+ head_from=-i
diff_opts="$diff_opts --cached";
diff_committed_only=;;
-w)
diff_opts="$diff_opts --cached";
diff_committed_only=;;
-w)
+ [ -z "$head_from" ] || die "-i and -w are mutually exclusive"
+ head_from=-w
diff_committed_only=;;
-*)
echo "Usage: tg [...] patch [-i | -w] [NAME]" >&2
diff_committed_only=;;
-*)
echo "Usage: tg [...] patch [-i | -w] [NAME]" >&2
+head="$(git symbolic-ref HEAD)"
+head="${head#refs/heads/}"
-[ -n "$name" -a -z "$diff_committed_only" ] &&
- die "-i/-w are mutually exclusive with NAME"
-
-[ -n "$name" ] || name="$(git symbolic-ref HEAD | sed 's#^refs/\(heads\|top-bases\)/##')"
+[ -n "$name" ] ||
+ name="$head"
base_rev="$(git rev-parse --short --verify "refs/top-bases/$name" 2>/dev/null)" ||
die "not a TopGit-controlled branch"
base_rev="$(git rev-parse --short --verify "refs/top-bases/$name" 2>/dev/null)" ||
die "not a TopGit-controlled branch"
-# if not index/worktree, topic is current branch
-[ -z "$topic" ] && topic="$name"
+if [ -n "$head_from" ] && [ "$name" != "$head" ]; then
+ die "$head_from makes only sense for the current branch"
+fi
-cat_file "$topic:.topmsg"
+cat_file "$name:.topmsg" $head_from
echo
[ -n "$(git grep $diff_opts '^[-]--' ${diff_committed_only:+"$name"} -- ".topmsg")" ] || echo '---'
echo
[ -n "$(git grep $diff_opts '^[-]--' ${diff_committed_only:+"$name"} -- ".topmsg")" ] || echo '---'
-echo "tg: ($base_rev..) $name (depends on: $(cat_file "$topic:.topdeps" | paste -s -d' '))"
+echo "tg: ($base_rev..) $name (depends on: $(cat_file "$name:.topdeps" $head_from | paste -s -d' '))"
branch_contains "$name" "$base_rev" ||
echo "tg: The patch is out-of-date wrt. the base! Run \`$tg update\`."
branch_contains "$name" "$base_rev" ||
echo "tg: The patch is out-of-date wrt. the base! Run \`$tg update\`."
-# cat_file "topic:file"
-# Like `git cat-file blob $1`, but topics '(i)' and '(w)' means index and worktree
+# cat_file TOPIC:PATH [FROM]
+# cat the file PATH from branch TOPIC when FROM is empty.
+# FROM can be -i or -w, than the file will be from the index or worktree,
+# respectively. The caller should than ensure that HEAD is TOPIC, to make sense.
- arg="$1"
- case "$arg" in
- '(w):'*)
- cat "${arg#(w):}"
+ path="$1"
+ case "${2-}" in
+ -w)
+ cat "$root_dir/${path#*:}"
# ':file' means cat from index
# ':file' means cat from index
- git cat-file blob "${arg#(i)}"
+ git cat-file blob ":${path#*:}"
+ ;;
+ '')
+ git cat-file blob "$path"
- git cat-file blob "$arg"
+ die "Wrong argument to cat_file: '$2'"