Patch: Fix --enable-win32-relocatable

[aspell-helper]

Martin Schlemmer <mschlemmer@sf> created a patch on 2010-04-19 15:32:33 UTC
(Orig. from https://sourceforge.net/p/aspell/patches/164)

Hi

I did see that there is an older patch to do this, but I do not use Visual Studio for building, but MSYS/MinGW. Also that patch do not handle cases where you link against the DLL but the linked application is not in the same directory as the DLL, which this one does.

Patch is against 0.60.6 release version. Ask if more information is required.

Regards

Martin

[aspell-helper]

Martin Schlemmer <mschlemmer@sf> commented on 2010-04-19 16:14:38 UTC

Sorry, uploaded earlier version with some issues.

[aspell-helper]

Kevin Atkinson <kevina@sf> commented on 2010-04-19 20:56:02 UTC

Please include a Copyright notice at the top of dir_utils.cpp. See other files for a model. The copyright should be yours. If you got the code from somewhere else please give credit, and briefly indicate what changes you made.

I will likely include in Aspell 0.60.7.

[aspell-helper]

Kevin Atkinson <kevina@sf> updated the issue on 2010-04-19 20:56:02 UTC

• milestone: --> 0.60

[aspell-helper]

Martin Schlemmer <mschlemmer@sf> commented on 2010-04-20 09:26:51 UTC

aspell-0.60.6-win32.patch (updated)
aspell-0.60.6-win32.patch

[aspell-helper]

Martin Schlemmer <mschlemmer@sf> commented on 2010-04-20 09:31:24 UTC

Added copyright, some notes, cleanups and tried to get the code formatting a bit more like the rest of Aspell. Let me know if anything is still out of order.

[aspell-helper]

Kevin Atkinson <kevina@sf> commented on 2010-04-20 09:54:07 UTC

Thanks,

Can you please look at the patch at http://www.wxperl.co.uk/aspell/ and tell me how it compares to yours. They both seam to do about the same thing and I can't include them both.

[aspell-helper]

Martin Schlemmer <mschlemmer@sf> commented on 2010-04-20 10:17:18 UTC

Hmm, guess Google is not the answer to everything.

Anyhow, in spirit they are the same, that one just assumed that the DLL will be called "libaspell-15.dll" which I did not want to assume. So I guess it will be a choice between an extra dependency (Psapi), or changing the string for the DLL name when a major version updates occurs.

I have no idea what people using VC++ are doing, so both could make assumptions about the name (for instance VC++ could just generate "aspell.dll"), but I guess that is something a person using that will have to attend to.

Lastly, the last hunk of that patch (touching file_util.cpp) is rather another bug which I also ran into, but just did not report yet.

[aspell-helper]

Kevin Atkinson <kevina@sf> commented on 2010-04-20 10:33:19 UTC

What is psapi, is it fairly common on Windows?

[aspell-helper]

Kevin Atkinson <kevina@sf> commented on 2010-04-20 10:34:59 UTC

Also, it seams like the DLL name should be known when compiling...

[aspell-helper]

Martin Schlemmer <mschlemmer@sf> commented on 2010-04-20 13:50:33 UTC

It is the API used for EnumProcessModules() and GetModuleFileNameEx(), and only available from Windows 2000 and later, so if Win9x should still be supported, it will be a problem.

I come more from a Linux/MinGW background than VC++, so I do not know what libtool does when using VC++. For MSYS/MinGW it generates it (like *nix environments) from the -version-info argument. Seems like it creates it for MSYS/MinGW as -<major - 1>.dll

Anyhow, either leave as is, don't version the DLL (which might get you into trouble later on), or just hope libtool keep its versioning and do it automatically.

I did last with that patch, and also fixed a problem where they changed the dict and data dirs, but did not reflect it during configure stage, so the data files still installed to lib/aspell-0.60.

Patch follows.

[aspell-helper]

Martin Schlemmer <mschlemmer@sf> commented on 2010-04-20 13:55:29 UTC

aspell-0.60.6-mingw-w64-2.patch (Modified patch from http://www.wxperl.co.uk/aspell/ )
aspell-0.60.6-mingw-w64-2.patch