Bug #48808 mysql-test-run.pl --debugger=devenv does not work

Description:
The generated command line is 
devenv path/to/mysqld.exe /debugexe <args>

According to documentation, /debugexe it should be passed before exe, after devenv

devenv /debugexe path/to/mysqld.exe <args>

Strange enough, currently debugger "partially" works even with wrong parameters,
though it mysqld started under debugger that way ignored the first argument of <args>. --defaults-group-prefix being the first argument is not passed, resulting in mysqld running on default port where mtr is not able to communicate with it.

How to repeat:
perl mysql-test-run.pl --debugger=devenv

(you look at the command line arguments passed to devenv using sysinternals
process explorer)

The fix  below works for me.

=== modified file 'mysql-test/mysql-test-run.pl'
--- mysql-test/mysql-test-run.pl        2009-11-09 11:32:48 +0000
+++ mysql-test/mysql-test-run.pl        2009-11-16 15:00:27 +0000
@@ -5149,8 +5149,9 @@
     # vc[express] /debugexe exe arg1 .. argn

     # Add /debugexe and name of the exe before args
+       unshift(@$$args, "$$exe");
     unshift(@$$args, "/debugexe");
-    unshift(@$$args, "$$exe");
+

     # Set exe to debuggername
     $$exe= $debugger;

A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/90810

2856 Bjorn Munch	2009-11-18
      Bug #48808 mysql-test-run.pl --debugger=devenv does not work
      Wrong argument order, fixed

Confirmed on Windows that it fails to run mysqld.exe without the fix, succeeds with the fix.

Pushed to 5.1-mtr, trunk-mtr, next-mr-mtr, 6.0-codebase-mtr

Pushed into 6.0.14-alpha (revid:alik@sun.com-20100220092844-jh07ayojyxn8uh9p) (version source revid:bjorn.munch@sun.com-20091119111729-sazk6qfk8qbwfq0y) (merge vers: 6.0.14-alpha) (pib:16)

Pushed into 5.5.3-m2 (revid:alik@sun.com-20100220092622-wvhh1vfy5tjq4mhu) (version source revid:bjorn.munch@sun.com-20091119092424-cpc87ef58xms6o9b) (merge vers: 5.5.0-beta) (pib:16)

Pushed into mysql-next-mr (revid:alik@sun.com-20100220092730-paoo5t9gcajs5dq8) (version source revid:bjorn.munch@sun.com-20091119092557-tm3cmjhd1jnlj0qc) (pib:16)

Changes to test suite. No changelog entry needed.

Setting report to Need Merge pending push to 5.1.x, Celosia.

Pushed into 5.1.45 (revid:joro@sun.com-20100301083827-xnimmrjg6bh33o1o) (version source revid:azundris@mysql.com-20100222175719-viuh0f3gdsrkgv0r) (merge vers: 5.1.45) (pib:16)

No changelog entry needed.

Pushed into 5.1.47-ndb-7.0.16 (revid:martin.skold@mysql.com-20100617114014-bva0dy24yyd67697) (version source revid:vasil.dimov@oracle.com-20100331130613-8ja7n0vh36a80457) (merge vers: 5.1.46) (pib:16)

Pushed into 5.1.47-ndb-6.2.19 (revid:martin.skold@mysql.com-20100617115448-idrbic6gbki37h1c) (version source revid:martin.skold@mysql.com-20100609140708-52rvuyq4q500sxkq) (merge vers: 5.1.45-ndb-6.2.19) (pib:16)

Pushed into 5.1.47-ndb-6.3.35 (revid:martin.skold@mysql.com-20100617114611-61aqbb52j752y116) (version source revid:vasil.dimov@oracle.com-20100331130613-8ja7n0vh36a80457) (merge vers: 5.1.46) (pib:16)