svn commit: trunk/busybox: include miscutils

[vda at busybox.net]

Author: vda
Date: 2008-12-03 02:46:12 -0800 (Wed, 03 Dec 2008)
New Revision: 24240

Log:
crond: use bb_setpgrp, not setpgrp
platform.h: make a bit more readable



Modified:
   trunk/busybox/include/platform.h
   trunk/busybox/miscutils/crond.c


Changeset:
Modified: trunk/busybox/include/platform.h
===================================================================
--- trunk/busybox/include/platform.h	2008-12-03 10:36:26 UTC (rev 24239)
+++ trunk/busybox/include/platform.h	2008-12-03 10:46:12 UTC (rev 24240)
@@ -78,7 +78,7 @@
 //__attribute__ ((__externally_visible__))
 #else
 # define EXTERNALLY_VISIBLE
-#endif /* GNUC >= 4.1 */
+#endif
 
 /* We use __extension__ in some places to suppress -pedantic warnings
    about GCC extensions.  This feature didn't work properly before
@@ -185,7 +185,7 @@
  * until userspace is widely fixed.  */
 #if (defined __INTEL_COMPILER && !defined __GNUC__) || \
 	(defined __GNUC__ && defined __STRICT_ANSI__)
-__extension__ typedef __signed__ long long __s64;
+__extension__ typedef long long __s64;
 __extension__ typedef unsigned long long __u64;
 #endif
 
@@ -296,30 +296,32 @@
 #endif
 
 #if (defined __digital__ && defined __unix__)
-#include <standards.h>
-#define HAVE_STANDARDS_H
-#include <inttypes.h>
-#define HAVE_INTTYPES_H
-#define PRIu32 "u"
 
+# include <standards.h>
+# define HAVE_STANDARDS_H
+# include <inttypes.h>
+# define HAVE_INTTYPES_H
+# define PRIu32 "u"
 /* use legacy setpgrp(pid_t,pid_t) for now.  move to platform.c */
-#define bb_setpgrp() do { pid_t __me = getpid(); setpgrp(__me,__me); } while (0)
+# define bb_setpgrp() do { pid_t __me = getpid(); setpgrp(__me,__me); } while (0)
 
-#if !defined ADJ_OFFSET_SINGLESHOT && defined MOD_CLKA && defined MOD_OFFSET
-#define ADJ_OFFSET_SINGLESHOT (MOD_CLKA | MOD_OFFSET)
-#endif
-#if !defined ADJ_FREQUENCY && defined MOD_FREQUENCY
-#define ADJ_FREQUENCY MOD_FREQUENCY
-#endif
-#if !defined ADJ_TIMECONST && defined MOD_TIMECONST
-#define ADJ_TIMECONST MOD_TIMECONST
-#endif
-#if !defined ADJ_TICK && defined MOD_CLKB
-#define ADJ_TICK MOD_CLKB
-#endif
+# if !defined ADJ_OFFSET_SINGLESHOT && defined MOD_CLKA && defined MOD_OFFSET
+#  define ADJ_OFFSET_SINGLESHOT (MOD_CLKA | MOD_OFFSET)
+# endif
+# if !defined ADJ_FREQUENCY && defined MOD_FREQUENCY
+#  define ADJ_FREQUENCY MOD_FREQUENCY
+# endif
+# if !defined ADJ_TIMECONST && defined MOD_TIMECONST
+#  define ADJ_TIMECONST MOD_TIMECONST
+# endif
+# if !defined ADJ_TICK && defined MOD_CLKB
+#  define ADJ_TICK MOD_CLKB
+# endif
 
-#else
-#define bb_setpgrp() setpgrp()
+#else /* !__digital__ */
+
+# define bb_setpgrp() setpgrp()
+
 #endif
 
 #if defined(__linux__)

Modified: trunk/busybox/miscutils/crond.c
===================================================================
--- trunk/busybox/miscutils/crond.c	2008-12-03 10:36:26 UTC (rev 24239)
+++ trunk/busybox/miscutils/crond.c	2008-12-03 10:46:12 UTC (rev 24240)
@@ -780,7 +780,7 @@
 			dup2(1, 2);
 		}
 		/* "standard" cron puts tasks in separate process groups */
-		setpgrp();
+		bb_setpgrp();
 		execlp(prog, prog, cmd, arg, NULL);
 		crondlog(ERR20 "can't exec, user %s cmd %s %s %s", user, prog, cmd, arg);
 		if (mail_filename) {
@@ -917,7 +917,7 @@
 			crondlog(LVL5 "child running %s", DEFAULT_SHELL);
 		}
 		/* "standard" cron puts tasks in separate process groups */
-		setpgrp();
+		bb_setpgrp();
 		execl(DEFAULT_SHELL, DEFAULT_SHELL, "-c", line->cl_Shell, NULL);
 		crondlog(ERR20 "can't exec, user %s cmd %s %s %s", user,
 				 DEFAULT_SHELL, "-c", line->cl_Shell);