mandir should be prefixed with $(prefix)

[GoogleCodeExporter]

commit 95 adds mandir to the Makefile, but missing $(prefix). Please add this 
to allow non "/" installations.

Original issue reported on code.google.com by ju.lec...@googlemail.com on 24 Jul 2012 at 6:27

[GoogleCodeExporter]

i did

prefix ?= /usr
-mandir ?= /usr/share/man/
+mandir ?= $(prefix)/share/man/

is this ok?

thanks

Original comment by abhirana on 25 Jul 2012 at 2:04

[GoogleCodeExporter]

Thats good, and in addition we need this

-        install -m 0644 doc/i7z.man $(mandir)man1/i7z.1
+        install -m 0644 doc/i7z.man $(DESTDIR)$(mandir)man1/i7z.1

Original comment by ju.lec...@googlemail.com on 25 Jul 2012 at 5:56

[GoogleCodeExporter]

updated in svn :)

Original comment by abhirana on 25 Jul 2012 at 6:16

[GoogleCodeExporter]

one little thing. we need to call install with "-D" to create missing 
directories.

Original comment by ju.lec...@googlemail.com on 26 Jul 2012 at 6:07

[GoogleCodeExporter]

sure

you meant -d right?

i added
install -d $(DESTDIR)$(sbindir) $(DESTDIR)$(docdir) $(DESTDIR)$(mandir)man1

thanks

Original comment by abhirana on 27 Jul 2012 at 2:20

[GoogleCodeExporter]

not exactly. You can pass -D (capital) which will do:

"create all leading components of DEST except the last, then copy SOURCE to 
DEST"

Original comment by ju.lec...@googlemail.com on 27 Jul 2012 at 6:07

[GoogleCodeExporter]

thanks. 

fixed (i hope) in the latest update.

Original comment by abhirana on 27 Jul 2012 at 6:44

[GoogleCodeExporter]

Original comment by abhirana on 13 Aug 2012 at 10:34

• Changed state: Fixed