Opened 15 years ago

Closed 11 years ago

#715 closed defect (fixed)

OSGeo4W patch for DLLs

Reported by: neteler Owned by: grass-dev@…
Priority: normal Milestone: 6.4.0
Component: Default Version: 6.4.0 RCs
Keywords: wingrass, osgeo4w Cc: jef
CPU: All Platform: MSWindows XP

Description

Attached patch was submitted by Juergen Fischer, please review.

Attachments (1)

dll_osgeo4w_fixes.diff​ (1.4 KB ) - added by neteler 15 years ago.
DLL patch

Download all attachments as: .zip

Change History (10)

by neteler, 15 years ago

Attachment: dll_osgeo4w_fixes.diff​ added

DLL patch

comment:1 by neteler, 15 years ago

Cc: jef added

in reply to:  description comment:2 by glynn, 15 years ago

Replying to neteler:

Attached patch was submitted by Juergen Fischer, please review.

Is there some reason that Shlib.make and Stlib.make can't use ARCH_CMD_OBJS, so that Makefiles can just override MOD_OBJS if they don't want the default?

Failing that, Lib.make should use similar logic to Rules.make, i.e. only define the variables if they aren't already defined.

Either way, Lib.make should not contain references to specific files (fmode.o and dllmain.o).

comment:3 by neteler, 15 years ago

jef: any suggestions for us?

comment:4 by jef, 15 years ago

The point is just that the DLLs need to be linked with dllmain.o and the executables with fmode.o.

comment:5 by neteler, 14 years ago

Keywords: osgeo4w added

Jef, is the patch still needed?

comment:6 by neteler, 14 years ago

Related (?) fix in G7: r43279

comment:7 by hamish, 12 years ago

still relevant?

comment:8 by neteler, 11 years ago

See also ticket #469

comment:9 by neteler, 11 years ago

Resolution: → fixed
Status: new → closed

Since no user complaints have been collected, closing for now.

(See also GRASS 6.4.svn (r35767, r35769) and 6.4.0svn (r35768, r35770) from #469)

Note: See TracTickets for help on using tickets.