Opened 12 years ago
Closed 12 years ago
#4629 closed defect (fixed)
`make man` no longer works due to presence of man directory in source
Reported by: | Sharpie | Owned by: | warmerdam |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | ConfigBuild | Version: | 1.9.0 |
Severity: | normal | Keywords: | |
Cc: |
Description
Because there is a man
directory in the GDAL 1.9.0 source, make man
does not work:
bash-3.2$ make man make: `man' is up to date.
Because man
exists as a directory, make
assumes the "target" has been built and does not execute the build steps for the rule.
Either:
- The
man
directory must be removed from the source distribution.
man
must be marked as a.PHONY
target inGNUmakefile
. This is probably a good idea in general.
Change History (4)
follow-up: 2 comment:1 by , 12 years ago
Component: | default → ConfigBuild |
---|---|
Status: | new → assigned |
comment:2 by , 12 years ago
Replying to warmerdam:
If the man pages are decent we should likely distribute them and ensure make install installs them as well as adding the .PHONY suggested.
That would be awesome. GDAL has a pile of dependencies---getting rid of Doxygen without sacrificing manpages would be very nice.
comment:3 by , 12 years ago
Grr. I prepared nice detailed notes on the changes I've applied, but the damn chrome undo/redo bullshit ate it and I won't rewrite it.
r24262, r24263, r24264, r24265, r24266, r24267
core changes backported to 1.9.
man page production should be smoother in the release process - should see this in 1.9.1.
comment:4 by , 12 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
It appears this directory is created in the dist files because mkgdaldist.sh does a "make docs" which creates it but does not populate it. If the man pages are decent we should likely distribute them and ensure make install installs them as well as adding the .PHONY suggested.
I'll look into this.