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

• Submitted: 16 Nov 2009 14:51
• Modified: 17 Jun 2010 19:38
• Reporter: Vladislav Vaintroub
• Status: Closed
• Category: Tools: MTR / mysql-test-run
• Severity: S3 (Non-critical)
• OS: Windows
• Assigned to: Bjørn Munch
• CPU Architecture: Any

[16 Nov 2009 14:51] Vladislav Vaintroub

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)

[16 Nov 2009 15:04] Vladislav Vaintroub

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;

[18 Nov 2009 9:28] Bugs System

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

[18 Nov 2009 9:30] Bjørn Munch

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

[19 Nov 2009 11:27] Bjørn Munch

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

[20 Feb 2010 9:32] Bugs System

Pushed into 6.0.14-alpha (revid:[e-mail]-20100220092844-jh07ayojyxn8uh9p) (version source revid:[e-mail]-20091119111729-sazk6qfk8qbwfq0y) (merge vers: 6.0.14-alpha) (pib:16)

[20 Feb 2010 9:34] Bugs System

Pushed into 5.5.3-m2 (revid:[e-mail]-20100220092622-wvhh1vfy5tjq4mhu) (version source revid:[e-mail]-20091119092424-cpc87ef58xms6o9b) (merge vers: 5.5.0-beta) (pib:16)

[20 Feb 2010 9:37] Bugs System

Pushed into mysql-next-mr (revid:[e-mail]-20100220092730-paoo5t9gcajs5dq8) (version source revid:[e-mail]-20091119092557-tm3cmjhd1jnlj0qc) (pib:16)

[23 Feb 2010 2:08] Paul DuBois

Changes to test suite. No changelog entry needed.

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

[1 Mar 2010 8:47] Bugs System

Pushed into 5.1.45 (revid:[e-mail]-20100301083827-xnimmrjg6bh33o1o) (version source revid:[e-mail]-20100222175719-viuh0f3gdsrkgv0r) (merge vers: 5.1.45) (pib:16)

[1 Mar 2010 16:27] Paul DuBois

No changelog entry needed.

[17 Jun 2010 12:17] Bugs System

Pushed into 5.1.47-ndb-7.0.16 (revid:[e-mail]-20100617114014-bva0dy24yyd67697) (version source revid:[e-mail]-20100331130613-8ja7n0vh36a80457) (merge vers: 5.1.46) (pib:16)

[17 Jun 2010 13:04] Bugs System

Pushed into 5.1.47-ndb-6.2.19 (revid:[e-mail]-20100617115448-idrbic6gbki37h1c) (version source revid:[e-mail]-20100609140708-52rvuyq4q500sxkq) (merge vers: 5.1.45-ndb-6.2.19) (pib:16)

[17 Jun 2010 13:45] Bugs System

Pushed into 5.1.47-ndb-6.3.35 (revid:[e-mail]-20100617114611-61aqbb52j752y116) (version source revid:[e-mail]-20100331130613-8ja7n0vh36a80457) (merge vers: 5.1.46) (pib:16)