bug#1129: [BusyBox] bug#1129: sync.c does not compile on RedHat 7.1

[Erik Andersen]

On Tue Mar 13, 2001 at 05:39:51PM +0100, Gratien D'haese (Hewlett-Packard) wrote:
> Package: busybox
> Version: 0.49
> Severity: normal
> 
> Hi,
> 
> I'm the maintainer of mkCDrec and one of my beta-tester on RedHat 7.1
> reported a bug against busybox' sync.c.
> On RedHat 7.1 gcc-2.96-75 is used as c compiler. The following makes
> sync.c compile properly
> 
> --- sync.c.orig Mon Mar   5 17:44:42 2001
> +++ sync.c      Sun Mar   4 17:34:14 2001
> @@ -29,5 +29,7 @@
>  {
> 	if (argc > 1 && **(argv + 1) == '-')
> 		usage(sync_usage);
> -	return(sync());
> +/*	return(sync());*/
> +	sync();
> +	return(0);
>  }
> 
> According to man 2 sync, the prototype for sync is:
> extern void sync(void)
> 
> Hope this fix is good enough?

How very odd.  This is what I see with glibc 2.1.3

    [andersen at slag busybox]$ grep "\bsync\b" /usr/include/unistd.h
    extern int sync __P ((void));

What glibc version is used on RedHat 7.1 and did the sync
prototype really change?  (Of course, this is mostly academic
since sync always returns 0, I'm just curious).

 -Erik

--
Erik B. Andersen   email:  andersen at lineo.com
--This message was written using 73% post-consumer electrons--