make {,re}installkernel{,.debug}

2009-07-09 Thread Brian Somers
 target install, the targets
@@ -269,10 +273,6 @@ realinstall: _kmodinstall
 _kmodinstall:
${INSTALL} -o ${KMODOWN} -g ${KMODGRP} -m ${KMODMODE} \
${_INSTALLFLAGS} ${PROG} ${DESTDIR}${KMODDIR}
-.if defined(DEBUG_FLAGS) && !defined(INSTALL_NODEBUG)
-   ${INSTALL} -o ${KMODOWN} -g ${KMODGRP} -m ${KMODMODE} \
-   ${_INSTALLFLAGS} ${PROG}.symbols ${DESTDIR}${KMODDIR}
-.endif
 
 .include 
 
@@ -290,6 +290,16 @@ _kldxref:
 
 .endif # !target(install)
 
+.if !target(install.debug)
+install.debug:
+.if defined(DEBUG_FLAGS)
+   ${INSTALL} -o ${KMODOWN} -g ${KMODGRP} -m ${KMODMODE} \
+   ${_INSTALLFLAGS} ${PROG}.symbols ${DESTDIR}${KMODDIR}
+.endif
+.endif # !target(install.debug)
+
+install.debug: install
+
 .if !target(load)
 load: ${PROG}
${KMODLOAD} -v ${.OBJDIR}/${PROG}
Index: UPDATING
===
--- UPDATING(revision 195527)
+++ UPDATING(working copy)
@@ -22,6 +22,11 @@ NOTE TO PEOPLE WHO THINK THAT FreeBSD 8.x IS SLOW:
to maximize performance.  (To disable malloc debugging, run
ln -s aj /etc/malloc.conf.)
 
+20090709:
+   The use of INSTALL_NODEBUG has been deprecated and symbol files
+   are no longer installed by default.  To install kernel and
+   module symbol files, use the {,re}installkernel.debug target.
+
 20090630:
The NFS_LEGACYRPC option has been removed along with the old
kernel RPC implementation that this option selected. Kernel
Index: Makefile
===
--- Makefile(revision 195527)
+++ Makefile(working copy)
@@ -11,9 +11,9 @@
 # world   - buildworld + installworld, no kernel.
 # buildkernel - Rebuild the kernel and the kernel-modules.
 # installkernel   - Install the kernel and the kernel-modules.
-# installkernel.debug
+# installkernel.debug - Also install kernel and module symbols.
 # reinstallkernel - Reinstall the kernel and the kernel-modules.
-# reinstallkernel.debug
+# reinstallkernel.debug - Also reinstall kernel and module symbols.
 # kernel  - buildkernel + installkernel.
 # doxygen - Build API documentation of the kernel, needs doxygen.
 # update  - Convenient way to update your source tree (cvs).
Index: Makefile.inc1
===
--- Makefile.inc1   (revision 195527)
+++ Makefile.inc1   (working copy)
@@ -682,6 +682,9 @@ distrib-dirs distribution:
 # for building kernels and only the first of these is designated
 # as the one being installed.
 #
+# If your kernel is built with DEBUG set, installkernel.debug
+# will additionally install kernel and module symbols.
+#
 # Note that we have to use TARGET instead of TARGET_ARCH when
 # we're in kernel-land. Since only TARGET_ARCH is (expected) to
 # be set to cross-build, we have to make sure TARGET is set
Index: share/mk/bsd.subdir.mk
===
--- share/mk/bsd.subdir.mk  (revision 195516)
+++ share/mk/bsd.subdir.mk  (working copy)
@@ -67,7 +67,7 @@ ${SUBDIR}: .PHONY
 
 .for __target in all all-man checkdpadd clean cleandepend cleandir \
 depend distribute lint maninstall manlint \
-obj objlink realinstall regress tags \
+obj objlink realinstall install.debug regress tags \
 ${SUBDIR_TARGETS}
 ${__target}: _SUBDIR
 .endfor


-- 
Brian Somers  
Don't _EVER_ lose your sense of humour !   


signature.asc
Description: PGP signature


Re: What's changed between 7.1 and 7.2

2009-07-09 Thread Nick Hibma
The debugging printf has been changed to not print anything if debuglevel is 
0.

Nick

> On 2009-07-08 19:28, Robert Watson wrote:
> > This is effectively debugging output that slipped into the release and
> > shouldn't have.  I believe it's now removed in 8.x, I'm not sure it's
> > been MFC'd to 7.x yet.
>
> It was indeed removed, with r193131.  The diff here:
>
> http://svn.freebsd.org/viewvc/base/head/sys/geom/label/g_label.c?view=pat
>ch&r1=193131&r2=193130&pathrev=193131
>
> should apply to 7.x without problems.
> ___
> freebsd-hackers@freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-hackers
> To unsubscribe, send any mail to
> "freebsd-hackers-unsubscr...@freebsd.org"



___
freebsd-hackers@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-hackers
To unsubscribe, send any mail to "freebsd-hackers-unsubscr...@freebsd.org"