Re: [edbrowse-dev] documentation respecting install prefix in cmake

[Dominique Martinet <asmadeus@codewreck.org>]

Hi Geoff,

Geoff McLane wrote on Thu, Dec 26, 2019:5B
> I started to look around at various other projects I can compile with CMake...
> find a lot of sometimes confusing, even contradictory... information...

That seem to be a recurring problem with build systems as new versions
come out...
If the MAN_INSTALL_DIR and co have been defined for enough versions I
think it is better, but it does not look like a default variable to me,
something to do with KDE install dirs perhaps?
At this point I think the ideal solution would be using GNUInstallDirs's
MANDIR and DOCDIR as I suggested earlier, but the safest would be as
little change as possible like I had done.


> why do we have 2 slightly different UnixManFile... debian vs freebsd...
> are the man app implementations /SO/ different... in a quick review, seems
> they could be one...

That is a good question, it looks like Alfonso Siciliano preferred
adding a freebsd man page to using the debian one from the git log?
I have added him in exlpicit CCs to this mail.

> And they could use like @EDB_VERSION@, @VER_DATE@, etc... even say a
> @DOC_INSTALL@, for the usersguide.html pointer... etc... etc...
>
> It seems the cmake default CMAKE_INSTALL_PREFIX of "/usr/local" is
> generally agreed as the /ROOT/...
> 
> So others components are "/ROOT/bin, lib, include"
> 
> And then the man being "/ROOT/share/man/man1"...
> 
> But it looks like some distros prefer "/ROOT/man/man1", which
> would also be ok in my Ubuntu debian based linux, in that there is
> a link "man -> share/man" ... must look in my Raspian RPI...
> 
> Then the docs to "/ROOT/share/doc/<project>"...

Ah, I had missed that FreeBSD wanted the man page in /usr/local/man
(without /share/) in our CMakeLists.
Looking at the FreeBSD release notes[1] that are a bit difficult to
read, the man pages should go either to /usr/share/man if the prefix is
/usr but if it is /usr/local then it should go directly to
/usr/local/man for some reason... Odd... But that means my fix is
incorrect.
I just checked and the MANDIR variable from GNUInstallDirs has the
correct exception for all BSDs, so I will update my patch to use that
instead.

[1] https://www.freebsd.org/cgi/man.cgi?query=hier&apropos=0&sektion=0&manpath=FreeBSD+6.1-RELEASE&format=html

> But like I say, who am I to suggest anything in unix ;=))
> 
> Must have been too full of the Christmas cheer...

Enjoy the remaining holidays as well :)

-- 
Dominique