Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > muc.lists.netbsd.source-changes > #156220 > unrolled thread

Re: CVS commit: src

Back to article view | Back to muc.lists.netbsd.source-changes

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: CVS commit: src Andrius V <vezhlys@gmail.com> - 2026-08-08 01:32 +0300
    Re: CVS commit: src Thor Lancelot Simon <tls@panix.com> - 2026-08-07 21:32 -0400

#156220 — Re: CVS commit: src

On Sun, May 10, 2026 at 5:19 PM Thor Lancelot Simon <tls@netbsd.org> wrote:
>
> Module Name:    src
> Committed By:   tls
> Date:           Sun May 10 14:19:11 UTC 2026
>
> Modified Files:
>         src/distrib/sets: sets.subr
>         src/distrib/sets/lists/debug: ad.mips ad.mipseb ad.mipsel md.emips
>             md.evbarm.earmv4 md.evbarm.earmv5 md.evbarm.earmv5eb md.evbppc
>             md.pmax mi
>         src/etc: Makefile
>         src/share/mk: bsd.own.mk
>         src/sys/arch/alpha/conf: INSTALL
>         src/sys/conf: Makefile.kern.inc
>
> Log Message:
> Fix MKDEBUGKERNEL vs MKDEBUG for kernel debug file sets.
>
> The problem manifests as checkflist failures when building ports that
> have extensive ALL_KERNELS but not...building all the kernels; notably
> the various "evb" ports with a bazillion kernels for a bazillion SoCs.
>
> The mk.conf(5) man page documents MKDEBUGKERNEL as controlling
> whether kernel debug files (netbsd-*.debug) appear in the
> distribution sets.  However, the prior implementation used MKDEBUG
> (the general userland debug flag) instead.
>
> This meant MKDEBUG=yes with MKDEBUGKERNEL=no incorrectly expected
> kernel debug files for every kernel config listed in ALL_KERNELS.
> When only a subset of kernels is built, checkflist fails with
> missing files.
>
> The fix is to make these variables fully conform to the longstanding
> documentation.  MKDEBUGKERNEL controls whether kernel debug symbols are
> built; MKDEBUG controls everything else.  If you want something like the
> old behavior but minus the bugs, set both.
>
>
> To generate a diff of this commit:
> cvs rdiff -u -r1.214 -r1.215 src/distrib/sets/sets.subr
> cvs rdiff -u -r1.87 -r1.88 src/distrib/sets/lists/debug/ad.mips
> cvs rdiff -u -r1.7 -r1.8 src/distrib/sets/lists/debug/ad.mipseb \
>     src/distrib/sets/lists/debug/ad.mipsel
> cvs rdiff -u -r1.3 -r1.4 src/distrib/sets/lists/debug/md.emips
> cvs rdiff -u -r1.1 -r1.2 src/distrib/sets/lists/debug/md.evbarm.earmv4 \
>     src/distrib/sets/lists/debug/md.evbarm.earmv5 \
>     src/distrib/sets/lists/debug/md.evbarm.earmv5eb
> cvs rdiff -u -r1.5 -r1.6 src/distrib/sets/lists/debug/md.evbppc
> cvs rdiff -u -r1.4 -r1.5 src/distrib/sets/lists/debug/md.pmax
> cvs rdiff -u -r1.512 -r1.513 src/distrib/sets/lists/debug/mi
> cvs rdiff -u -r1.475 -r1.476 src/etc/Makefile
> cvs rdiff -u -r1.1481 -r1.1482 src/share/mk/bsd.own.mk
> cvs rdiff -u -r1.120 -r1.121 src/sys/arch/alpha/conf/INSTALL
> cvs rdiff -u -r1.305 -r1.306 src/sys/conf/Makefile.kern.inc
>
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.
>

Hi,

I believe this commit fixes PR 59233 and PR 57846 (MKDEBUG doesn't
fail on current now for evbmips release, NetBSD 11 still does).

Not sure how important is to pullup these changes to netbsd-10 or
netbsd-11? Or should we just close those tickets?

Regards,
Andrius V

--
Posted automagically by a mail2news gateway at muc.de e.V.
Please direct questions, flames, donations, etc. to news-admin@muc.de

[toc] | [next] | [standalone]

#156221

My intention is to put in pullup requests for most of my bug fixes that
went into -current late in the NetBSD-11 release cycle now that NetBSD 11
has been released.  It will take me a few weeks.

Thor

