[GRASS-dev] [GRASS GIS] #2996: Lidar modules broken in 64bit WinGRASS

#2996: Lidar modules broken in 64bit WinGRASS
-------------------------+-------------------------------------------------
  Reporter: wenzeslaus | Owner: grass-dev@…
      Type: defect | Status: reopened
  Priority: blocker | Milestone: 7.2.0
Component: Packaging | Version: svn-releasebranch70
Resolution: | Keywords: lasinfo, r.in.lidar, v.in.lidar,
                         | r3.in.lidar, libLAS
       CPU: x86-64 | Platform: Unspecified
-------------------------+-------------------------------------------------

Comment (by martinl):

In [changeset:"70110" 70110]:
{{{
#!CommitTicketReference repository="" revision="70110"
wingrass: attempt to fix standalone installer to download and execute
vcredist exe files - add missing global vars (see #2996)
}}}

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/2996#comment:41&gt;
GRASS GIS <https://grass.osgeo.org>

#2996: Lidar modules broken in 64bit WinGRASS
-------------------------+-------------------------------------------------
  Reporter: wenzeslaus | Owner: grass-dev@…
      Type: defect | Status: reopened
  Priority: blocker | Milestone: 7.2.0
Component: Packaging | Version: svn-releasebranch70
Resolution: | Keywords: lasinfo, r.in.lidar, v.in.lidar,
                         | r3.in.lidar, libLAS
       CPU: x86-64 | Platform: Unspecified
-------------------------+-------------------------------------------------

Comment (by martinl):

In [changeset:"70113" 70113]:
{{{
#!CommitTicketReference repository="" revision="70113"
wingrass: attempt to fix standalone installer to download and execute
vcredist - fix typo (see #2996)
}}}

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/2996#comment:42&gt;
GRASS GIS <https://grass.osgeo.org>

#2996: Lidar modules broken in 64bit WinGRASS
-------------------------+-------------------------------------------------
  Reporter: wenzeslaus | Owner: grass-dev@…
      Type: defect | Status: reopened
  Priority: blocker | Milestone: 7.2.0
Component: Packaging | Version: svn-releasebranch70
Resolution: | Keywords: lasinfo, r.in.lidar, v.in.lidar,
                         | r3.in.lidar, libLAS
       CPU: x86-64 | Platform: Unspecified
-------------------------+-------------------------------------------------

Comment (by martinl):

Replying to [comment:40 hellik]:
> which vcredist.exe-versions are needed depends upon the dependencies
built upon.

I did quick investigation:

{{{
32bit:

grep msvcrt2005 * --include='setup.hint' -R

-> no dep

grep msvcrt2008 * --include='setup.hint' -R

-> 1 dep

msvcrt/setup.hint:requires: msvcrt2008 msvcrt2010

grep msvcrt2010 * --include='setup.hint' -R

-> 5 deps (none of them are part of standalone installer)

gdal/gdal-dev/gdal-dev-sosi/setup.hint:requires: gdal-dev msvcrt2010
gdal/gdal-dev/gdal-dev-filegdb/setup.hint:requires: gdal-dev msvcrt2010
gdal/gdal-dev/gdal-dev-oracle/setup.hint:requires: gdal-dev oci msvcrt2010
libjpeg
icu/icu-libs/setup.hint:requires: msvcrt2010
msvcrt/setup.hint:requires: msvcrt2008 msvcrt2010

grep msvcrt2013 * --include='setup.hint' -R

-> 1 dep (part of standalone installer)

libpq/setup.hint:requires: shell msvcrt2013 libintl

grep msvcrt2015 * --include='setup.hint' -R

-> no dep
}}}

{{{
64bit:

grep msvcrt2008 * --include='setup.hint' -R

-> 1 dep

msvcrt/setup.hint:requires: msvcrt2008 msvcrt2010

grep msvcrt2010 * --include='setup.hint' -R

-> 5 deps (none of them part of standalone installer)

gdal/gdal-dev/gdal-dev-sosi/setup.hint:requires: gdal-dev msvcrt2010
gdal/gdal-dev/gdal-dev-filegdb/setup.hint:requires: gdal-dev msvcrt2010
gdal/gdal-dev/gdal-dev-oracle/setup.hint:requires: gdal-dev oci msvcrt2010
libjpeg
icu/icu-libs/setup.hint:requires: msvcrt2010
msvcrt/setup.hint:requires: msvcrt2008 msvcrt2010

grep msvcrt2012 * --include='setup.hint' -R

-> 4 deps (none of them part of standalone installer)

boost/boost-devel/setup.hint:requires: msvcrt2012
boost/boost-libs/setup.hint:requires: msvcrt2012
boost/setup.hint:requires: msvcrt2012
pcl/setup.hint:requires: flann eigen boost msvcrt2012

grep msvcrt2013 * --include='setup.hint' -R

-> 4 deps (liblas part of standalone installer)

hexer/setup.hint:requires: msvcrt gdal msvcrt2013
liblas/setup.hint:requires: shell libtiff libgeotiff gdal111dll proj proj-
hpgn proj-datumgrid proj-vdatum laszip msvcrt2013
libspatialindex/setup.hint:requires: shell msvcrt2013
points2grid/setup.hint:requires: shell msvcrt2013
}}}

So it seems that we need (liblas, libpq) for standalone installer just
msvcrt2013
(both on 32bit and 64bit). To be safe we could add msvcrt2008 and
mscvrt2010 (which are dependencies of msvcrt package). On 32bit
platform also msvcrt2012 (bacause of boost-libs). Then the result will
be exactly same as with osgeo4w installer.

In summary:

{{{
32bit: msvcrt2008,msvcrt2010,msvcrt2012,msvcrt2013

64bit: msvcrt2008,msvcrt2010,msvcrt2013
}}}

Any comments?

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/2996#comment:43&gt;
GRASS GIS <https://grass.osgeo.org>

#2996: Lidar modules broken in 64bit WinGRASS
-------------------------+-------------------------------------------------
  Reporter: wenzeslaus | Owner: grass-dev@…
      Type: defect | Status: reopened
  Priority: blocker | Milestone: 7.2.0
Component: Packaging | Version: svn-releasebranch70
Resolution: | Keywords: lasinfo, r.in.lidar, v.in.lidar,
                         | r3.in.lidar, libLAS
       CPU: x86-64 | Platform: Unspecified
-------------------------+-------------------------------------------------

Comment (by martinl):

Replying to [comment:43 martinl]:
> In summary:
>
> {{{
> 32bit: msvcrt2008,msvcrt2010,msvcrt2012,msvcrt2013
>
> 64bit: msvcrt2008,msvcrt2010,msvcrt2013
> }}}

Correct summary:

{{{
32bit: msvcrt2008,msvcrt2010,msvcrt2013

64bit: msvcrt2008,msvcrt2010,msvcrt2012,msvcrt2013
}}}

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/2996#comment:44&gt;
GRASS GIS <https://grass.osgeo.org>

#2996: Lidar modules broken in 64bit WinGRASS
-------------------------+-------------------------------------------------
  Reporter: wenzeslaus | Owner: grass-dev@…
      Type: defect | Status: reopened
  Priority: blocker | Milestone: 7.2.0
Component: Packaging | Version: svn-releasebranch70
Resolution: | Keywords: lasinfo, r.in.lidar, v.in.lidar,
                         | r3.in.lidar, libLAS
       CPU: x86-64 | Platform: Unspecified
-------------------------+-------------------------------------------------

Comment (by martinl):

In [changeset:"70114" 70114]:
{{{
#!CommitTicketReference repository="" revision="70114"
wingrass: attempt to fix standalone installer to download and execute
vcredist - cleanup dependencies (see #2996)
}}}

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/2996#comment:45&gt;
GRASS GIS <https://grass.osgeo.org>

#2996: Lidar modules broken in 64bit WinGRASS
-------------------------+-------------------------------------------------
  Reporter: wenzeslaus | Owner: grass-dev@…
      Type: defect | Status: reopened
  Priority: blocker | Milestone: 7.2.0
Component: Packaging | Version: svn-releasebranch70
Resolution: | Keywords: lasinfo, r.in.lidar, v.in.lidar,
                         | r3.in.lidar, libLAS
       CPU: x86-64 | Platform: Unspecified
-------------------------+-------------------------------------------------

Comment (by martinl):

Replying to [comment:44 martinl]:
> Correct summary:
>
> {{{
> 32bit: msvcrt2008,msvcrt2010,msvcrt2013
>
> 64bit: msvcrt2008,msvcrt2010,msvcrt2012,msvcrt2013
> }}}

Done r70114

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/2996#comment:46&gt;
GRASS GIS <https://grass.osgeo.org>

#2996: Lidar modules broken in 64bit WinGRASS
-------------------------+-------------------------------------------------
  Reporter: wenzeslaus | Owner: grass-dev@…
      Type: defect | Status: reopened
  Priority: blocker | Milestone: 7.2.0
Component: Packaging | Version: svn-releasebranch70
Resolution: | Keywords: lasinfo, r.in.lidar, v.in.lidar,
                         | r3.in.lidar, libLAS
       CPU: x86-64 | Platform: Unspecified
-------------------------+-------------------------------------------------

Comment (by hellik):

Replying to [comment:44 martinl]:
> Replying to [comment:43 martinl]:
> > In summary:
> >
> > {{{
> > 32bit: msvcrt2008,msvcrt2010,msvcrt2012,msvcrt2013
> >
> > 64bit: msvcrt2008,msvcrt2010,msvcrt2013
> > }}}
>
> Correct summary:
>
> {{{
> 32bit: msvcrt2008,msvcrt2010,msvcrt2013
>
> 64bit: msvcrt2008,msvcrt2010,msvcrt2012,msvcrt2013
> }}}
>

Looks good.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/2996#comment:47&gt;
GRASS GIS <https://grass.osgeo.org>

#2996: Lidar modules broken in 64bit WinGRASS
-------------------------+-------------------------------------------------
  Reporter: wenzeslaus | Owner: grass-dev@…
      Type: defect | Status: reopened
  Priority: blocker | Milestone: 7.2.0
Component: Packaging | Version: svn-releasebranch70
Resolution: | Keywords: lasinfo, r.in.lidar, v.in.lidar,
                         | r3.in.lidar, libLAS
       CPU: x86-64 | Platform: Unspecified
-------------------------+-------------------------------------------------

Comment (by martinl):

In [changeset:"70123" 70123]:
{{{
#!CommitTicketReference repository="" revision="70123"
winGRASS installer: define ARCHIVE_NAME2012 only for 64bit (see #2996)
}}}

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/2996#comment:48&gt;
GRASS GIS <https://grass.osgeo.org>

#2996: Lidar modules broken in 64bit WinGRASS
-------------------------+-------------------------------------------------
  Reporter: wenzeslaus | Owner: grass-dev@…
      Type: defect | Status: reopened
  Priority: blocker | Milestone: 7.2.0
Component: Packaging | Version: svn-releasebranch70
Resolution: | Keywords: lasinfo, r.in.lidar, v.in.lidar,
                         | r3.in.lidar, libLAS
       CPU: x86-64 | Platform: Unspecified
-------------------------+-------------------------------------------------

Comment (by martinl):

In [changeset:"70126" 70126]:
{{{
#!CommitTicketReference repository="" revision="70126"
wingrass: attempt to fix standalone installer to download and execute
vcredist (see #2996)
           (relbr72: merge 70106, 70110, 70113, 70114, 70123 from trunk)
}}}

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/2996#comment:49&gt;
GRASS GIS <https://grass.osgeo.org>

#2996: Lidar modules broken in 64bit WinGRASS
-------------------------+-------------------------------------------------
  Reporter: wenzeslaus | Owner: grass-dev@…
      Type: defect | Status: reopened
  Priority: blocker | Milestone: 7.2.0
Component: Packaging | Version: svn-releasebranch70
Resolution: | Keywords: lasinfo, r.in.lidar, v.in.lidar,
                         | r3.in.lidar, libLAS
       CPU: x86-64 | Platform: Unspecified
-------------------------+-------------------------------------------------

Comment (by martinl):

I tested the latest G73 (1) on freshly installed Windows 8.1 64bit. The
Lidar modules seems to work. Please could you confirm?

(1)
https://wingrass.fsv.cvut.cz/grass73/x86_64/WinGRASS-7.3.svn-r70124-101
-Setup-x86_64.exe

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/2996#comment:50&gt;
GRASS GIS <https://grass.osgeo.org>

#2996: Lidar modules broken in 64bit WinGRASS
-------------------------+-------------------------------------------------
  Reporter: wenzeslaus | Owner: grass-dev@…
      Type: defect | Status: reopened
  Priority: blocker | Milestone: 7.2.0
Component: Packaging | Version: svn-releasebranch70
Resolution: | Keywords: lasinfo, r.in.lidar, v.in.lidar,
                         | r3.in.lidar, libLAS
       CPU: x86-64 | Platform: Unspecified
-------------------------+-------------------------------------------------

Comment (by annakrat):

Replying to [comment:50 martinl]:
> I tested the latest G73 (1) on freshly installed Windows 8.1 64bit. The
Lidar modules seems to work. Please could you confirm?
>
> (1)
https://wingrass.fsv.cvut.cz/grass73/x86_64/WinGRASS-7.3.svn-r70124-101
-Setup-x86_64.exe

Yes, it works. I suggest to close this for now. I am still experiencing
crashing on one computer with older Windows installation, but it's hard to
tell what the problem could be.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/2996#comment:51&gt;
GRASS GIS <https://grass.osgeo.org>

#2996: Lidar modules broken in 64bit WinGRASS
-------------------------+-------------------------------------------------
  Reporter: wenzeslaus | Owner: grass-dev@…
      Type: defect | Status: reopened
  Priority: blocker | Milestone: 7.0.6
Component: Packaging | Version: svn-releasebranch70
Resolution: | Keywords: lasinfo, r.in.lidar, v.in.lidar,
                         | r3.in.lidar, libLAS
       CPU: x86-64 | Platform: Unspecified
-------------------------+-------------------------------------------------
Changes (by martinl):

* milestone: 7.2.0 => 7.0.6

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/2996#comment:52&gt;
GRASS GIS <https://grass.osgeo.org>

#2996: Lidar modules broken in 64bit WinGRASS
-------------------------+-------------------------------------------------
  Reporter: wenzeslaus | Owner: grass-dev@…
      Type: defect | Status: closed
  Priority: blocker | Milestone: 7.0.6
Component: Packaging | Version: svn-releasebranch70
Resolution: fixed | Keywords: lasinfo, r.in.lidar, v.in.lidar,
                         | r3.in.lidar, libLAS
       CPU: x86-64 | Platform: Unspecified
-------------------------+-------------------------------------------------
Changes (by martinl):

* status: reopened => closed
* resolution: => fixed

Comment:

In [changeset:"70128" 70128]:
{{{
#!CommitTicketReference repository="" revision="70128"
wingrass: attempt to fix standalone installer to download and execute
vcredist (fix #2996)
     (relbr70: merge 70106, 70110, 70113, 70114, 70123 from trunk)
}}}

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/2996#comment:53&gt;
GRASS GIS <https://grass.osgeo.org>