Hi Glynn,

On Thu, 5 Nov 2009, Glynn Clements wrote:

> 
> Raymond Burns wrote:
> 
> > I was looking for something else when I noticed this.
> > 
<snip>

> 
> What problem is this supposed to fix?
> 

ERROR: MAPSET PERMANENT - permission denied

346 times on the current build.

In previous cvs pulls, each make iteration rebuilt the html pages and
presented this message which leaves the lasting impression the target
did not get made correctly and therefore may have left targets beyond
it in the makefile un made.

The most recent cvs pull appears to recognise the completed target so
the message occurs only on the first make iteration.

> Also, I can't see how the patched version can work, as nothing tries
> to make the "htmldesc" target.

It did not, this as a revision I believe does, though it does not
paste well either. At this point, if there is possible useful output
from the execution of the code with --html-description there may be
reason to ignore the patch and retain the message, if there is no
other useful output of the --html-description I would just as soon not
see an implied permission violation. On a new platform. I get enough
of those that are real.

diff -u include/Make/Html.make.orig include/Make/Html.make                      
  
--- include/Make/Html.make.orig Tue Nov 10 09:18:51 2009
+++ include/Make/Html.make      Tue Nov 10 09:18:57 2009
@@ -34,7 +34,7 @@
        GISBASE=$(RUN_GISBASE) \
        PATH="$(BIN):$$PATH" \

$(LD_LIBRARY_PATH_VAR)="$(BIN):$(ARCH_LIBDIR):$($(LD_LIBRARY_PATH_VAR))"
\
-       LC_ALL=C $(1) --html-description < /dev/null | grep -v
'</body>\|</html>' > $(PGM).tmp.html ; true
+       LC_ALL=C $(1) --html-description < /dev/null 2>/dev/null |
grep -v '</body>\|</html>' > $(PGM).tmp.html ; true
 
 $(HTMLDIR)/$(PGM).html: $(HTMLSRC)
        if [ "$(HTMLSRC)" != "" ] ; then $(call htmldesc,$<) ; fi

 
> 
> -- 
> Glynn Clements <[email protected]>
> 

Thanx
  Ray

--- include/Make/Html.make.orig Tue Nov 10 09:18:51 2009
+++ include/Make/Html.make      Tue Nov 10 09:18:57 2009
@@ -34,7 +34,7 @@
        GISBASE=$(RUN_GISBASE) \
        PATH="$(BIN):$$PATH" \
        
$(LD_LIBRARY_PATH_VAR)="$(BIN):$(ARCH_LIBDIR):$($(LD_LIBRARY_PATH_VAR))" \
-       LC_ALL=C $(1) --html-description < /dev/null | grep -v 
'</body>\|</html>' > $(PGM).tmp.html ; true
+       LC_ALL=C $(1) --html-description < /dev/null 2>/dev/null | grep -v 
'</body>\|</html>' > $(PGM).tmp.html ; true
 
 $(HTMLDIR)/$(PGM).html: $(HTMLSRC)
        if [ "$(HTMLSRC)" != "" ] ; then $(call htmldesc,$<) ; fi
_______________________________________________
grass-dev mailing list
[email protected]
http://lists.osgeo.org/mailman/listinfo/grass-dev

Reply via email to