Re: CVS commit: src

2012-11-29 Thread Nick Hudson

On 11/30/12 02:22, Emmanuel Dreyfus wrote:

Matthias Drochner  wrote:


I've added (locally)

#if defined(_NETBSD_SOURCE)
#define _INCOMPLETE_XOPEN_C063
#endif

Thank you fox the fix.



It's not committed yet if that's what you mean.

Or am I blind?

NIck


Re: CVS commit: src

2012-11-29 Thread Emmanuel Dreyfus
Matthias Drochner  wrote:

> I've added (locally)
> 
> #if defined(_NETBSD_SOURCE)
> #define _INCOMPLETE_XOPEN_C063
> #endif

Thank you fox the fix.

-- 
Emmanuel Dreyfus
http://hcpnet.free.fr/pubz
m...@netbsd.org


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

2012-11-29 Thread Marc Balmer
Comments inline...


> Module Name:  src
> Committed By: reed
> Date: Thu Nov 29 23:23:23 UTC 2012
> 
> Added Files:
>   src/usr.bin/mkubootimage: mkubootimage.1
> 
> Log Message:
> Add a manual for mkubootimage(1). It was missing.
> It had one review on netbsd-docs@.
> I didn't add the manual to the Makefile nor distrib set yet --
> waiting for more review or improvements first.
> 
> 
> To generate a diff of this commit:
> cvs rdiff -u -r0 -r1.1 src/usr.bin/mkubootimage/mkubootimage.1
> 
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.
> 
> Added files:
> 
> Index: src/usr.bin/mkubootimage/mkubootimage.1
> diff -u /dev/null src/usr.bin/mkubootimage/mkubootimage.1:1.1
> --- /dev/null Thu Nov 29 23:23:23 2012
> +++ src/usr.bin/mkubootimage/mkubootimage.1   Thu Nov 29 23:23:22 2012
> @@ -0,0 +1,162 @@
> +.\"  $NetBSD: mkubootimage.1,v 1.1 2012/11/29 23:23:22 reed Exp $
> +.\"
> +.\" Copyright (c) 2012 The NetBSD Foundation, Inc.
> +.\" All rights reserved.
> +.\"
> +.\" This code is derived from software contributed to The NetBSD Foundation
> +.\" by Jeremy C. Reed
> +.\"
> +.\" 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.
> +.\"
> +.Dd November 28, 2012
> +.Dt MKUBOOTIMAGE 1
> +.Os
> +.Sh NAME
> +.Nm mkubootimage
> +.Nd create U-Boot loader image
> +.Sh SYNOPSIS
> +.Nm
> +.Fl A No ( arm Ns | Ns mips Ns | Ns mips64 Ns | Ns powerpc )
> +.Fl a Ar address
> +.Op Fl C No Po bz2 Ns | Ns gz Ns | Ns lzma Ns | Ns lzo Ns | Ns none Pc
> +.Op Fl E Ar address
> +.Op Fl e Ar address
> +.Op Fl h
> +.Op Fl m Ar magic
> +.Fl n Ar image
> +.Op Fl O No Po freebsd Ns | Ns linux Ns | Ns netbsd Ns | Ns openbsd Pc
> +.Fl T No ( fs Ns | Ns kernel Ns | Ns ramdisk Ns | Ns standalone )
> +.Ar source destination
> +.\"
> +.Sh DESCRIPTION
> +The
> +.Nm
> +utility is used to create an image for Das U-Boot, the
> +.Qq Universal Boot Loader .
> +.Pp
> +The arguments are as follows:
> +.Bl -tag -width indent
> +.It Fl A No ( arm Ns | Ns mips Ns | Ns mips64 Ns | Ns powerpc )
> +Defines the architecture.
> +This is required.
> +.It Fl a Ar address
> +Sets the image load address.
> +This is an integer between 0 and ULONG_MAX.

Maybe note where ULONG_MAX is defined or specify the value here

"and ULONG_MAX as defined in //X.h"

> +This is required.
> +.It Fl C No ( bz2 Ns | Ns gz Ns | Ns lzma Ns | Ns lzo Ns | Ns none Ns )
> +Defines the compression.
> +The default is
> +.Qq none .
> +.It Fl E Ar address
> +Define the entry point.
> +This is an integer between 0 and ULONG_MAX.
> +This option is the same as
> +.Fl e
> +but it inverts the bytes to convert the 32 bit integer
> +from little to big endian (or vice versa).
> +If
> +.Fl E
> +or
> +.Fl e
> +are not set, the entry point defaults to the
> +image load address
> +.Pq Fl a .
> +.It Fl e Ar address
> +Define the entry point.
> +This is an integer between 0 and ULONG_MAX.
> +If
> +.Fl E
> +or
> +.Fl e
> +are not set, the entry point defaults to the
> +image load address
> +.Pq Fl a .
> +.It Fl h
> +Display the usage and exit.
> +.It Fl m Ar magic
> +Set the magic.
> +This is an integer between 0 and ULONG_MAX.
> +The default is
> +.Qq 0x27051956.
> +.It Fl n Ar image
> +Defines the image name.
> +This is required.
> +.It Fl O No ( freebsd Ns | Ns linux Ns | Ns netbsd Ns | Ns openbsd )
> +Defines the operating system type.
> +The default OS name is
> +.Qq netbsd .
> +.It Fl T No ( fs Ns | Ns kernel Ns | Ns ramdisk Ns | Ns standalone )
> +Defines the image type.
> +This is required.
> +.El
> +.Pp
> +The required
> +.Ar source
> +argument is the filename of the kernel that is read.
> +The required
>

