* [Buildroot] [PATCH] make strace buildable
@ 2007-07-16 20:32 Yann E. MORIN
2007-07-24 17:33 ` Yann E. MORIN
2007-09-20 14:45 ` Bernhard Fischer
0 siblings, 2 replies; 6+ messages in thread
From: Yann E. MORIN @ 2007-07-16 20:32 UTC (permalink / raw)
To: buildroot
Hello all!
The second hunk in package/strace/strace-undef-syscall.patch is incorrect:
--- strace-4.5.15.orig/linux/syscallent.h 2007-02-04 19:03:48.000000000 +0100
+++ strace-4.5.15/linux/syscallent.h 2007-02-04 19:11:04.000000000 +0100
@@ -130,7 +130,10 @@
{ 2, TF, sys_statfs, "statfs" }, /* 99 */
{ 2, TD, sys_fstatfs, "fstatfs" }, /* 100 */
{ 3, 0, sys_ioperm, "ioperm" }, /* 101 */
- { 2, TD, sys_socketcall, "socketcall", SYS_socketcall }, /* 102 */
+ { 2, TD, sys_socketcall, "socketcall",
+#ifdef __NR_socketcall
+ SYS_socketcall }, /* 102 */
+#endif
{ 3, 0, sys_syslog, "syslog" }, /* 103 */
{ 3, 0, sys_setitimer, "setitimer" }, /* 104 */
{ 2, 0, sys_getitimer, "getitimer" }, /* 105 */
@@ -145,7 +148,10 @@
{ 4, TP, sys_wait4, "wait4", SYS_wait4 }, /* 114 */
{ 1, 0, sys_swapoff, "swapoff" }, /* 115 */
{ 1, 0, sys_sysinfo, "sysinfo" }, /* 116 */
- { 6, 0, sys_ipc, "ipc", SYS_ipc }, /* 117 */
+ { 6, 0, sys_ipc, "ipc",
+#ifdef __NR_ipc
+ SYS_ipc }, /* 117 */
+#endif
{ 1, TD, sys_fsync, "fsync" }, /* 118 */
{ 1, TS, sys_sigreturn, "sigreturn" }, /* 119 */
{ 5, TP, sys_clone, "clone", SYS_clone }, /* 120 */
If either or both of __NR_socketcall and __NR_ipc are not defined, then the
closing curly braces are omitted, thus resulting in incorrect C statements.
The attached patch to r19112 solves the issue.
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +0/33 662376056 | Software Designer | \ / CAMPAIGN | ^ |
| --==< ?_? >==-- ?------------.-------: X AGAINST | /e\ There is no |
| http://ymorin.is-a-geek.org/ | (*_*) | / \ HTML MAIL | """ conspiracy. |
?------------------------------?-------?------------------?--------------------?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: strace-syscallent.h-r19112.patch
Type: text/x-diff
Size: 1645 bytes
Desc: not available
Url : http://busybox.net/lists/buildroot/attachments/20070716/61ad059d/attachment.bin
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH] make strace buildable
2007-07-16 20:32 [Buildroot] [PATCH] make strace buildable Yann E. MORIN
@ 2007-07-24 17:33 ` Yann E. MORIN
2007-07-24 17:53 ` Bernhard Fischer
2007-09-20 14:45 ` Bernhard Fischer
1 sibling, 1 reply; 6+ messages in thread
From: Yann E. MORIN @ 2007-07-24 17:33 UTC (permalink / raw)
To: buildroot
Hello!
Any taker?
On Monday 16 July 2007 22:32, Yann E. MORIN wrote:
> The second hunk in package/strace/strace-undef-syscall.patch is incorrect:
>
> --- strace-4.5.15.orig/linux/syscallent.h 2007-02-04 19:03:48.000000000 +0100
> +++ strace-4.5.15/linux/syscallent.h 2007-02-04 19:11:04.000000000 +0100
> @@ -130,7 +130,10 @@
> { 2, TF, sys_statfs, "statfs" }, /* 99 */
> { 2, TD, sys_fstatfs, "fstatfs" }, /* 100 */
> { 3, 0, sys_ioperm, "ioperm" }, /* 101 */
> - { 2, TD, sys_socketcall, "socketcall", SYS_socketcall }, /* 102 */
> + { 2, TD, sys_socketcall, "socketcall",
> +#ifdef __NR_socketcall
> + SYS_socketcall }, /* 102 */
> +#endif
> { 3, 0, sys_syslog, "syslog" }, /* 103 */
> { 3, 0, sys_setitimer, "setitimer" }, /* 104 */
> { 2, 0, sys_getitimer, "getitimer" }, /* 105 */
> @@ -145,7 +148,10 @@
> { 4, TP, sys_wait4, "wait4", SYS_wait4 }, /* 114 */
> { 1, 0, sys_swapoff, "swapoff" }, /* 115 */
> { 1, 0, sys_sysinfo, "sysinfo" }, /* 116 */
> - { 6, 0, sys_ipc, "ipc", SYS_ipc }, /* 117 */
> + { 6, 0, sys_ipc, "ipc",
> +#ifdef __NR_ipc
> + SYS_ipc }, /* 117 */
> +#endif
> { 1, TD, sys_fsync, "fsync" }, /* 118 */
> { 1, TS, sys_sigreturn, "sigreturn" }, /* 119 */
> { 5, TP, sys_clone, "clone", SYS_clone }, /* 120 */
>
> If either or both of __NR_socketcall and __NR_ipc are not defined, then the
> closing curly braces are omitted, thus resulting in incorrect C statements.
>
> The attached patch to r19112 solves the issue.
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +0/33 662376056 | Software Designer | \ / CAMPAIGN | ^ |
| --==< ?_? >==-- ?------------.-------: X AGAINST | /e\ There is no |
| http://ymorin.is-a-geek.org/ | (*_*) | / \ HTML MAIL | """ conspiracy. |
?------------------------------?-------?------------------?--------------------?
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH] make strace buildable
2007-07-24 17:33 ` Yann E. MORIN
@ 2007-07-24 17:53 ` Bernhard Fischer
0 siblings, 0 replies; 6+ messages in thread
From: Bernhard Fischer @ 2007-07-24 17:53 UTC (permalink / raw)
To: buildroot
On Tue, Jul 24, 2007 at 07:33:05PM +0200, Yann E. MORIN wrote:
>Hello!
>
>Any taker?
Merged as r19254.
Still catching up with my mail-backlog, sorry..
Thanks!
cheers,
Bernhard
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH] make strace buildable
2007-07-16 20:32 [Buildroot] [PATCH] make strace buildable Yann E. MORIN
2007-07-24 17:33 ` Yann E. MORIN
@ 2007-09-20 14:45 ` Bernhard Fischer
2007-09-20 14:48 ` Bernhard Fischer
1 sibling, 1 reply; 6+ messages in thread
From: Bernhard Fischer @ 2007-09-20 14:45 UTC (permalink / raw)
To: buildroot
Yann,
On Mon, Jul 16, 2007 at 10:32:43PM +0200, Yann E. MORIN wrote:
>Hello all!
>
>The second hunk in package/strace/strace-undef-syscall.patch is incorrect:
>
>--- strace-4.5.15.orig/linux/syscallent.h 2007-02-04 19:03:48.000000000 +0100
>+++ strace-4.5.15/linux/syscallent.h 2007-02-04 19:11:04.000000000 +0100
>@@ -130,7 +130,10 @@
> { 2, TF, sys_statfs, "statfs" }, /* 99 */
> { 2, TD, sys_fstatfs, "fstatfs" }, /* 100 */
> { 3, 0, sys_ioperm, "ioperm" }, /* 101 */
>- { 2, TD, sys_socketcall, "socketcall", SYS_socketcall }, /* 102 */
>+ { 2, TD, sys_socketcall, "socketcall",
>+#ifdef __NR_socketcall
>+ SYS_socketcall }, /* 102 */
>+#endif
> { 3, 0, sys_syslog, "syslog" }, /* 103 */
> { 3, 0, sys_setitimer, "setitimer" }, /* 104 */
> { 2, 0, sys_getitimer, "getitimer" }, /* 105 */
>@@ -145,7 +148,10 @@
> { 4, TP, sys_wait4, "wait4", SYS_wait4 }, /* 114 */
> { 1, 0, sys_swapoff, "swapoff" }, /* 115 */
> { 1, 0, sys_sysinfo, "sysinfo" }, /* 116 */
>- { 6, 0, sys_ipc, "ipc", SYS_ipc }, /* 117 */
>+ { 6, 0, sys_ipc, "ipc",
>+#ifdef __NR_ipc
>+ SYS_ipc }, /* 117 */
>+#endif
> { 1, TD, sys_fsync, "fsync" }, /* 118 */
> { 1, TS, sys_sigreturn, "sigreturn" }, /* 119 */
> { 5, TP, sys_clone, "clone", SYS_clone }, /* 120 */
>
>If either or both of __NR_socketcall and __NR_ipc are not defined, then the
>closing curly braces are omitted, thus resulting in incorrect C statements.
>
>The attached patch to r19112 solves the issue.
>
>Regards,
>Yann E. MORIN.
>
>--
>.-----------------.--------------------.------------------.--------------------.
>| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
>| +0/33 662376056 | Software Designer | \ / CAMPAIGN | ^ |
>| --==< ?_? >==-- ?------------.-------: X AGAINST | /e\ There is no |
>| http://ymorin.is-a-geek.org/ | (*_*) | / \ HTML MAIL | """ conspiracy. |
>?------------------------------?-------?------------------?--------------------?
>Index: buildroot/package/strace/strace-undef-syscall.patch
>===================================================================
>--- buildroot/package/strace/strace-undef-syscall.patch (revision 19112)
>+++ buildroot/package/strace/strace-undef-syscall.patch (working copy)
>@@ -15,27 +15,29 @@
> { 4, 0, sys_epoll_ctl, "epoll_ctl" }, /* 255 */
> --- strace-4.5.15.orig/linux/syscallent.h 2007-02-04 19:03:48.000000000 +0100
> +++ strace-4.5.15/linux/syscallent.h 2007-02-04 19:11:04.000000000 +0100
>-@@ -130,7 +130,10 @@
>+@@ -130,7 +130,11 @@
> { 2, TF, sys_statfs, "statfs" }, /* 99 */
> { 2, TD, sys_fstatfs, "fstatfs" }, /* 100 */
> { 3, 0, sys_ioperm, "ioperm" }, /* 101 */
> - { 2, TD, sys_socketcall, "socketcall", SYS_socketcall }, /* 102 */
>-+ { 2, TD, sys_socketcall, "socketcall",
>++ { 2, TD, sys_socketcall, "socketcall"
> +#ifdef __NR_socketcall
>-+ SYS_socketcall }, /* 102 */
>++ , SYS_socketcall
> +#endif
>++ }, /* 102 */
Hm. I just ment to push this change upstream but..
Now if there is no socketcall, where does that closing curly bracket
come from, again?
I think it should be right here, but i can't see it?
> { 3, 0, sys_syslog, "syslog" }, /* 103 */
> { 3, 0, sys_setitimer, "setitimer" }, /* 104 */
> { 2, 0, sys_getitimer, "getitimer" }, /* 105 */
>-@@ -145,7 +148,10 @@
>+@@ -145,7 +149,11 @@
> { 4, TP, sys_wait4, "wait4", SYS_wait4 }, /* 114 */
> { 1, 0, sys_swapoff, "swapoff" }, /* 115 */
> { 1, 0, sys_sysinfo, "sysinfo" }, /* 116 */
> - { 6, 0, sys_ipc, "ipc", SYS_ipc }, /* 117 */
>-+ { 6, 0, sys_ipc, "ipc",
>++ { 6, 0, sys_ipc, "ipc"
> +#ifdef __NR_ipc
>-+ SYS_ipc }, /* 117 */
>++ , SYS_ipc
> +#endif
>++ }, /* 117 */
ditto. +}, is missing, no?
> { 1, TD, sys_fsync, "fsync" }, /* 118 */
> { 1, TS, sys_sigreturn, "sigreturn" }, /* 119 */
> { 5, TP, sys_clone, "clone", SYS_clone }, /* 120 */
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH] make strace buildable
2007-09-20 14:45 ` Bernhard Fischer
@ 2007-09-20 14:48 ` Bernhard Fischer
2007-09-20 16:34 ` Yann E. MORIN
0 siblings, 1 reply; 6+ messages in thread
From: Bernhard Fischer @ 2007-09-20 14:48 UTC (permalink / raw)
To: buildroot
On Thu, Sep 20, 2007 at 04:45:27PM +0200, Bernhard Fischer wrote:
>Yann,
>> { 3, 0, sys_ioperm, "ioperm" }, /* 101 */
>> - { 2, TD, sys_socketcall, "socketcall", SYS_socketcall }, /* 102 */
>>-+ { 2, TD, sys_socketcall, "socketcall",
>>++ { 2, TD, sys_socketcall, "socketcall"
>> +#ifdef __NR_socketcall
>>-+ SYS_socketcall }, /* 102 */
>>++ , SYS_socketcall
>> +#endif
>>++ }, /* 102 */
----------------------------------------------------------------^
It's right here on the far right. Everything is fine, sorry for the
noise..
>
>Hm. I just ment to push this change upstream but..
>Now if there is no socketcall, where does that closing curly bracket
>come from, again?
>I think it should be right here, but i can't see it?
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH] make strace buildable
2007-09-20 14:48 ` Bernhard Fischer
@ 2007-09-20 16:34 ` Yann E. MORIN
0 siblings, 0 replies; 6+ messages in thread
From: Yann E. MORIN @ 2007-09-20 16:34 UTC (permalink / raw)
To: buildroot
Bernhard,
On Thursday 20 September 2007 16:48:44 Bernhard Fischer wrote:
> On Thu, Sep 20, 2007 at 04:45:27PM +0200, Bernhard Fischer wrote:
> >>++ }, /* 102 */
> ----------------------------------------------------------------^
> It's right here on the far right. Everything is fine, sorry for the
> noise..
No problem... It also took me two minutes to find it again! ;-)
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +0/33 662376056 | Software Designer | \ / CAMPAIGN | ^ |
| --==< ?_? >==-- ?------------.-------: X AGAINST | /e\ There is no |
| http://ymorin.is-a-geek.org/ | (*_*) | / \ HTML MAIL | """ conspiracy. |
?------------------------------?-------?------------------?--------------------?
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2007-09-20 16:34 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-07-16 20:32 [Buildroot] [PATCH] make strace buildable Yann E. MORIN
2007-07-24 17:33 ` Yann E. MORIN
2007-07-24 17:53 ` Bernhard Fischer
2007-09-20 14:45 ` Bernhard Fischer
2007-09-20 14:48 ` Bernhard Fischer
2007-09-20 16:34 ` Yann E. MORIN
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox