On Sun, 2010-08-08 at 08:10 -0400, Gaetan Nadon wrote:
> It would be preferable to do the LN_S change in a separate patch as it
> distract from the core changes.

Using LN_S is necessary for MinGW, which is one of the two systems
affected by the patch, so it's not really a separate thing.

> Is the library extension renaming part of the exec-hook? Why does it
> have to change?

Huh?

> I had a hard time understand the creative explanation "must remain !
> PLATFORM_WIN32".
> Perhaps something like "must be done on all platforms including
> WIN32".

No, it must remain in a !PLATFORM_WIN32 conditional block.

> One more thing, which isn't related to your path, does the "clean" and
> "uninstall" target work correctly, given the
> work done in the exec hook?

Actually, now that you mention it, there should be an uninstall-local to
remove the symlinks created by install-exec-hook, but there is not.


Yaakov
Cygwin/X


_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to