|
From: Nicholas N. <nj...@cs...> - 2005-06-26 22:53:15
|
On Mon, 27 Jun 2005, Dirk Mueller wrote: > another one from the "doesn't build" department: > > valgrind.vs is used for linking "stage2", which versions the exported symbols > with VALGRIND_2_1. > > I see several problems with that: > > a) VALGRIND_2_1 wasn't updated to VALGRIND_2_4 > b) stage2 is installed in the valgrind lib subdir, so it doesn't matter > c) stage2 is a plain binary, nothing links against it > > > Newer linkers correctly complain about c) and abort the build, because trying > to version a binary simply doesn't make any sense. At least as far as I can > imagine. > > So, can I just remove that? I thought the point of this file was to force any symbol that doesn't match any of the given prefixes (vgPlain_, etc) to be not exported. And the prefixes are out of date now. I'd be fine with it going. It's caused me troubles in the past by secretly hiding certain symbols that would normally have been public. It doesn't seem to add much. Jeremy, I believe you added it -- does it have some other purpose? N |