Re: CVS commit: src

2012-11-29 Thread Matthias Drochner

On Thu, 29 Nov 2012 19:20:14 +
Nick Hudson  wrote:
> On Sunday 18 November 2012 17:41:55 Emmanuel Dreyfus wrote:
> > -#if defined(_INCOMPLETE_XOPEN_C063)
> > +#if (_POSIX_C_SOURCE - 0) >= 200809L || (_XOPEN_SOURCE - 0 >= 700)
> > || \
> > +defined(_INCOMPLETE_XOPEN_C063) || defined(_KERNEL)
>
> Shouldn't this be
>
> #if (_POSIX_C_SOURCE - 0) >= 200809L || (_XOPEN_SOURCE - 0 >= 700) ||
> \ defined(_INCOMPLETE_XOPEN_C063) || defined(_NETBSD_SOURCE)

I've added (locally)

#if defined(_NETBSD_SOURCE)
#define _INCOMPLETE_XOPEN_C063
#endif

to sys/types.h in the meantime, as a quick hack to fix
build problems with various userland software.
Has about the same effect -- something like this
is definitely necessary.

nest regards
Matthias




Forschungszentrum Juelich GmbH
52425 Juelich
Sitz der Gesellschaft: Juelich
Eingetragen im Handelsregister des Amtsgerichts Dueren Nr. HR B 3498
Vorsitzender des Aufsichtsrats: MinDir Dr. Karl Eugen Huthmacher
Geschaeftsfuehrung: Prof. Dr. Achim Bachem (Vorsitzender),
Karsten Beneke (stellv. Vorsitzender), Prof. Dr.-Ing. Harald Bolt,
Prof. Dr. Sebastian M. Schmidt




Re: CVS commit: src

2012-11-29 Thread Emmanuel Dreyfus
Nick Hudson  wrote:

> Shouldn't this be
> 
> #if (_POSIX_C_SOURCE - 0) >= 200809L || (_XOPEN_SOURCE - 0 >= 700) || \
> defined(_INCOMPLETE_XOPEN_C063) || defined(_NETBSD_SOURCE)

I suspect you are right.

-- 
Emmanuel Dreyfus
http://hcpnet.free.fr/pubz
m...@netbsd.org


Re: CVS commit: src

2012-11-29 Thread Nick Hudson
On Sunday 18 November 2012 17:41:55 Emmanuel Dreyfus wrote:
> -#if defined(_INCOMPLETE_XOPEN_C063)
> +#if (_POSIX_C_SOURCE - 0) >= 200809L || (_XOPEN_SOURCE - 0 >= 700) || \
> +defined(_INCOMPLETE_XOPEN_C063) || defined(_KERNEL)

Shouldn't this be

#if (_POSIX_C_SOURCE - 0) >= 200809L || (_XOPEN_SOURCE - 0 >= 700) || \
defined(_INCOMPLETE_XOPEN_C063) || defined(_NETBSD_SOURCE)

?

Nick


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

2012-11-29 Thread Christos Zoulas
In article ,
dieter roelants   wrote:
>On Thu, Nov 1, 2012 at 4:26 AM, Christos Zoulas  wrote:
>> Module Name:src
>> Committed By:   christos
>> Date:   Thu Nov  1 03:26:44 UTC 2012
>>
>> Modified Files:
>> src/distrib/sets/lists/base: mi
>> src/distrib/sets/lists/comp: mi
>>
>> Log Message:
>> mv npfctl from /usr/sbin to /sbin so it is available before /usr is mounted.
>
>I guess libnpf should be moved to /lib/.

It has to; thanks for noticing.

christos



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

2012-11-29 Thread dieter roelants
On Thu, Nov 1, 2012 at 4:26 AM, Christos Zoulas  wrote:
> Module Name:src
> Committed By:   christos
> Date:   Thu Nov  1 03:26:44 UTC 2012
>
> Modified Files:
> src/distrib/sets/lists/base: mi
> src/distrib/sets/lists/comp: mi
>
> Log Message:
> mv npfctl from /usr/sbin to /sbin so it is available before /usr is mounted.

I guess libnpf should be moved to /lib/.