On Mon, Apr 24, 2006 at 07:11:09PM +0200, Bernd Ernesti wrote: >Forwared because my original mail was stopper by the subscriber check. > >----- Forwarded message from Bernd Ernesti <veego@xxxxxxxxxx> ----- > >Date: Sun, 23 Apr 2006 12:06:52 +0200 >From: Bernd Ernesti <veego@xxxxxxxxxx> >Subject: Problem with 4.5.99.903 on NetBSD >To: devel@xxxxxxxxxxx > >Hi, > >I send David Dawes a mail last month that there was a problem with >his change #231. >Not all problems were fixed since then. I don't recall it -- it probably got lost amongst the spam. >The remaining problem is that you can't use >#define InstallManPageSource NO >in host.def. > >This would result in an error during the cleaning stage of a make World: > >cleaning in programs/Xserver/hw/xfree86... >make: "/xc/programs/Xserver/hw/xfree86/Makefile" line 1120: Missing dependency operator >make: Fatal errors encountered -- cannot continue > >These are the lines around 1120, where 1120 is the InstallManPageLongBase one: > >> cleandir:: >> $(RM) XF86Config.$(MANNEWSUFFIX) >> >> InstallManPageLongBase(XF86Config,$(FILEMANDIR),XF86Config,$(FILEMANSUFFIX)) >> >> install:: > >Looking further into the NetBSD.cf changes made be belive that the >change in 3.130 contains some problems: > >+#define InstallGenManPageLong(file,destdir,dest,suffix) @@\ >+BuildInstallHtmlManPage(file,dest,suffix) @@\ >+ @@\ >+CppManTarget(file, $(EXTRAMANDEFS)) @@\ >+ @@\ >+InstallManPageLongBase(file,destdir,dest,suffix) > >and > >-InstallManPageAliasesBase(file,destdir,aliases) >+InstallManPageAliasesBase(file,destdir,suffix,aliases) > >IMHO there is an 'Os' prefix missing for InstallManPageLongBase and >InstallManPageAliasesBase. I agree. I'll fix these. >Attached is an NetBSD.cf to sync it with the one used in our cvs repository: >- Added some brackets to make it easier to read and changed some logic > for the NetBSD version check >- Adding support for HasArc4Random >and a fix for the InstallManPageSource problem which seems to work for me. > >I just did a complete new build, while using InstallManPageSource set to NO. >And the unformated manpages where not installed. Given where we are in the release cycle, I'm inclined to leave the other changes, until after 4.6.0. Thanks for the feedback and patch. David _______________________________________________ Devel@xxxxxxxxxxx http://XFree86.Org/mailman/listinfo/devel
