Commit 5f6d7353 authored by Neil Conway's avatar Neil Conway

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
parent 840df515
...@@ -10,9 +10,10 @@ if (-f "$ARGV[0]/$defname.def") { ...@@ -10,9 +10,10 @@ if (-f "$ARGV[0]/$defname.def") {
print "Generating $defname.DEF from directory $ARGV[0]\n"; print "Generating $defname.DEF from directory $ARGV[0]\n";
while (<$ARGV[0]/*>) { while (<$ARGV[0]/*.obj>) {
print "."; 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>) { while (<F>) {
s/\(\)//g; s/\(\)//g;
next unless /^\d/; next unless /^\d/;
...@@ -31,6 +32,7 @@ while (<$ARGV[0]/*>) { ...@@ -31,6 +32,7 @@ while (<$ARGV[0]/*>) {
push @def, $pieces[6]; push @def, $pieces[6];
} }
close(F); close(F);
unlink("symbols.out");
} }
print "\n"; print "\n";
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment