Re: CVS commit: src/lib/libc/time

2015-10-31 Thread Aleksej Saushev
"Brian Ginsbach" 
writes:

> Module Name:  src
> Committed By: ginsbach
> Date: Fri Oct 30 01:49:36 UTC 2015
>
> Modified Files:
>   src/lib/libc/time: strptime.c
>
> Log Message:
> Reject timezone offsets more than 12 hours (east or west).

WTF?? There exist (or existed) zones +13 and +14.


> To generate a diff of this commit:
> cvs rdiff -u -r1.51 -r1.52 src/lib/libc/time/strptime.c
>
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.
>
>
> Modified files:
>
>
> Index: src/lib/libc/time/strptime.c
> diff -u src/lib/libc/time/strptime.c:1.51 src/lib/libc/time/strptime.c:1.52
> --- src/lib/libc/time/strptime.c:1.51 Thu Oct 29 19:18:19 2015
> +++ src/lib/libc/time/strptime.c  Fri Oct 30 01:49:36 2015
> @@ -1,4 +1,4 @@
> -/*   $NetBSD: strptime.c,v 1.51 2015/10/29 19:18:19 christos Exp $   */
> +/*   $NetBSD: strptime.c,v 1.52 2015/10/30 01:49:36 ginsbach Exp $   */
>  
>  /*-
>   * Copyright (c) 1997, 1998, 2005, 2008 The NetBSD Foundation, Inc.
> @@ -31,7 +31,7 @@
>  
>  #include 
>  #if defined(LIBC_SCCS) && !defined(lint)
> -__RCSID("$NetBSD: strptime.c,v 1.51 2015/10/29 19:18:19 christos Exp $");
> +__RCSID("$NetBSD: strptime.c,v 1.52 2015/10/30 01:49:36 ginsbach Exp $");
>  #endif
>  
>  #include "namespace.h"
> @@ -626,6 +626,8 @@ loadzone:
>   default:
>   return NULL;
>   }
> + if (offs > (12 * SECSPERHOUR))
> + return NULL;
>   if (neg)
>   offs = -offs;
>   tm->tm_isdst = 0;   /* XXX */
>

-- 
HE CE3OH...



Re: CVS commit: src/usr.bin/sed

2015-03-01 Thread Aleksej Saushev
chris...@astron.com (Christos Zoulas) writes:

> In article <20150228215653.f2f0...@cvs.netbsd.org>,
> Aleksej Saushev  wrote:
>>-=-=-=-=-=-
>>
>>Module Name:  src
>>Committed By: asau
>>Date: Sat Feb 28 21:56:53 UTC 2015
>>
>>Modified Files:
>>  src/usr.bin/sed: compile.c extern.h main.c process.c
>>
>>Log Message:
>>Improve modularity of "sed" source:
>>- move program source input subroutines into compiler part;
>>- move data I/O subroutines into processor part.
>
> It there a good reason for that? We try to keep it synced with FreeBSD..

It makes code a lot cleaner and allows embedding it.

If that's problem, I'll submit changes to FreeBSD.


-- 
HE CE3OH...



Re: CVS commit: src/share/misc

2013-06-03 Thread Aleksej Saushev
"Blue Rats"  writes:

> Module Name:  src
> Committed By: rodent
> Date: Sun Jun  2 16:37:37 UTC 2013
>
> Modified Files:
>   src/share/misc: domains
>
> Log Message:
> +sx Sint Maarten
> -yu Yugoslavia; deprecated on 2010-03-30 [1]

Have all addresses in that domain expired?

I think that it would be better to keep this information,
since HTML links may still exist (and they do most probably).

> [1]:
> http://web.archive.org/web/20100201171046/http://www.rnids.rs/en/node/67
>
>
> To generate a diff of this commit:
> cvs rdiff -u -r1.7 -r1.8 src/share/misc/domains

-- 
BECHA...
   CKOPO CE3OH...



Re: CVS commit: src/lib/libc/arch/arm/softfloat

2013-01-26 Thread Aleksej Saushev
"Matt Thomas"  writes:

> Module Name:  src
> Committed By: matt
> Date: Sat Jan 26 07:08:14 UTC 2013
>
> Modified Files:
>   src/lib/libc/arch/arm/softfloat: arm-gcc.h
>
> Log Message:
> Appease clang by making 64-bit literals use ULL
>
>
> To generate a diff of this commit:
> cvs rdiff -u -r1.3 -r1.4 src/lib/libc/arch/arm/softfloat/arm-gcc.h
>
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.
>
>
> Modified files:
>
>
> Index: src/lib/libc/arch/arm/softfloat/arm-gcc.h
> diff -u src/lib/libc/arch/arm/softfloat/arm-gcc.h:1.3 
> src/lib/libc/arch/arm/softfloat/arm-gcc.h:1.4
> --- src/lib/libc/arch/arm/softfloat/arm-gcc.h:1.3 Sat Dec 24 21:11:16 2005
> +++ src/lib/libc/arch/arm/softfloat/arm-gcc.h Sat Jan 26 07:08:14 2013
> @@ -1,4 +1,4 @@
> -/* $NetBSD: arm-gcc.h,v 1.3 2005/12/24 21:11:16 perry Exp $ */
> +/* $NetBSD: arm-gcc.h,v 1.4 2013/01/26 07:08:14 matt Exp $ */
>  
>  /*
>  
> ---
> @@ -71,7 +71,7 @@ name for the 64-bit integer type.  Some 
>  defined as the identity macro:  `#define LIT64( a ) a'.
>  
> ---
>  */
> -#define LIT64( a ) a##LL
> +#define LIT64( a ) a##ULL
>  #endif

That's a bit strange.
Don't we have UINT64_C for this?
Or is too early for UINT64_C to be defined?


-- 
HE CE3OH...



Re: CVS commit: src/share/mk

2013-01-20 Thread Aleksej Saushev
"Christos Zoulas" 
writes:

> Module Name:  src
> Committed By: christos
> Date: Thu Jan 17 15:42:59 UTC 2013
>
> Modified Files:
>   src/share/mk: bsd.prog.mk
>
> Log Message:
> Now that we have separate debug sets, add -g if ${MKDEBUG} is set for
> program builds since we don't need to install that symbol files.
>
>
> To generate a diff of this commit:
> cvs rdiff -u -r1.278 -r1.279 src/share/mk/bsd.prog.mk
>
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.
>
>
> Modified files:
>
>
> Index: src/share/mk/bsd.prog.mk
> diff -u src/share/mk/bsd.prog.mk:1.278 src/share/mk/bsd.prog.mk:1.279
> --- src/share/mk/bsd.prog.mk:1.278Fri Aug 24 16:26:24 2012
> +++ src/share/mk/bsd.prog.mk  Thu Jan 17 10:42:59 2013
> @@ -1,4 +1,4 @@
> -#$NetBSD: bsd.prog.mk,v 1.278 2012/08/24 20:26:24 jmmv Exp $
> +#$NetBSD: bsd.prog.mk,v 1.279 2013/01/17 15:42:59 christos Exp $
>  #@(#)bsd.prog.mk 8.2 (Berkeley) 4/2/94
>  
>  .ifndef HOSTPROG
> @@ -49,6 +49,9 @@ LDFLAGS+=   ${PIE_LDFLAGS}
>  .endif
>  
>  CFLAGS+= ${COPTS}
> +.if defined(MKDEBUG) && (${MKDEBUG} != "no")
> +CFLAGS+= -g
> +.endif
>  OBJCFLAGS+=  ${OBJCOPTS}
>  MKDEP_SUFFIXES?= .o .ln

Well...
What about "DBG"?

Now it isn't clear what the difference between DBG, COPTS, and CFLAGS is
at all.


-- 
HE CE3OH...



Re: CVS commit: [netbsd-5] src/share/misc

2012-10-23 Thread Aleksej Saushev
"Jeff Rizzo"  writes:

> Module Name:  src
> Committed By: riz
> Date: Wed Oct 24 03:08:09 UTC 2012
>
> Modified Files:
>   src/share/misc [netbsd-5]: bsd-family-tree
>
> Log Message:
> Pull up following revision(s) (requested by jnemeth in ticket #1804):
>   share/misc/bsd-family-tree: revision 1.45
> Sync with FreeBSD rev. 241698.
> Adds:
> - FreeBSD 8.32012-04-18 [FBD]
> - OpenBSD 5.12012-05-01 [OBD]
> - Mac OS X 10.82012-07-25 [APL]
> - NetBSD 6.02012-10-17 [NBD]
>
>
> To generate a diff of this commit:
> cvs rdiff -u -r1.40.12.2 -r1.40.12.3 src/share/misc/bsd-family-tree
>
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.
>
>
> Modified files:
>
>
> Index: src/share/misc/bsd-family-tree
> diff -u src/share/misc/bsd-family-tree:1.40.12.2 
> src/share/misc/bsd-family-tree:1.40.12.3
> --- src/share/misc/bsd-family-tree:1.40.12.2  Mon Jan 26 00:52:25 2009
> +++ src/share/misc/bsd-family-treeWed Oct 24 03:08:08 2012
> @@ -25,7 +25,7 @@ Seventh Edition (V7)|
>\/ |
>   3BSD|
>|  |
> -   4.0BSD   2.7.9BSD
> +   4.0BSD2.79BSD
>|  |
> 4.1BSD --> 2.8BSD
>|  |

Is the change of "2.7.9" to "2.79" here and below intended?

> @@ -62,9 +62,9 @@ Tenth Edition | 
> ||  4.4BSD|
> ||/ | |
> ||   4.4BSD-Encumbered  | |
> -   |   - NetBSD 0.8|   BSD/386 1.0
> -   |  / |  | |
> -FreeBSD 1.0 <'   NetBSD 0.9|   BSD/386 1.1
> +   | NetBSD 0.8|   BSD/386 1.0
> +   ||  | |
> +FreeBSD 1.0  NetBSD 0.9|   BSD/386 1.1
> ||   .- 4.4BSD Lite   |
>  FreeBSD 1.1 |  /   /   | \   |
> || /   /|  \  |
> @@ -151,7 +151,7 @@ FreeBSD 4.0  |  |  |  |  | NetBSD 1.
>   |   FreeBSD 4.5 ||  ||| |
>   |  | \   |  |||   BSD/OS 4.3
>   |   FreeBSD 4.6   \  |  ||OpenBSD 3.1   |
> - |  |   \ |  | NetBSD 1.5.3|  |
> + |  |   \ |  | NetBSD 1.5.3| |
>   |   FreeBSD 4.6.2 Mac OS X  | | |
>   |  |10.2| | |
>   |   FreeBSD 4.7  |  | | |
> @@ -226,10 +226,39 @@ FreeBSD 5.2   |  |  
>   | |  6.4 |  | |   |
>   | |  |  | |   |
>   |  FreeBSD 7.1   |  | |   |
> - | |  |  | |   |
> - | V  |  | |   |
> + | |  |  | |DragonFly 
> 2.2.0
> + |  FreeBSD 7.2   |   NetBSD 5.0   OpenBSD 4.5 |
> + | \  |  |||   |
> + |  |  Mac OS X  |||   |
> + |  |10.6|||   |
> + |  | |  |||DragonFly 
> 2.4.0
> + |  | |  ||OpenBSD 4.6 |
> + |  | |  |||   |
> + *--FreeBSD | |  |||   |
> + |8.0   | |  |||   |
> + | | FreeBSD  |  |||   |
> + | |   7.3|  |||DragonFly 
> 2.6.0
> + | || |  ||OpenBSD 4.7 |
> + |  FreeBSD | |  |||   |
> + |8.1   | |  |||   |
> + | || |  |||DragonFly 
> 2.8.2
> + | || |  ||OpenBSD 4.8 |
> + | || |  | NetBSD 5.1  |   |
> + |  FreeBSD  FreeBSD  |  | |   |
> + |8.2  7.4|  | |   DragonFly 
> 2.10.1

Re: CVS commit: src/games/adventure

2012-10-12 Thread Aleksej Saushev
"David A. Holland" 
writes:

> Module Name:  src
> Committed By: dholland
> Date: Fri Oct 12 10:38:53 UTC 2012
>
> Modified Files:
>   src/games/adventure: wizard.c
>
> Log Message:
> Pass -Wstrict-overflow.
>
>
> To generate a diff of this commit:
> cvs rdiff -u -r1.14 -r1.15 src/games/adventure/wizard.c
>
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.
>
>
> Modified files:
>
>
> Index: src/games/adventure/wizard.c
> diff -u src/games/adventure/wizard.c:1.14 src/games/adventure/wizard.c:1.15
> --- src/games/adventure/wizard.c:1.14 Tue Aug 25 06:56:52 2009
> +++ src/games/adventure/wizard.c  Fri Oct 12 10:38:53 2012

...

> @@ -130,19 +130,19 @@ wizard(void)
>  void
>  ciao(void)
>  {
> - char   *c;
> - charfname[80];
> + char fname[80];
> + size_t pos;
>  
>   printf("What would you like to call the saved version?\n");
>   /* XXX - should use fgetln to avoid arbitrary limit */
> - for (c = fname; c < fname + sizeof fname - 1; c++) {
> + for (pos = 0; pos < sizeof(fname - 1); pos++) {

"sizeof(fname-1)"??

(Isn't PATH_MAX better approximation to file name limit?)

>   int ch;
>   ch = getchar();
>   if (ch == '\n' || ch == EOF)
>   break;
> - *c = ch;
> + fname[pos] = ch;
>   }
> - *c = 0;
> + fname[pos] = '\0';
>   if (save(fname) != 0)
>   return; /* Save failed */
>   printf("To resume, say \"adventure %s\".\n", fname);

-- 
HE CE3OH...



Re: CVS commit: src/etc

2012-08-01 Thread Aleksej Saushev
"Julian Fagir"  writes:

> Module Name:  src
> Committed By: jdf
> Date: Mon Jul 30 22:13:38 UTC 2012
>
> Modified Files:
>   src/etc: daily
>
> Log Message:
> Call `makemandb -q` instead of `makemandb`, as proposed by Edgar Fuss on
> tech-userlevel on 20th of July 2012, 12:38.

Why does it not follow "silent-if-succesful" pattern?


-- 
HE CE3OH...



Re: CVS commit: src/etc

2012-06-05 Thread Aleksej Saushev
"Izumi Tsutsui" 
writes:

> Module Name:  src
> Committed By: tsutsui
> Date: Tue Jun  5 13:20:01 UTC 2012
>
> Modified Files:
>   src/etc: MAKEDEV.tmpl
>
> Log Message:
> Invoke MAKEDEV.local via $HOST_SH (default ${HOST_SH:=sh}) instead of
> hardcoded "sh" to avoid unexpected errors on cross build hosts in
> src/distrib/common/Makefile.makedev etc.
>
> Should fix PR toolchain/46537, ok apb@
>
>
> To generate a diff of this commit:
> cvs rdiff -u -r1.155 -r1.156 src/etc/MAKEDEV.tmpl
>
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.
>
>
> Modified files:
>
>
> Index: src/etc/MAKEDEV.tmpl
> diff -u src/etc/MAKEDEV.tmpl:1.155 src/etc/MAKEDEV.tmpl:1.156
> --- src/etc/MAKEDEV.tmpl:1.155Sat May 26 00:36:15 2012
> +++ src/etc/MAKEDEV.tmpl  Tue Jun  5 13:20:01 2012
> @@ -2092,9 +2093,9 @@ local)
>   umask 0
>   if [ -n "$count_nodes" ]; then
>   count_nodes=$((count_nodes + \
> - $(linecount "$(sh "$0.local" $opts -s all)") ))
> + $(linecount "$("$HOST_SH" "$0.local" $opts -s 
> all)") ))

Are \\s really not needed here?


-- 
HE CE3OH...


Re: CVS commit: src

2012-03-30 Thread Aleksej Saushev
Jukka Ruohonen  writes:

> On Thu, Mar 29, 2012 at 08:40:53AM +0200, Alan Barrett wrote:
>> n=4 would probably be enough iterations, and I'd suggest checking 
>
> Adjusted.
>
> By the way, some general RNG tests should be added too. I vaguely remember
> the so-called "diehard" tests somewhere from the 1990s -- perhaps something
> like that?

I don't think Marsaglia tests fit well operating system testing.


-- 
HE CE3OH...



Re: CVS commit: [netbsd-6] src/etc/root

2012-03-18 Thread Aleksej Saushev
  Hello,

Bernd Ernesti  writes:

> Hi Havard,
>
> see the issue at the end.
>
> On Sun, Mar 18, 2012 at 01:20:02AM +0400, Aleksej Saushev wrote:
>> "Manuel Bouyer"  writes:
>> 
>> > Module Name:   src
>> > Committed By:  bouyer
>> > Date:  Sat Mar 17 17:22:54 UTC 2012
>> >
>> > Modified Files:
>> >src/etc/root [netbsd-6]: dot.cshrc dot.profile
>> >
>> > Log Message:
>> > Pull up following revision(s) (requested by he in ticket #109):
>> >etc/root/dot.cshrc: revision 1.23
>> >etc/root/dot.cshrc: revision 1.24
>> >etc/root/dot.profile: revision 1.26
>> > Can't use Bourne shell syntax here... (Even in comments, for PKG_PATH.)
>> > Point first to 6.0 packages, then to packages for 5.1 or 5.0.
>> > The latter have reduced usefullness in -current or netbsd-6 until
>> > we have a compat50 package.
>> >
>> >
>> > To generate a diff of this commit:
>> > cvs rdiff -u -r1.22 -r1.22.4.1 src/etc/root/dot.cshrc
>> > cvs rdiff -u -r1.25 -r1.25.4.1 src/etc/root/dot.profile
>> >
>> > Please note that diffs are not public domain; they are subject to the
>> > copyright notices on the relevant files.
>> >
>> >
>> > Modified files:
>> ...
>> > Index: src/etc/root/dot.profile
>> > diff -u src/etc/root/dot.profile:1.25 src/etc/root/dot.profile:1.25.4.1
>> > --- src/etc/root/dot.profile:1.25  Tue Jun 21 05:31:29 2011
>> > +++ src/etc/root/dot.profile   Sat Mar 17 17:22:54 2012
>> > @@ -1,11 +1,12 @@
>> > -# $NetBSD: dot.profile,v 1.25 2011/06/21 05:31:29 erh Exp $
>> > +# $NetBSD: dot.profile,v 1.25.4.1 2012/03/17 17:22:54 bouyer Exp $
>> >  
>> >  export PATH=/sbin:/usr/sbin:/bin:/usr/bin:/usr/pkg/sbin:/usr/pkg/bin
>> >  export 
>> > PATH=${PATH}:/usr/X11R7/bin:/usr/X11R6/bin:/usr/local/sbin:/usr/local/bin
>> >  
>> >  # Uncomment the following line(s) to install binary packages
>> >  # from ftp.NetBSD.org via pkg_add.  (See also pkg_install.conf)
>> > -#export PKG_PATH="ftp://ftp.NetBSD.org/pub/pkgsrc/packages/NetBSD/$(uname 
>> > -m)/5.1/All"
>> > +#export PKG_PATH=ftp://ftp.NetBSD.org/pub/pkgsrc/packages/NetBSD/$(uname 
>> > -m)/6.0/All
>> > +#export 
>> > PKG_PATH="${PKG_PATH};ftp://ftp.NetBSD.org/pub/pkgsrc/packages/NetBSD/$(uname
>> >  -m)/5.1/All"
>> >  #export 
>> > PKG_PATH="${PKG_PATH};ftp://ftp.NetBSD.org/pub/pkgsrc/packages/NetBSD/$(uname
>> >  -m)/5.0/All"
>> >  
>> >  export BLOCKSIZE=1k
>> 
>> Why were quotes dropped?
>
> You are asking the wrong person. That change come from the HEAD commit
> by Havard, this is just the pullup.

I understand it. I must have missed the original commit mail.

I'm just a bit puzzled why it was changed to a different style.


-- 
HE CE3OH...



Re: CVS commit: [netbsd-6] src/etc/root

2012-03-17 Thread Aleksej Saushev
"Manuel Bouyer"  writes:

> Module Name:  src
> Committed By: bouyer
> Date: Sat Mar 17 17:22:54 UTC 2012
>
> Modified Files:
>   src/etc/root [netbsd-6]: dot.cshrc dot.profile
>
> Log Message:
> Pull up following revision(s) (requested by he in ticket #109):
>   etc/root/dot.cshrc: revision 1.23
>   etc/root/dot.cshrc: revision 1.24
>   etc/root/dot.profile: revision 1.26
> Can't use Bourne shell syntax here... (Even in comments, for PKG_PATH.)
> Point first to 6.0 packages, then to packages for 5.1 or 5.0.
> The latter have reduced usefullness in -current or netbsd-6 until
> we have a compat50 package.
>
>
> To generate a diff of this commit:
> cvs rdiff -u -r1.22 -r1.22.4.1 src/etc/root/dot.cshrc
> cvs rdiff -u -r1.25 -r1.25.4.1 src/etc/root/dot.profile
>
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.
>
>
> Modified files:
...
> Index: src/etc/root/dot.profile
> diff -u src/etc/root/dot.profile:1.25 src/etc/root/dot.profile:1.25.4.1
> --- src/etc/root/dot.profile:1.25 Tue Jun 21 05:31:29 2011
> +++ src/etc/root/dot.profile  Sat Mar 17 17:22:54 2012
> @@ -1,11 +1,12 @@
> -#$NetBSD: dot.profile,v 1.25 2011/06/21 05:31:29 erh Exp $
> +#$NetBSD: dot.profile,v 1.25.4.1 2012/03/17 17:22:54 bouyer Exp $
>  
>  export PATH=/sbin:/usr/sbin:/bin:/usr/bin:/usr/pkg/sbin:/usr/pkg/bin
>  export 
> PATH=${PATH}:/usr/X11R7/bin:/usr/X11R6/bin:/usr/local/sbin:/usr/local/bin
>  
>  # Uncomment the following line(s) to install binary packages
>  # from ftp.NetBSD.org via pkg_add.  (See also pkg_install.conf)
> -#export PKG_PATH="ftp://ftp.NetBSD.org/pub/pkgsrc/packages/NetBSD/$(uname 
> -m)/5.1/All"
> +#export PKG_PATH=ftp://ftp.NetBSD.org/pub/pkgsrc/packages/NetBSD/$(uname 
> -m)/6.0/All
> +#export 
> PKG_PATH="${PKG_PATH};ftp://ftp.NetBSD.org/pub/pkgsrc/packages/NetBSD/$(uname 
> -m)/5.1/All"
>  #export 
> PKG_PATH="${PKG_PATH};ftp://ftp.NetBSD.org/pub/pkgsrc/packages/NetBSD/$(uname 
> -m)/5.0/All"
>  
>  export BLOCKSIZE=1k

Why were quotes dropped?


-- 
HE CE3OH...



Re: CVS commit: src/sys/modules/zfs

2012-02-05 Thread Aleksej Saushev
"Christos Zoulas" 
writes:

> Module Name:  src
> Committed By: christos
> Date: Sat Feb  4 18:28:53 UTC 2012
>
> Modified Files:
>   src/sys/modules/zfs: Makefile.zfsmod
>
> Log Message:
> Remove -g -O0. It does not work on amd64 at all.
> If you want debugging on, add it yourself.
>
>
> To generate a diff of this commit:
> cvs rdiff -u -r1.1 -r1.2 src/sys/modules/zfs/Makefile.zfsmod
>
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.
>
>
> Modified files:
>
>
> Index: src/sys/modules/zfs/Makefile.zfsmod
> diff -u src/sys/modules/zfs/Makefile.zfsmod:1.1 
> src/sys/modules/zfs/Makefile.zfsmod:1.2
> --- src/sys/modules/zfs/Makefile.zfsmod:1.1   Tue Jan 18 17:21:22 2011
> +++ src/sys/modules/zfs/Makefile.zfsmod   Sat Feb  4 13:28:53 2012
> @@ -126,5 +126,5 @@ CPPFLAGS+=-std=c99
>  CPPFLAGS+=   -D_NFS_NFS_H_
>  CPPFLAGS+=   -D_PROPLIB_ZFS_CONFLICT
>  
> -CFLAGS+= -DDIAGNOSTIC -g -O0
> +CFLAGS+= -DDIAGNOSTIC
>  # -DZFS_DEBUG

Shouldn't it become CPPFLAGS?


-- 
HE CE3OH...



Re: CVS commit: src/share/examples

2011-11-12 Thread Aleksej Saushev
"Julio Merino"  writes:

> Module Name:  src
> Committed By: jmmv
> Date: Sat Nov 12 01:18:41 UTC 2011
>
> Modified Files:
>   src/share/examples: Makefile
> Added Files:
>   src/share/examples/asm: Makefile Makefile.inc.inst README
>   src/share/examples/asm/hello: Makefile Makefile.inst powerpc.s
>
> Log Message:
> Add assembly language examples in a new 'asm' subdirectory.
>
> At the moment this only provides an example for a "Hello world" program
> for powerpc.  (Bear with me: this is my very first assembly program in
> powerpc; if you find anything stupid in the code, please fix it!)
>
>
> To generate a diff of this commit:
> cvs rdiff -u -r1.19 -r1.20 src/share/examples/Makefile
> cvs rdiff -u -r0 -r1.1 src/share/examples/asm/Makefile \
> src/share/examples/asm/Makefile.inc.inst src/share/examples/asm/README
> cvs rdiff -u -r0 -r1.1 src/share/examples/asm/hello/Makefile \
> src/share/examples/asm/hello/Makefile.inst \
> src/share/examples/asm/hello/powerpc.s

...

> Index: src/share/examples/asm/hello/Makefile.inst
> diff -u /dev/null src/share/examples/asm/hello/Makefile.inst:1.1
> --- /dev/null Sat Nov 12 01:18:41 2011
> +++ src/share/examples/asm/hello/Makefile.instSat Nov 12 01:18:41 2011
> @@ -0,0 +1,12 @@
> +# $NetBSD: Makefile.inst,v 1.1 2011/11/12 01:18:41 jmmv Exp $
> +
> +.include 
> +
> +.if exists(${MACHINE_ARCH}.s)
> +PROG=hello
> +MAN= # empty
> +LDFLAGS= -nostdlib
> +SRCS=${MACHINE_ARCH}.s
> +.endif
> +
> +.include 
> Index: src/share/examples/asm/hello/powerpc.s
> diff -u /dev/null src/share/examples/asm/hello/powerpc.s:1.1
> --- /dev/null Sat Nov 12 01:18:41 2011
> +++ src/share/examples/asm/hello/powerpc.sSat Nov 12 01:18:41 2011
> @@ -0,0 +1,74 @@
> +# $NetBSD: powerpc.s,v 1.1 2011/11/12 01:18:41 jmmv Exp $
> +#
> +# Copyright (c) 2011 The NetBSD Foundation, Inc.
> +# All rights reserved.
> +#
> +# Redistribution and use in source and binary forms, with or without
> +# modification, are permitted provided that the following conditions are
> +# met:
> +#
> +# 1. Redistributions of source code must retain the above copyright
> +#notice, this list of conditions and the following disclaimer.
> +# 2. Redistributions in binary form must reproduce the above copyright
> +#notice, this list of conditions and the following disclaimer in the
> +#documentation and/or other materials provided with the distribution.
> +#
> +# THIS SOFTWARE IS PROVIDED BY THE NETBSD FOUNDATION, INC. AND CONTRIBUTORS
> +# ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
> +# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A
> +# PARTICULAR PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL THE FOUNDATION OR
> +# CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
> +# EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
> +# PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
> +# PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF
> +# LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
> +# NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
> +# SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
> +#
> +
> +# 
> +
> +.section ".note.netbsd.ident", "a"
> + # This ELF section is used by the kernel to determine, among other
> + # things, the system call interface used by the binary.
> + #
> + # See http://www.netbsd.org/docs/kernel/elf-notes.html for more
> + # details.
> +
> + .int 7  # Length of the OS name field below.
> + .int 4  # Length of the description field below.
> + .int 0x01   # The type of the note: NetBSD OS Version.
> + .ascii  "NetBSD\0\0"# The OS name, padded to 8 bytes.
> + .int 0x23b419a0 # The description value; 5.99.56.
> +
> +# 

#include 
#ifdef __NetBSD_Version__
.section ".note.netbsd.ident", "a"
.long   2f-1f
.long   4f-3f
.long   1
1:  .asciz  "NetBSD"
2:  .p2align 2
3:  .long   __NetBSD_Version__
4:  .p2align 2
#endif

"#ifdef" because it works (worked, at least) in FreeBSD as well.

But then it should be called ".S".
Note capital "S", standard rule is to run that through CPP.


-- 
HE CE3OH...


pgpO3vbUvjjrn.pgp
Description: PGP signature


Re: CVS commit: src/etc

2011-07-29 Thread Aleksej Saushev
"Simon Burge"  writes:

> Module Name:  src
> Committed By: simonb
> Date: Thu Jul 28 22:28:07 UTC 2011
>
> Modified Files:
>   src/etc: ntp.conf
>
> Log Message:
> Restore "duplicate" entries, but use 0. and 1. names to ensure that
> same hosts aren't used by both entries.

>  # Northern Europe
> -#server  de.pool.ntp.org
> +#server  0.de.pool.ntp.org
> +#server  1.de.pool.ntp.org
>  #server  dk.pool.ntp.org

This removes more useful information on how to setup NTP.
I believe that I don't need to remember how many groups there're in
ru.pool.ntp.org when DNS does that for me.
(I think that previous version is correct, at least ru.pool.ntp.org
seems to cycle through all servers in subpools.)


-- 
HE CE3OH...



Re: CVS commit: src/share/misc

2011-02-10 Thread Aleksej Saushev
"Jukka Ruohonen"  writes:

> Module Name:  src
> Committed By: jruoho
> Date: Thu Feb 10 17:52:18 UTC 2011
>
> Modified Files:
>   src/share/misc: acronyms.comp
>
> Log Message:
> ACID, BFS, CIL, DBMS, DFA, FSM, GPS, LLVM, MCC, ML, NFA, NP, NTM, OOSE, OTS,
> PDA, RSS, RUP, SDL, SDT, TECO, TM, TP, UCS, XOR.
>
>
> To generate a diff of this commit:
> cvs rdiff -u -r1.116 -r1.117 src/share/misc/acronyms.comp
>
>
> Index: src/share/misc/acronyms.comp
> diff -u src/share/misc/acronyms.comp:1.116 src/share/misc/acronyms.comp:1.117
> --- src/share/misc/acronyms.comp:1.116Mon Jan 17 22:08:30 2011
> +++ src/share/misc/acronyms.comp  Thu Feb 10 17:52:18 2011
> @@ -373,6 +380,7 @@
>  MBR  master boot record
>  MC   memory controller
>  MCA  machine check architecture
> +MCC  multiversion concurrency control
>  MCE  machine check exception
>  MCGA multicolor graphics adapter
>  MCH  memory controller hub

Isn't it called "MVCC" usually? "Multiple version concurrency control."
Honestly, it's the first time I see this abbreviation for MVCC.


-- 
HE CE3OH...


Re: CVS commit: src/usr.bin/hexdump

2010-11-27 Thread Aleksej Saushev
"David A. Holland" 
writes:

> Module Name:  src
> Committed By: dholland
> Date: Sat Nov 27 00:42:58 UTC 2010
>
> Modified Files:
>   src/usr.bin/hexdump: odsyntax.c
>
> Log Message:
> PR 44156: od -? prints usage for "hexdump" rather than "od"
>
>
> To generate a diff of this commit:
> cvs rdiff -u -r1.26 -r1.27 src/usr.bin/hexdump/odsyntax.c
>
> +static void
> +odusage(void)
> +{
> + (void)warnx("Usage: od [-aBbcDdeFfHhIiLlOovXx] [-A base] [-j skip]"
> + " [-N length]");
> + (void)warnx("   [-t type_string] [[+]offset[.][Bb]] [file ...]");
> + exit(1);
> +}

I still think that "warnx" is wrong way.

It is inconsistent with all the rest system, try comparing this with
output of "hexdump -?", "tar -?", "ls -?" etc etc.


-- 
HE CE3OH...



Re: CVS commit: src/distrib/sets/lists/tests

2010-11-05 Thread Aleksej Saushev
"Antti Kantee"  writes:

> Module Name:  src
> Committed By: pooka
> Date: Fri Nov  5 11:32:43 UTC 2010
>
> Modified Files:
>   src/distrib/sets/lists/tests: mi
>
> Log Message:
> +tp

Can we document what "+tp" means?


-- 
HE CE3OH...



Re: CVS commit: src/games/factor

2010-05-16 Thread Aleksej Saushev
Joerg Sonnenberger  writes:

> On Mon, May 17, 2010 at 12:47:53AM +0400, Aleksej Saushev wrote:
>> Joerg Sonnenberger  writes:
>> 
>> > On Sun, May 16, 2010 at 08:12:57PM +0400, Aleksej Saushev wrote:
>> >> I have spent several years studying math and I say that this approach is 
>> >> wrong
>> >> both in mathematical and procedural sense. It is perfectly valid to count 
>> >> 0,
>> >> +1 and -1 as prime numbers and thus factor any finite ones, it just 
>> >> involves
>> >> a bit more math than it is taught in school.
>> >
>> > Where? This is not about what is taught in school, but the generally
>> > accepted definition of "prime".
>> 
>> It is generally accepted that everything happens on the will of Allah,
>> but we're not on theological dispute. What matters here is that you
>> violate established procedures on disputed issues in case where it is
>> more or less clear way how to resolve them. I find it bad attitude of yours
>> that you:
>> 1. "Fix" disputed issues before the dispute is resolved thus
>> forcing others to follow "look, it's already there" approach.
>> 2. "Fix" them contrary to what would be consensus.
>> 3. Use arguments ex cathedra to support your actions without critically
>> considering them.
>> 4. Habitualy manage things this way.
>
> You still have done nothing to support your argument. I am still waiting
> to see any sort of authoritive reference why -1, 0 or 1 are primes or
> why this numbers have a non-empty, finite, unique set of prime factors.
> Ad hominem is not changing the lack of support.

Consider natural numbers as finite cardinals, the way set theorists do.
Factorization of a number a is non-empty set {} (f_i are distinct)
such that product f_i^n_i = a. It is easy to see that there exist numbers
which occur in any their factorization. These are called "prime" numbers.
And zero and unit are among them. Note that you have to introduce
"non-empty" to treat unit rather than zero. That makes zero no less
prime than anything else and even a bit more prime than unit.

When you construct integer numbers, you see that the same applies to "-1".
That you prefer to choose "2" as prime rather than "-2" happens due to the
wish to preserve (where applicable) theorems when reducing back to natural
numbers.

Prime factorization is factorization of minimal cardinality where all
factors are prime (and orders are minimal possible, if you use model as
above, or sum of orders, if you like that better).

Note that you can model factorization any way you like it, e.g. using
mapping of starting interval, if you like that approach more.
Though you may have to define equality on factorizations and consider
equality classes.

Using this kind of constructions is a homework for math students
when they study set theory. Yes, this isn't taught in elementary school,
perhaps this may be told only in special courses devoted to foundations
of mathematics or to mathematical logic. I don't know the structure of
math education there.

Anyway this construction covers all integers and is superset of
what you were taught in school.


-- 
HE CE3OH...


pgpJyGNTvCCZz.pgp
Description: PGP signature


Re: CVS commit: src/games/factor

2010-05-16 Thread Aleksej Saushev
Martin Husemann  writes:

> On Sun, May 16, 2010 at 07:30:54PM +, David Holland wrote:
>> of which at least the 0 case is clearly incorrect...
>
> That obviously depends on the meaning you give the output, which is
> not well defined in the man page and obviously a lot people around
> here disagree about.
>
> I would say the list of prime factors of 0 is empty, so "0:\n" makes
> perfect sense to me.

Which is wrong because product over empty set, tuple, or list, depending
on how you define it, is multiplicative unit. And zero isn't multiplicative 
unit.
This isn't arbitrary, it makes less special cases such as in

forall x, y  (product over x)*(product over y) = (product over union(x, y))

If any of x and y is empty, you should either have unit product or special-case.

If you studied computer science, you know that

[1]> (and)
T
[2]> (*)
1
[3]> (or)
NIL
[4]> (+)
0

and corresponding in Scheme

guile> (and)
#t
guile> (*)
1
guile> (or)
#f
guile> (+)
0

And this again isn't neither accidential nor arbitrary.


-- 
HE CE3OH...


pgpukBAePKL1P.pgp
Description: PGP signature


Re: CVS commit: src/games/factor

2010-05-16 Thread Aleksej Saushev
Joerg Sonnenberger  writes:

> On Sun, May 16, 2010 at 08:12:57PM +0400, Aleksej Saushev wrote:
>> I have spent several years studying math and I say that this approach is 
>> wrong
>> both in mathematical and procedural sense. It is perfectly valid to count 0,
>> +1 and -1 as prime numbers and thus factor any finite ones, it just involves
>> a bit more math than it is taught in school.
>
> Where? This is not about what is taught in school, but the generally
> accepted definition of "prime".

It is generally accepted that everything happens on the will of Allah,
but we're not on theological dispute. What matters here is that you
violate established procedures on disputed issues in case where it is
more or less clear way how to resolve them. I find it bad attitude of yours
that you:
1. "Fix" disputed issues before the dispute is resolved thus
forcing others to follow "look, it's already there" approach.
2. "Fix" them contrary to what would be consensus.
3. Use arguments ex cathedra to support your actions without critically
considering them.
4. Habitualy manage things this way.


-- 
HE CE3OH...


pgp1SMim3Jo8D.pgp
Description: PGP signature


Re: CVS commit: src/games/factor

2010-05-16 Thread Aleksej Saushev
Joerg Sonnenberger  writes:

> On Sun, May 16, 2010 at 02:10:35PM +0400, Aleksej Saushev wrote:
>> Following the logic Joerg uses, one should reject all arguments to sqrt,
>> asin, acos, atan, clog, casinh, cacosh, and other inverse functions just
>> because they have more than one branch. In "fundamental theory of 
>> mathematics"
>> be it geometry, real or complex analysis, or anything else, this approach
>> is found counterproductive.
>
> Please check the definition of the functions you are using. You are
> confusing basic mathematic properties. sqrt does not have more than one
> branch. If you want to solve a quadrativ equation, you have to check the
> different (complex) roots. Similar for the inverse functions -- they are
> defined to return the principle value. This restriction is a simple
> result of the trigonometric functions not etc not being injective.

I know that pretty fine, and I know that some people define principle
values differently, the way it is more convenient to them. If you
operate in terms of principle values, then your approach boils down to
"I've failed to define principle value for factorisation of integer
numbers and thus reject cases I couldn't understand." Arbitrary
rejection of cases you don't understand while others do is far from
mathematics.


-- 
HE CE3OH...


pgpVUXPrfQYdm.pgp
Description: PGP signature


Re: CVS commit: src/games/factor

2010-05-16 Thread Aleksej Saushev
Joerg Sonnenberger  writes:

> On Sun, May 16, 2010 at 11:03:42AM +0400, Aleksej Saushev wrote:
>> > Yes and it was quite interesting.
>> 
>> I don't like your single-handed decision, referring to wiz or kristaps
>> doesn't make honour to any of them (did Thomas present any arguments in
>> public at all?). Essentially, you're using force of first-hand attack to
>> resolve dispute in your favour.
>
> No, I am refering to those from the field of math. The "ask an expert"
> approach.

I have spent several years studying math and I say that this approach is wrong
both in mathematical and procedural sense. It is perfectly valid to count 0,
+1 and -1 as prime numbers and thus factor any finite ones, it just involves
a bit more math than it is taught in school.


-- 
HE CE3OH...


pgpniKrNtoSbm.pgp
Description: PGP signature


Re: CVS commit: src/games/factor

2010-05-16 Thread Aleksej Saushev
Martin Husemann  writes:

> On Sun, May 16, 2010 at 09:54:49AM +0100, David Laight wrote:
>> The notion of 'primes' is valid in Z - the definition of a prime is a
>> number that has no non-unit factors.
>
> Well, I only took the forced (for CS students) math courses at university,
> and it's been quite some time, but I would have defined a prime as a natural
> number > 1. For what it's worth, wikipedia seems to agree with me ;-)
> (duck)

If everyone followed this approach always, you wouldn't know that zero
is a valid number. For a long time it wasn't so. Any course in history
of mathematics mentions this. The same course most likely mentions Kuhn,
"normal science," groupthink, and other related stuff.

I'm starting to be really afraid of libm project. It involves some quite
fresh results from non-trivial mathematics, some of them were obtained
within recent 5 years.


-- 
HE CE3OH...


pgpaxruLoFXBk.pgp
Description: PGP signature


Re: CVS commit: src/games/factor

2010-05-16 Thread Aleksej Saushev
David Laight  writes:

> On Sat, May 15, 2010 at 09:22:39PM +, Joerg Sonnenberger wrote:
>> 
>> Log Message:
>> Follow the Fundamental Theory of Algebra. Disallow factorising of
>> numbers less than 2 as it is not
>> - naturally unique (negative numbers)
>> - finite (0)
>> - non-empty (1)
>
> The 'Natural numbers (N)' are the positive integers, for which both addition
> and multiplication are defined.

This heavily depends on school. At least set theorists define natural numbers
as finite cardinals, many other logicians and some algebraists do the same,
this makes zero natural number.

> To make an additive group you need to include zero, zero is also needed to
> make a number field.
> If you include the subtraction operator then you need to include the
> negative numbers - this gives the 'Integers (Z)'.
> The notion of 'primes' is valid in Z - the definition of a prime is a
> number that has no non-unit factors.
> The units of Z are +1 and -1, so both +2 and -2 are primes (etc).

This depends on definition of unit. Yours isn't canonical but it may
be reasonable under some assumptions.

> So nothing about algebra stops you factoring negative numbers.
> However, since the 'prime factors' should be prime numbers, they
> shouldn't include -1, but maybe the smallest factor should be negative.

While it may be controversial whether to count 1 and -1 as prime,
it is perfectly sensible to do so.

Again, this boils down to difference in definitions and it should be
discussed rather than rejected ex cathedra.

Following the logic Joerg uses, one should reject all arguments to sqrt,
asin, acos, atan, clog, casinh, cacosh, and other inverse functions just
because they have more than one branch. In "fundamental theory of mathematics"
be it geometry, real or complex analysis, or anything else, this approach
is found counterproductive.


-- 
HE CE3OH...


pgpjONTT8b0v3.pgp
Description: PGP signature


Re: CVS commit: src/games/factor

2010-05-16 Thread Aleksej Saushev
Joerg Sonnenberger  writes:

> On Sat, May 15, 2010 at 09:44:34PM +, David Holland wrote:
>> On Sat, May 15, 2010 at 09:22:39PM +, Joerg Sonnenberger wrote:
>>  > Modified Files:
>>  >   src/games/factor: factor.6 factor.c
>>  > 
>>  > Log Message:
>>  > Follow the Fundamental Theory of Algebra. Disallow factorising of
>>  > numbers less than 2 as it is not
>>  > - naturally unique (negative numbers)
>>  > - finite (0)
>>  > - non-empty (1)
>>  > 
>>  > Discussed with the kristaps and wiz
>> 
>> Did you bother to read tech-userlevel before doing this?
>
> Yes and it was quite interesting.

I don't like your single-handed decision, referring to wiz or kristaps
doesn't make honour to any of them (did Thomas present any arguments in
public at all?). Essentially, you're using force of first-hand attack to
resolve dispute in your favour.

All three of your arguments in commit log are simply wrong, I pointed
you to that but you ignored it.

Please, back it out and present what you refer to as "Fundamental Theory
of Algebra" on tech-userlevel. I don't believe it exists, and if you
want it to be taken as argument, present overview of text books where
it is discussed rather than given as a matter of fact.

We diverge in approaches to libm validation and verification already,
and I don't want to trash numeric stuff just because of your attitude
to act ignoring anyone else.


-- 
HE CE3OH...


pgppS8AfHYUmz.pgp
Description: PGP signature


CVS commit: src/sys/dev/i2o

2010-02-19 Thread Aleksej Saushev
Module Name:src
Committed By:   asau
Date:   Fri Feb 19 12:36:15 UTC 2010

Modified Files:
src/sys/dev/i2o: iop.c

Log Message:
Don't mix BUS_DMASYNC_PREREAD and BUS_DMASYNC_POSTWRITE.
Suggested by , fixes PR kern/42662


To generate a diff of this commit:
cvs rdiff -u -r1.77 -r1.78 src/sys/dev/i2o/iop.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.



CVS commit: src/tools/compat

2010-01-15 Thread Aleksej Saushev
Module Name:src
Committed By:   asau
Date:   Fri Jan 15 11:27:01 UTC 2010

Modified Files:
src/tools/compat: configure

Log Message:
Regenerate.


To generate a diff of this commit:
cvs rdiff -u -r1.70 -r1.71 src/tools/compat/configure

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.



CVS commit: src/tools/compat

2010-01-15 Thread Aleksej Saushev
Module Name:src
Committed By:   asau
Date:   Fri Jan 15 11:26:25 UTC 2010

Modified Files:
src/tools/compat: configure.ac nbtool_config.h.in

Log Message:
Test for  and provide HAVE_SYS_ENDIAN_H.
Fixes FreeBSD hosted builds (PR 42440 and PR 42470).

Approved by .


To generate a diff of this commit:
cvs rdiff -u -r1.70 -r1.71 src/tools/compat/configure.ac
cvs rdiff -u -r1.23 -r1.24 src/tools/compat/nbtool_config.h.in

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.