Opened 16 years ago
Closed 12 years ago
#2595 closed defect (fixed)
mapscript.dll bad name for Java mapscript interface
Reported by: | warmerdam | Owned by: | unicoletti |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | MapScript-Java | Version: | 5.0 |
Severity: | normal | Keywords: | java |
Cc: |
Description
I think it is a bad idea to build the support JNI DLL for Java mapscript as mapscript.dll as this may well conflict with support DLLs for other swig bindings. I propose it be renamed mapscript_java.dll.
Attachments (1)
Change History (7)
follow-up: 2 comment:1 by , 16 years ago
by , 16 years ago
Attachment: | module.diff added |
---|
Patch to switch name to mapscript_java.dll and libmapscript_java.so.
comment:2 by , 16 years ago
Status: | new → assigned |
---|
comment:4 by , 14 years ago
Milestone: | 5.6 release |
---|
I have no reason to believe the suggested renaming was done. I would suggest this ticket be de-milestoned but left open for such a time as we have a java maintainer mapscript active enough to consider it.
I have taken the liberty of de-milestoning it.
comment:5 by , 12 years ago
Done in r13166 The shared library name is libjavamapscript.so I have done the necessary renaming for Windows Visual Studio, but I don't have (actually never had) a Windows build environment so I call for somebody else to test it.
I will leave the ticket opened for some time to track breaking changes.
Attaching a patch that I think would do the trick, though I've only tried it on win32. I've also applied a patch (r7527) to remove explicit loading of library "mapscript" in the examples code.