<div dir="ltr"><div><br><br>On Fri, Feb 16, 2018 at 12:47 PM, Moritz Lennert <<a href="mailto:mlennert@club.worldonline.be">mlennert@club.worldonline.be</a>> wrote:<br>><br>> Moving this over to grass-dev...<br>><br>> On 16/02/18 10:14, Markus Metz wrote:<br>>><br>>><br>>><br>>> On Fri, Feb 16, 2018 at 10:05 AM, Markus Neteler <<a href="mailto:neteler@osgeo.org">neteler@osgeo.org</a> <mailto:<a href="mailto:neteler@osgeo.org">neteler@osgeo.org</a>>> wrote:<br>>>  ><br>>>  > On Fri, Feb 16, 2018 at 9:42 AM, Markus Metz<br>>>  > <<a href="mailto:markus.metz.giswork@gmail.com">markus.metz.giswork@gmail.com</a> <mailto:<a href="mailto:markus.metz.giswork@gmail.com">markus.metz.giswork@gmail.com</a>>> wrote:<br>>>  > ...<br>>>  > > I can't reproduce with your compiler flags. A wild guess: in draw.c:L3 try<br>>>  > > to replace inline with static<br>>>  ><br>>>  > Yes, this helps. I didn't submit the change, will you do that?<br>>><br>>> done in r72247<br>><br>><br>><br>> Even though everything seems to install alright, I get an error message:<br>><br>> g.extension r.pi<br>> Fetching <r.pi> from GRASS GIS Addons repository (be patient)...<br>> Compiling...<br>> Installing...<br>> make[1]: *** No rule to make target 'install'.  Stop.<br>> Updating addons metadata file...<br>> Installation of <r.pi> successfully finished<br>><br>> I imagine this due to the fact that I run GRASS directly from the source directory and not from ?<br><br></div>It seems that there is no mechanism to install a library as addon. It is also possible that something is missing in the Makefile for r.pi.library because that library is unknown to the GRASS make system.<br><div>><br>> But when I try to install it using the system-wide installation (Debian package, grass-dev is installed), I get:<br>><br>> g.extension r.pi<br>> Fetching <r.pi> from GRASS GIS Addons repository (be patient)...<br>> Compiling...<br>> /usr/bin/ld: cannot open output file /usr/lib/grass74/lib/<a href="http://libgrass_rpi.7.4.0.so">libgrass_rpi.7.4.0.so</a>: Permission denied<br>> collect2: error: ld returned 1 exit status<br>> make[1]: *** [/usr/lib/grass74/lib/<a href="http://libgrass_rpi.7.4.0.so">libgrass_rpi.7.4.0.so</a>] Error 1<br>> /bin/sh: 1: cannot create /usr/lib/grass74/error.log: Permission denied<br>> make: *** [r.pi.library] Error 2<br>> ERROR: Compilation failed, sorry. Please check above error messages.<br>><br>> i.e. it tries to install the lib into the system-wide installation directory, not the user .grass7/addons directory.<br><br></div><div>the addons directory is supposed to hold executables and scripts, not libraries. At least I could not find a mechanism in the grass startup script where LD_LIBRARY_PATH is adjusted to also include some directory within the addons directory. I am not sure if it is worth the trouble to modify the addons mechanism to support addon libraries, particularly if only the r.pi suite is affected.<br><br></div><div>Markus M<br><br></div><div>><br>> Removing also doesn't seem to work:<br>><br>> g.extension r.pi op=remove -f<br>> ERROR: Unable to remove file<br>>        '/home/mlennert/.grass7/addons/docs/man/man1/man1/r.pi.corr.mw.1'<br>> ERROR: Unable to remove file<br>>        '/home/mlennert/.grass7/addons/docs/man/man1/man1/r.pi.lm.1'<br>> ERROR: Unable to remove file<br>>        '/home/mlennert/.grass7/addons/docs/man/man1/man1/r.pi.grow.1'<br>> ERROR: Unable to remove file<br>>        '/home/mlennert/.grass7/addons/docs/man/man1/man1/r.pi.rectangle.1'<br>> Updating addons metadata file...<br>> Extension <r.pi> successfully uninstalled.<br>><br>> but<br>><br>> ls .grass7/addons/bin/r.pi*<br>> .grass7/addons/bin/r.pi.corearea   .grass7/addons/bin/r.pi.graph .grass7/addons/bin/r.pi.nlm.circ<br>> .grass7/addons/bin/<a href="http://r.pi.csr.mw">r.pi.csr.mw</a>     .grass7/addons/bin/r.pi.graph.dec .grass7/addons/bin/r.pi.nlm.stats<br>> .grass7/addons/bin/r.pi.energy     .grass7/addons/bin/<a href="http://r.pi.graph.pr">r.pi.graph.pr</a> .grass7/addons/bin/r.pi.odc<br>> .grass7/addons/bin/<a href="http://r.pi.energy.pr">r.pi.energy.pr</a>  .grass7/addons/bin/r.pi.graph.red .grass7/addons/bin/<a href="http://r.pi.prob.mw">r.pi.prob.mw</a><br>> .grass7/addons/bin/r.pi.enn        .grass7/addons/bin/r.pi.import .grass7/addons/bin/r.pi.prox<br>> .grass7/addons/bin/<a href="http://r.pi.enn.pr">r.pi.enn.pr</a>     .grass7/addons/bin/r.pi.index .grass7/addons/bin/r.pi.searchtime<br>> .grass7/addons/bin/r.pi.export     .grass7/addons/bin/r.pi.neigh .grass7/addons/bin/<a href="http://r.pi.searchtime.mw">r.pi.searchtime.mw</a><br>> .grass7/addons/bin/r.pi.fnn        .grass7/addons/bin/r.pi.nlm   .grass7/addons/bin/<a href="http://r.pi.searchtime.pr">r.pi.searchtime.pr</a><br>><br>> I guess we need some more work on how to handle these suites...<br>><br>> Moritz<br><br></div></div>