On Sat, Aug 08, 2026 at 01:32:24AM +0300, Andrius V wrote:
> On Sun, May 10, 2026 at 5:19???PM Thor Lancelot Simon <tls@netbsd.org> wrote:
> >
> > Module Name:    src
> > Committed By:   tls
> > Date:           Sun May 10 14:19:11 UTC 2026
> >
> > Modified Files:
> >         src/distrib/sets: sets.subr
> >         src/distrib/sets/lists/debug: ad.mips ad.mipseb ad.mipsel md.emips
> >             md.evbarm.earmv4 md.evbarm.earmv5 md.evbarm.earmv5eb md.evbppc
> >             md.pmax mi
> >         src/etc: Makefile
> >         src/share/mk: bsd.own.mk
> >         src/sys/arch/alpha/conf: INSTALL
> >         src/sys/conf: Makefile.kern.inc
> >
> > Log Message:
> > Fix MKDEBUGKERNEL vs MKDEBUG for kernel debug file sets.
> >
> > The problem manifests as checkflist failures when building ports that
> > have extensive ALL_KERNELS but not...building all the kernels; notably
> > the various "evb" ports with a bazillion kernels for a bazillion SoCs.
> >
> > The mk.conf(5) man page documents MKDEBUGKERNEL as controlling
> > whether kernel debug files (netbsd-*.debug) appear in the
> > distribution sets.  However, the prior implementation used MKDEBUG
> > (the general userland debug flag) instead.
> >
> > This meant MKDEBUG=yes with MKDEBUGKERNEL=no incorrectly expected
> > kernel debug files for every kernel config listed in ALL_KERNELS.
> > When only a subset of kernels is built, checkflist fails with
> > missing files.
> >
> > The fix is to make these variables fully conform to the longstanding
> > documentation.  MKDEBUGKERNEL controls whether kernel debug symbols are
> > built; MKDEBUG controls everything else.  If you want something like the
> > old behavior but minus the bugs, set both.
> >
> >
> > To generate a diff of this commit:
> > cvs rdiff -u -r1.214 -r1.215 src/distrib/sets/sets.subr
> > cvs rdiff -u -r1.87 -r1.88 src/distrib/sets/lists/debug/ad.mips
> > cvs rdiff -u -r1.7 -r1.8 src/distrib/sets/lists/debug/ad.mipseb \
> >     src/distrib/sets/lists/debug/ad.mipsel
> > cvs rdiff -u -r1.3 -r1.4 src/distrib/sets/lists/debug/md.emips
> > cvs rdiff -u -r1.1 -r1.2 src/distrib/sets/lists/debug/md.evbarm.earmv4 \
> >     src/distrib/sets/lists/debug/md.evbarm.earmv5 \
> >     src/distrib/sets/lists/debug/md.evbarm.earmv5eb
> > cvs rdiff -u -r1.5 -r1.6 src/distrib/sets/lists/debug/md.evbppc
> > cvs rdiff -u -r1.4 -r1.5 src/distrib/sets/lists/debug/md.pmax
> > cvs rdiff -u -r1.512 -r1.513 src/distrib/sets/lists/debug/mi
> > cvs rdiff -u -r1.475 -r1.476 src/etc/Makefile
> > cvs rdiff -u -r1.1481 -r1.1482 src/share/mk/bsd.own.mk
> > cvs rdiff -u -r1.120 -r1.121 src/sys/arch/alpha/conf/INSTALL
> > cvs rdiff -u -r1.305 -r1.306 src/sys/conf/Makefile.kern.inc
> >
> > Please note that diffs are not public domain; they are subject to the
> > copyright notices on the relevant files.
> >
> 
> Hi,
> 
> I believe this commit fixes PR 59233 and PR 57846 (MKDEBUG doesn't
> fail on current now for evbmips release, NetBSD 11 still does).
> 
> Not sure how important is to pullup these changes to netbsd-10 or
> netbsd-11? Or should we just close those tickets?
> 
> Regards,
> Andrius V

-- 
Thor Lancelot Simon	                                     tls@panix.com
  "Indeed, someone who is a citizen in a democracy is often not one in
   an oligarchy."
					- Aristotle, Politics III.1

--
Posted automagically by a mail2news gateway at muc.de e.V.
Please direct questions, flames, donations, etc. to news-admin@muc.de

[toc] | [prev] | [standalone]

Back to top | Article view | muc.lists.netbsd.source-changes

csiph-web