Re: CVS commit: src/tests/lib/librt

2012-03-17 Thread Jukka Ruohonen
On Sat, Mar 17, 2012 at 10:14:16PM -0400, Christos Zoulas wrote:
> Module Name:  src
> Committed By: christos
> Date: Sun Mar 18 02:14:16 UTC 2012
> 
> Modified Files:
>   src/tests/lib/librt: t_sched.c
> 
> Log Message:
> this should be fixed.

Hmm. But now it fails with the third check:

pid_t p = getpid();   
   
/*
 * IEEE Std 1003.1-2008: if the supplied pid is zero,
 * the parameters for the calling process are returned.
 */
ATF_REQUIRE(sched_getparam(0, &s1) == 0);
ATF_REQUIRE(sched_getparam(p, &s2) == 0);
[HERE]  ATF_REQUIRE(s1.sched_priority == s2.sched_priority);

- Jukka.


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

2012-03-17 Thread Bernd Ernesti
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.profileSat 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.

Bernd



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: [netbsd-5-1] src

2012-03-17 Thread Manuel Bouyer
On Sat, Mar 17, 2012 at 07:17:04PM +, Manuel Bouyer wrote:
> Module Name:  src
> Committed By: bouyer
> Date: Sat Mar 17 19:17:04 UTC 2012
> 
> Modified Files:
>   src/doc [netbsd-5-1]: CHANGES-5.1.3
>   src/usr.bin/ktruss [netbsd-5-1]: makeerrnos.sh
> 
> Log Message:
> ticket 1742

Message for makeerrnos.sh should have been:

Pull up following revision(s) (requested by dyoung in ticket #1742):
usr.bin/ktruss/makeerrnos.sh: revision 1.6
Use 'sort -n -k 3' instead of 'sort -n +2' since the former is more
portable.  The latter is not supported by Mac OS X Lion.

fixed in repository.
-- 
Manuel Bouyer 
 NetBSD: 26 ans d'experience feront toujours la difference
--


Re: CVS commit: src/sbin/ifconfig

2012-03-17 Thread Nicolas Joly
On Sat, Mar 17, 2012 at 06:53:31PM +0200, Jukka Ruohonen wrote:
> On Fri, Mar 16, 2012 at 10:25:08PM -0400, Christos Zoulas wrote:
> > Module Name:src
> > Committed By:   christos
> > Date:   Sat Mar 17 02:25:08 UTC 2012
> > 
> > Modified Files:
> > src/sbin/ifconfig: af_inetany.c
> > 
> > Log Message:
> > PR/43141: Tobias Nygren: Print an error on unknown interfaces.
> 
> Now this is trival and it is fixed. But I guess there is a bug here in ATF.
> The test only checks that the exit code is not zero:
> 
> atf_check -s not-exit:0 ifconfig nonexistent0 1.2.3.4/24
> 
> But why is stderr being involved?

atf_check defaults to '-s exit:0 -o empty -e empty'.
Adding '-e ignore' should do the trick.

-- 
Nicolas Joly

Projects and Developments in Bioinformatics
Institut Pasteur, Paris.


Re: CVS commit: [netbsd-6] src/sys/uvm

2012-03-17 Thread Manuel Bouyer
On Sat, Mar 17, 2012 at 05:29:34PM +, Manuel Bouyer wrote:
> Module Name:  src
> Committed By: bouyer
> Date: Sat Mar 17 17:29:34 UTC 2012
> 
> Modified Files:
>   src/sys/uvm [netbsd-6]: uvm_km.c
> 
> Log Message:
> Pull up following revision(s) (requested by rmind in ticket #113):
>   sys/uvm/uvm_km.c: revision 1.123
> uvm_km_kmem_alloc: return ENOMEM on failure in PMAP_MAP_POOLPAGE case.

This was requested by chs, not rmind ...

-- 
Manuel Bouyer 
 NetBSD: 26 ans d'experience feront toujours la difference
--


Re: CVS commit: src/sbin/ifconfig

2012-03-17 Thread Jukka Ruohonen
On Fri, Mar 16, 2012 at 10:25:08PM -0400, Christos Zoulas wrote:
> Module Name:  src
> Committed By: christos
> Date: Sat Mar 17 02:25:08 UTC 2012
> 
> Modified Files:
>   src/sbin/ifconfig: af_inetany.c
> 
> Log Message:
> PR/43141: Tobias Nygren: Print an error on unknown interfaces.

Now this is trival and it is fixed. But I guess there is a bug here in ATF.
The test only checks that the exit code is not zero:

atf_check -s not-exit:0 ifconfig nonexistent0 1.2.3.4/24

But why is stderr being involved?

Fail: stderr not empty
--- /dev/null   2012-03-17 15:38:12.0 +
+++ /tmp/check.09347a/stderr2012-03-17 15:40:11.0 +
@@ -0,0 +1 @@
+ifconfig: commit_address: getifinfo: Device not configured

- Jukka.


Re: CVS commit: src/sys/sys

2012-03-17 Thread Ryo ONODERA
Hi,

From: "Christos Zoulas" , Date: Wed, 14 Mar 2012 20:09:08 
-0400

> Module Name:  src
> Committed By: christos
> Date: Thu Mar 15 00:09:08 UTC 2012
> 
> Modified Files:
>   src/sys/sys: cdefs.h
> 
> Log Message:
> - this file is still used from assembly. protect inlines against __ASSEMBLER__
> - rename __zero -> __zeroll, __negative -> __negative_p
> 
> 
> To generate a diff of this commit:
> cvs rdiff -u -r1.93 -r1.94 src/sys/sys/cdefs.h
> 
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.

It seems that this change cause PR pkg/46206.
Can anyone investigate this?

-- 
Ryo ONODERA // ryo...@yk.rim.or.jp
PGP fingerprint = 82A2 DC91 76E0 A10A 8ABB  FD1B F404 27FA C7D1 15F3




Re: CVS commit: src/sys/sys

2012-03-17 Thread David Holland
On Fri, Mar 09, 2012 at 10:38:03AM -0500, Christos Zoulas wrote:
 > Modified Files:
 >  src/sys/sys: gmon.h
 > 
 > Log Message:
 > use explicit cast for RHS of shift operator since semantics change from
 > traditional c to ansi.
 > 
 > To generate a diff of this commit:
 > cvs rdiff -u -r1.9 -r1.10 src/sys/sys/gmon.h

Also, shouldn't that 8 be CHAR_BIT?

-- 
David A. Holland
dholl...@netbsd.org