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)

module.diff (1.2 KB ) - added by warmerdam 16 years ago.
Patch to switch name to mapscript_java.dll and libmapscript_java.so.

Download all attachments as: .zip

Change History (7)

comment:1 by warmerdam, 16 years ago

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.

by warmerdam, 16 years ago

Attachment: module.diff added

Patch to switch name to mapscript_java.dll and libmapscript_java.so.

in reply to:  1 comment:2 by unicoletti, 16 years ago

Status: newassigned

Replying to warmerdam:

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.

comment:3 by hobu, 14 years ago

Is this bug dead? Please close if it is.

comment:4 by warmerdam, 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 unicoletti, 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.

comment:6 by unicoletti, 12 years ago

Resolution: fixed
Status: assignedclosed

closing

Note: See TracTickets for help on using tickets.