[GRASS-dev] [GRASS-SVN] r66413 - grass-addons/grass7/vector/v.kriging

Vaclav Petras wenzeslaus at gmail.com
Tue Oct 6 05:38:44 PDT 2015


On Tue, Oct 6, 2015 at 4:22 AM, Markus Neteler <neteler at osgeo.org> wrote:
>
> On Tue, Oct 6, 2015 at 10:03 AM, Moritz Lennert
> <mlennert at club.worldonline.be> wrote:
> > On 05/10/15 16:49, Vaclav Petras wrote:
> ...
> >> Sorry for being -pedantic but I don't think we should leave the code
> >> messy.
> >
> > +1
>
> Then it needs to be removed everywhere here:

What we need however, is to figure out why it is not working on that Debian
server. make distclean? svn revert? Dockerfile for Debian would be helpful
in this case to have reproducible environment.

> cd grass-addons
> find . -type f | grep -v pristin | xargs grep 'la.h>'
> ./grass6/imagery/i.spec.unmix/global.h:#include <grass/la.h>
> ./grass6/imagery/i.spec.unmix/la_extra.c:#include <grass/la.h>
> ./grass6/imagery/i.spec.grass63/include/gmath.h:#include <grass/la.h>
> ./grass6/imagery/i.spec.grass63/lib/gmath/la.c:#include <grass/la.h>
> ./grass7/imagery/i.spec.unmix/global.h:#include <grass/la.h>
> ./grass7/vector/v.kriging/local_proto.h:#include <grass/la.h>

I would remove it only in 7. It should work and it worked without it
(except for that Debian server). I don't know what is the situation in 6.
Let's not be bothered with code correctness for 6, the only priority there
is that the main code works and addons are ported to 7.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/grass-dev/attachments/20151006/9ecbab33/attachment.html>


More information about the grass-dev mailing list