Opened 7 years ago

Closed 4 years 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 by martinl, 7 years ago

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

in reply to:  1 ; comment:2 by wenzeslaus, 7 years ago

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 by neteler, 6 years ago

Milestone: 7.1.07.2.0

Milestone renamed

comment:4 by neteler, 6 years ago

Milestone: 7.2.07.2.1

Ticket retargeted after milestone closed

comment:5 by martinl, 5 years ago

Milestone: 7.2.17.2.2

comment:6 by neteler, 5 years ago

Milestone: 7.2.27.2.3

Ticket retargeted after milestone closed

comment:7 by martinl, 4 years ago

Milestone: 7.2.3

Ticket retargeted after milestone closed

comment:8 by martinl, 4 years ago

Milestone: 7.2.4

comment:9 by neteler, 4 years ago

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

comment:10 by neteler, 4 years ago

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.

in reply to:  10 comment:11 by hellik, 4 years ago

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

in reply to:  2 ; comment:12 by hellik, 4 years ago

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 4 years ago by hellik (previous) (diff)

comment:13 by martinl, 4 years ago

Milestone: 7.2.47.4.2

comment:14 by martinl, 4 years ago

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

in reply to:  12 ; comment:15 by hellik, 4 years ago

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.

in reply to:  15 comment:16 by hellik, 4 years ago

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 by martinl, 4 years ago

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

comment:18 by neteler, 4 years ago

Can the ticket be closed?

comment:19 by martinl, 4 years ago

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.