Opened 5 years ago

Closed 14 months ago

#2631 closed defect (fixed)

Future winGRASS needs python-ply package

Reported by: wenzeslaus Owned by: grass-dev@…
Priority: normal Milestone: 7.4.2
Component: Temporal Version: svn-trunk
Keywords: OSGeo4W, winGRASS, decencies, requirements Cc:
CPU: Unspecified Platform: MSWindows 8

Description

WinGRASS installer (and OSGeo4W) will need python-ply package as specified in the source:grass/trunk/REQUIREMENTS.html:

Python PLY Library (Python Lex-Yacc) ("python-ply", needed for the temporal algebra in tgis) http://www.dabeaz.com/ply

Currently, temporal algebra does not work, see for example this tests result:

The workaround would be to install python-ply manually but that's a little bit challenging since winGRASS/OSGeo4W is using its own Python.

This is for 7.1 because temporal algebra is only there. So it might be enough to solve it right before 7.1 is released. The python-ply requirement is actually listed also for 7.0 but I think its not true:

Change History (19)

comment:1 Changed 5 years ago by martinl

First of all python-ply must be introduced to OSGeo4W framework. Please report this issue on their tracker.

comment:2 in reply to:  1 ; Changed 5 years ago by wenzeslaus

Replying to martinl:

First of all python-ply must be introduced to OSGeo4W framework. Please report this issue on their tracker.

Done. This ticket now depends on OSGeo4W ticket number 461:

Any idea about this dependency in 7.0?

comment:3 Changed 4 years ago by neteler

Milestone: 7.1.07.2.0

Milestone renamed

comment:4 Changed 3 years ago by neteler

Milestone: 7.2.07.2.1

Ticket retargeted after milestone closed

comment:5 Changed 3 years ago by martinl

Milestone: 7.2.17.2.2

comment:6 Changed 2 years ago by neteler

Milestone: 7.2.27.2.3

Ticket retargeted after milestone closed

comment:7 Changed 20 months ago by martinl

Milestone: 7.2.3

Ticket retargeted after milestone closed

comment:8 Changed 20 months ago by martinl

Milestone: 7.2.4

comment:9 Changed 15 months ago by neteler

We just got hit by this problem at GeoSTAT 2018 teaching - any solution?

comment:10 Changed 15 months ago by neteler

In https://download.osgeo.org/osgeo4w/x86_64/release/grass/grass/setup.hint the package "python-pip" should be added.

Ideally also "python install PLY" should be executed as compile time of WinGRASS / be an OSGeo4W dependency, to avoid that https://grass.osgeo.org/grass74/manuals/t.rast.algebra.html is delivered broken to Windows users.

comment:11 in reply to:  10 Changed 15 months ago by hellik

Replying to neteler:

In https://download.osgeo.org/osgeo4w/x86_64/release/grass/grass/setup.hint the package "python-pip" should be added.

pip added as dependency in winGRASS by r73171, r73172

comment:12 in reply to:  2 ; Changed 15 months ago by hellik

Replying to wenzeslaus:

Replying to martinl:

First of all python-ply must be introduced to OSGeo4W framework. Please report this issue on their tracker.

Done. This ticket now depends on OSGeo4W ticket number 461:

just updated ticket 461 with a packaged ply for OSgeo4W.

packaged waiting there for being approved

Last edited 15 months ago by hellik (previous) (diff)

comment:13 Changed 15 months ago by martinl

Milestone: 7.2.47.4.2

comment:14 Changed 15 months ago by martinl

I have installed python-pip in build environment. It should appear in the next daily build.

comment:15 in reply to:  12 ; Changed 15 months ago by hellik

Replying to hellik:

Replying to wenzeslaus:

Replying to martinl:

First of all python-ply must be introduced to OSGeo4W framework. Please report this issue on their tracker.

Done. This ticket now depends on OSGeo4W ticket number 461:

just updated ticket 461 with a packaged ply for OSgeo4W.

packaged waiting there for being approved

ply is now in OSGeo4W thanks to jef, see

https://lists.osgeo.org/pipermail/osgeo4w-dev/2018-August/003629.html

for closing this ticket, setup.hints has to be updated too.

comment:16 in reply to:  15 Changed 15 months ago by hellik

Replying to hellik:

ply is now in OSGeo4W thanks to jef, see

https://lists.osgeo.org/pipermail/osgeo4w-dev/2018-August/003629.html

for closing this ticket, setup.hints has to be updated too.

added by trunk: r73185,releasebranch_7_4: r73187

please test with the next daily builds

comment:17 Changed 15 months ago by martinl

I have installed python-ply in packaging environment. It should appear in the next daily build.

comment:18 Changed 14 months ago by neteler

Can the ticket be closed?

comment:19 Changed 14 months ago by martinl

Resolution: fixed
Status: newclosed

Tested daily build at https://wingrass.fsv.cvut.cz/grass74/x86_64/. Seems to be OK.

Note: See TracTickets for help on using tickets.