IcedTea-Web add --mandir switch not honored

[Omair Majid]

On 02/18/2011 08:24 AM, Dr Andrew John Hughes wrote:
> On 18:37 Thu 17 Feb     , DJ Lucas wrote:
>> Trivial fix:
>>
>> --- icedtea-web-1.0.1-orig/Makefile.in    2011-02-14 13:04:17.000000000
>> -0600
>> +++ icedtea-web-1.0.1/Makefile.in    2011-02-17 18:32:46.000000000 -0600
>> @@ -626,8 +626,8 @@ install-exec-local:
>>    @ENABLE_PLUGIN_TRUE@    ${INSTALL_DATA}
>> $(abs_top_builddir)/liveconnect/lib/classes.jar
>> $(DESTDIR)$(prefix)$(JRE_DIR_PREFIX)/lib/plugin.jar
>>
>>    install-data-local:
>> -    ${mkinstalldirs} -d $(DESTDIR)$(prefix)/man/man1
>> -    ${INSTALL_DATA} $(NETX_SRCDIR)/javaws.1 $(DESTDIR)$(prefix)/man/man1
>> +    ${mkinstalldirs} -d $(DESTDIR)$(mandir)/man1
>> +    ${INSTALL_DATA} $(NETX_SRCDIR)/javaws.1 $(DESTDIR)$(mandir)/man1
>>        ${INSTALL_DATA} $(NETX_RESOURCE_DIR)/about.jnlp
>> $(DESTDIR)$(prefix)$(JRE_DIR_PREFIX)/lib
>>    @ENABLE_DOCS_TRUE@    ${mkinstalldirs} $(DESTDIR)$(htmldir)
>>    @ENABLE_DOCS_TRUE@    (cd ${abs_top_builddir}/docs/netx; \
>>

Thanks for the patch!

>
> Ok for HEAD.  For 1.0, it was agreed to retain the JDK directory structure as
> that version requires it.  Hence the deliberate ignorance of $(mandir).

I have pushed a slightly modified version of this patch to IcedTea-Web 
HEAD [1].

Thanks,
Omair

[1] http://icedtea.classpath.org/hg/icedtea-web/rev/87d12ebb7047