Re: [PATCH 2/3] Port to AIX

2011-03-10 Thread Herbert Xu
On Wed, Jan 19, 2011 at 05:15:49AM +, Brian Koropoff wrote:
> - AIX lacks a WCOREDUMP macro.  It's just used to
>   append "(core dumped)" to the crash message, so
>   #ifdef around it.
> 
> - For some reason, the nl program on AIX defaults
>   to not printing line numbers ("-b n"), even though
>   the spec says it should default to "-b t".
>   Explicitly pass "-b a" for good measure in mkbuiltins.
> 
> Signed-off-by: Brian Koropoff 

Patch applied.  Thanks.
-- 
Email: Herbert Xu 
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
--
To unsubscribe from this list: send the line "unsubscribe dash" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 2/3] Port to AIX

2011-01-18 Thread Brian Koropoff
- AIX lacks a WCOREDUMP macro.  It's just used to
  append "(core dumped)" to the crash message, so
  #ifdef around it.

- For some reason, the nl program on AIX defaults
  to not printing line numbers ("-b n"), even though
  the spec says it should default to "-b t".
  Explicitly pass "-b a" for good measure in mkbuiltins.

Signed-off-by: Brian Koropoff 
---
 src/jobs.c |2 ++
 src/mkbuiltins |2 +-
 2 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/src/jobs.c b/src/jobs.c
index fdb122a..88b5792 100644
--- a/src/jobs.c
+++ b/src/jobs.c
@@ -431,9 +431,11 @@ sprint_status(char *s, int status, int sigonly)
 #endif
}
col = fmtstr(s, 32, strsignal(st));
+#ifdef WCOREDUMP
if (WCOREDUMP(status)) {
col += fmtstr(s + col, 16, " (core dumped)");
}
+#endif
} else if (!sigonly) {
if (st)
col = fmtstr(s, 16, "Done(%d)", st);
diff --git a/src/mkbuiltins b/src/mkbuiltins
index bb1e2a4..495274e 100644
--- a/src/mkbuiltins
+++ b/src/mkbuiltins
@@ -99,7 +99,7 @@ cat <<\!
  */
 
 !
-sed 's/-[a-z]*//' $temp2 | nl -v 0 | LC_COLLATE=C sort -u -k 3,3 |
+sed 's/-[a-z]*//' $temp2 | nl -b a -v 0 | LC_COLLATE=C sort -u -k 3,3 |
 tr abcdefghijklmnopqrstuvwxyz ABCDEFGHIJKLMNOPQRSTUVWXYZ |
awk '{  printf "#define %s (builtincmd + %d)\n", $3, $1}'
 printf '\n#define NUMBUILTINS %d\n' $(wc -l < $temp2)
-- 
1.7.1


--
To unsubscribe from this list: send the line "unsubscribe dash" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html