Attached patch fixes two problems:

1) gendef works from inside visual studio - use a tempfile instead of
redirection, because for some reason you can't redirect dumpbin from
inside (patch from Joachim Wieland)
2) gendef must process only *.obj, or you get weird errors in some build
scenarios when it tries to process a logfile

Magnus Hagander

diff --git a/src/tools/msvc/gendef.pl b/src/tools/msvc/gendef.pl
index d5d9bc78c7d4c0f063974d7317db63d2e8a576d7..675d8423b4032fffdb598461fae85dd39cd48c09 100644 (file)
--- a/src/tools/msvc/gendef.pl
+++ b/src/tools/msvc/gendef.pl
@@ -10,9 +10,10 @@ if (-f "$ARGV[0]/$defname.def") {
 
 print "Generating $defname.DEF from directory $ARGV[0]\n";
 
-while (<$ARGV[0]/*>) {
+while (<$ARGV[0]/*.obj>) {
     print ".";
-   open(F,"dumpbin /symbols $_|") || die "Could not open $_\n";
+   system("dumpbin /symbols /out:symbols.out $_ >NUL") && die "Could not call dumpbin";
+   open(F, "<symbols.out") || die "Could not open symbols.out for $_\n";
    while (<F>) {
        s/\(\)//g;
        next unless /^\d/;
@@ -31,6 +32,7 @@ while (<$ARGV[0]/*>) {
        push @def, $pieces[6];
    }
    close(F);
+   unlink("symbols.out");
 }
 print "\n";