Re: [Balloon] Further build issues

Top Page
Attachments:
Message as email
+ (text/plain)
Delete this message
Reply to this message
Author: Charles Manning
Date:  
To: balloon
Subject: Re: [Balloon] Further build issues
On Thursday 08 April 2010 19:47:30 Bob Dunlop wrote:
> On Thu, Apr 08 at 09:39, Charles Manning wrote:
> > During the initrd building, the building of makedev failed due to the
> > compiler being -Werror and system() return value being ignored. I got
> > past that with the following patch. I'm sure this should rather be fixed
> > elsewhere.
>
> A simpler patch would be to replace system() by calling sync() direct.
>
> $ diff -u makedevs.c~ makedevs.c
> --- makedevs.c~    2010-04-08 08:55:31.000000000 +1200
> +++ makedevs.c    2010-04-08 08:56:56.608535407 +1200
> @@ -528,7 +528,7 @@
>      }
>      fclose(table);

>
> -    system("/bin/sync");
> +    sync();

>
>      return 0;
>  }


Well that's certainly neater but I was a bit concerned that perhaps running
through system() might be doing something subtly different.

The other much uglier one-liner is
-    system("/bin/sync");
+    { int e = system("/bin/sync"); }


which should probably earn me a thrashing.

-- Charles