Use the topological order when exporting explicitly listed branches.

When exporting explicitly listed branches (tg export -b), the branch
dependencies were not taken into account.

Signed-off-by: martin f. krafft <madduck@debian.org>
Tested-by: martin f. krafft <madduck@debian.org>

diff --git a/.topmsg b/.topmsg
index 2da25051e958746e69fd2ba23f05ec07eeed784a..c3b92472c1c8a1777291d38342887c525adeee7e 100644 (file)
--- a/.topmsg
+++ b/.topmsg
@@ -6,4 +6,8 @@ not, when really -b should just default to the name of the current branch.
 
 -b could also make sense with the collapse driver.
 
+Debian bug: #506284
+Credits: Niko Tyni <ntyni@debian.org> for spotting the bug and sending an
+ initial fix.
+
 Signed-off-by: martin f. krafft <madduck@debian.org>

diff --git a/tg-export.sh b/tg-export.sh
index 52af88dfd14e891627b145f63702b1212155a82c..037b9910517fc1ffd1fbb79150b88a58d229e16e 100644 (file)
--- a/tg-export.sh
+++ b/tg-export.sh
@@ -181,9 +181,9 @@ if [ -z "$branches" ]; then
        recurse_deps driver "$name"
        (_ret=0; _dep="$name"; _name=""; _dep_is_tgish=1; driver)
 else
-       echo "$branches" | tr ',' '\n' | while read _dep; do
-               _dep_is_tgish=1
-               $driver
+       echo "$branches" | tr ',' '\n' | while read name; do
+               recurse_deps driver "$name"
+               (_ret=0; _dep="$name"; _name=""; _dep_is_tgish=1; driver)
        done
        name="$(echo "$branches" | sed 's/.*,//')"
 fi