[GRASS-dev] Re: photo.* and description.html

[Markus Neteler]

(moving to list)

On Sun, May 3, 2009 at 9:54 AM, Hamish <hamish_b at yahoo.com> wrote:
>
>> in GRASS 6 I get:
>>
>> Following modules are missing the 'description.html' file
>> in src code:
>> photo.elev
>> photo.target
>>
>> Can we blacklist them in tools//build_html_index.sh? Or bad
>> side effects?
>
>
> both those have skip-me overrides in their Makefiles:
> ------
> htmletc:
>        @echo no docs needed for $(PGM)
> ------

That seems to be ignored here? But yes, see below:

> as recently as 6.4.0rc3 they built correctly without generating a
> photo.*.tmp.html file and "Following modules are missing the
> 'description.html' file" complaints.
>
>
> this is the result of r36693 for Module.make
>   http://trac.osgeo.org/grass/changeset/36693
>
> specifically
>  $(MAKE)  htmletc
> + $(MAKE)  manetc
>
>
> htmletc is bypassed but manetc is not.

I see.

> I am not sure if manetc: should check if htmletc: was run, or if the
> Makefiles should override both htmletc: and manetc:, or something else.

Since manetc depends on a previous htmletc run, it should perhaps
check (not sure how).

> partly because I'm not sure how the new manetc logic works: does it now
> build each man page as it goes,

Yes, in case there is only one (so not for r.mapcalc and a few others).

> in addition to rebuilding them after
> the end of the compile? or is that only meant for individual module
> rebuilds sometime later?

The global man creation catches all left-over cases such as
r.mapcalc/r3.mapcalc.

> because the Makefiles already say "skip me" I think fixing the build
> toolchain is more correct than putting them on the build_html_index
> bypass list and ignoring the problem.

OK - so we need more magic in the man part.

>
> RC3:
> ------
> photo.elev$ make
> Makefile:15: warning: overriding commands for target `htmletc'
> .../../../include/Make/Html.make:53: warning: ignoring old commands for target `htmletc'
> make  htmletc
> make[1]: Entering directory `/usr/local/src/grass/grass-6.4.0RC3/imagery/i.ortho.photo/photo.elev'
> Makefile:15: warning: overriding commands for target `htmletc'
> .../../../include/Make/Html.make:53: warning: ignoring old commands for target `htmletc'
> no docs needed for photo.elev
> make[1]: Leaving directory `/usr/local/src/grass/grass-6.4.0RC3/imagery/i.ortho.photo/photo.elev'
> ------
>
> RC4:
> ------
> photo.elev$ make
> Makefile:15: warning: overriding commands for target `htmletc'
> .../../../include/Make/Html.make:53: warning: ignoring old commands for target `htmletc'
> make  htmletc
> make[1]: Entering directory `/usr/local/src/grass/grass-6.4.0RC4/imagery/i.ortho.photo/photo.elev'
> Makefile:15: warning: overriding commands for target `htmletc'
> .../../../include/Make/Html.make:53: warning: ignoring old commands for target `htmletc'
> no docs needed for photo.elev
> make[1]: Leaving directory `/usr/local/src/grass/grass-6.4.0RC4/imagery/i.ortho.photo/photo.elev'
> make  manetc
> make[1]: Entering directory `/usr/local/src/grass/grass-6.4.0RC4/imagery/i.ortho.photo/photo.elev'
> Makefile:15: warning: overriding commands for target `htmletc'
> .../../../include/Make/Html.make:53: warning: ignoring old commands for target `htmletc'
> make /usr/local/src/grass/grass-6.4.0RC4/dist.i686-pc-linux-gnu/man/man1/photo.elev.1 MANSRC=/usr/local/src/grass/grass-6.4.0RC4/dist.i686-pc-linux-gnu/docs/html/photo.elev.html
> make[2]: Entering directory `/usr/local/src/grass/grass-6.4.0RC4/imagery/i.ortho.photo/photo.elev'
> Makefile:15: warning: overriding commands for target `htmletc'
> .../../../include/Make/Html.make:53: warning: ignoring old commands for target `htmletc'
> make[2]: `/usr/local/src/grass/grass-6.4.0RC4/dist.i686-pc-linux-gnu/man/man1/photo.elev.1' is up to date.
> make[2]: Leaving directory `/usr/local/src/grass/grass-6.4.0RC4/imagery/i.ortho.photo/photo.elev'
> make[1]: Leaving directory `/usr/local/src/grass/grass-6.4.0RC4/imagery/i.ortho.photo/photo.elev'
> ------

Yes, it was added after RC3 to simplify the installation of GRASS Addons
(all I have implemented to have a local "make install" target which is
convenient
for Addons and local recompilations in a production system).

Markus