Ticket #519 (closed task: fixed)

Opened 4 years ago

Last modified 10 months ago

osgeo4w build

Reported by: jef Owned by: grass-dev@…
Priority: major Milestone: 6.4.0
Component: Default Version: svn-develbranch6
Keywords: Cc: martinl
Platform: All CPU: All

Description

attached the modifications I did to my GRASS tree to build for OSGeo4W

Attachments

519.diff Download (14.1 KB) - added by jef 4 years ago.
mingw_updated.2.diff Download (3.3 KB) - added by neteler 4 years ago.
patch reduced to outstanding item
mingw_updated.diff Download (3.3 KB) - added by neteler 4 years ago.
patch reduced to outstanding item
mingw_updated3.diff Download (1.6 KB) - added by neteler 4 years ago.
patch reduced to outstanding changes
mingw_updated4.diff Download (0.7 KB) - added by neteler 3 years ago.
patch reduced to outstanding issues

Change History

Changed 4 years ago by jef

  Changed 4 years ago by neteler

  • platform changed from Unspecified to All
  • cpu changed from Unspecified to All

I have merged in the changes from Glynn (mailing list).

Questions: is the NOFILTEROUT=1 stuff needed (see include/Make/Lib.make and lib/gis/Makefile in the path)?

I have taken out in the revised patch:

Index: lib/g3d/g3dmask.c
===================================================================
--- lib/g3d/g3dmask.c   (revision 36200)
+++ lib/g3d/g3dmask.c   (working copy)
@@ -309,12 +309,12 @@
        for (dy = y; dy < rows; dy++) {
            for (dx = x; dx < cols; dx++) {
                G3D_MASKNUM(map, dx, dy, dz, tile, type);
-               tile += length;
+               (unsigned char *)tile += length;
            }

-           tile += xLength;
+           (unsigned char *)tile += xLength;
        }
-       tile += yLength;
+       (unsigned char *)tile += yLength;
     }
 }

since it doesn't compile on Linux (C++ syntax?).

Please review the mingw_updated.diff patch and suggest for lib/g3d/g3dmask.c.

follow-up: ↓ 4   Changed 4 years ago by martinl

wxGUI-related part of the patch submitted to devbr6 ( http://trac.osgeo.org/grass/changeset/36220). The rest in progress...

Martin

  Changed 4 years ago by martinl

  • cc martinl added

in reply to: ↑ 2   Changed 4 years ago by martinl

Replying to martinl:

wxGUI-related part of the patch submitted to devbr6 ( http://trac.osgeo.org/grass/changeset/36220). The rest in progress...

Please can someone review unsubmitted part of the patch:

M      raster/Makefile
M      lib/g3d/g3dcache.c
M      lib/g3d/g3dmask.c
M      lib/gis/spawn.c
M      lib/gis/Makefile
M      lib/gis/gdal.c
M      include/Make/Lib.make
M      include/Make/Shlib.make

I don't feel enough familiar with building system in GRASS.

M.

Changed 4 years ago by neteler

patch reduced to outstanding item

Changed 4 years ago by neteler

patch reduced to outstanding item

  Changed 4 years ago by neteler

I have reduced the attached patch to the outstanding items except for the lib/g3d/g3dmask.c

part which is only in the original submission. For that, see comment 1 above.

Changed 4 years ago by neteler

patch reduced to outstanding changes

follow-up: ↓ 7   Changed 4 years ago by neteler

Patch reduced to outstanding items (with modification for raster/Makefile). Left out lib/g3d/g3dmask.c as unclear how to write it correctly (see first comment here). Please test + comment.

The other issues are in 6.4.0svn and 6.5svn now.

Changed 3 years ago by neteler

patch reduced to outstanding issues

in reply to: ↑ 6   Changed 10 months ago by hellik

  • status changed from new to closed
  • resolution set to fixed

Replying to neteler:

[...] The other issues are in 6.4.0svn and 6.5svn now.

patches applied and a lot of fixes done in the last months.

closing ticket, reopen if needed.

Helmut

Note: See TracTickets for help on using tickets.