Re: PR/53103 CVS commit: src/sys

[christos%zoulas.com@localhost (Christos Zoulas)]

The following reply was made to PR lib/53103; it has been noted by GNATS.

From: christos%zoulas.com@localhost (Christos Zoulas)
To: Izumi Tsutsui <tsutsui%ceres.dti.ne.jp@localhost>
Cc: gnats-bugs%NetBSD.org@localhost, pullup-8%netbsd.org@localhost
Subject: Re: PR/53103 CVS commit: src/sys
Date: Sun, 23 Sep 2018 22:46:34 -0400

 On Sep 24, 11:29am, tsutsui%ceres.dti.ne.jp@localhost (Izumi Tsutsui) wrote:
 -- Subject: Re: PR/53103 CVS commit: src/sys
 
 | > I guess we should bump the version to:
 | > 
 | > #define        __NetBSD_Version__      800000100       /* NetBSD 8.00.01 */
 | > 
 | > and make sure that the change is pulled up to all the -8 branches.
 | 
 | I'm afraid this fails because the version strings in the comment
 | at the line is used as uname(3)'s version ("osrelease[] in the kernel)
 | by sys/conf/osrelease.sh and sys/conf/newvers.sh.
 | 
 | On the other hand osrelease.sh also claims:
 | 
 |  https://nxr.netbsd.org/xref/src/sys/conf/osrelease.sh?r=1.122#39
 | >> # Search for line
 | >> # #define __NetBSD_Version__ <ver_num> /* NetBSD <ver_text> */
 | >> #
 | >> # <ver_num> and <ver_text> should match!
 | 
 | Is there any document how __NetBSD__ in param.h should be updated,
 | especially for release branches?
 
 I am not sure how it fails. It prints:
 8.0.01
 
 It is either that or 8.01... I always intended that the pp was reserved
 for patches :-)
 
 christos