* [Bug translator/22278] New: the nss client code doesn't handle '-I DIR' well
@ 2017-10-10 16:44 dsmith at redhat dot com
2017-10-10 16:47 ` [Bug translator/22278] " fche at redhat dot com
` (5 more replies)
0 siblings, 6 replies; 7+ messages in thread
From: dsmith at redhat dot com @ 2017-10-10 16:44 UTC (permalink / raw)
To: systemtap
https://sourceware.org/bugzilla/show_bug.cgi?id=22278
Bug ID: 22278
Summary: the nss client code doesn't handle '-I DIR' well
Product: systemtap
Version: unspecified
Status: NEW
Severity: normal
Priority: P2
Component: translator
Assignee: systemtap at sourceware dot org
Reporter: dsmith at redhat dot com
Target Milestone: ---
Using a custom tapset directory works fine:
====
# cat tapset/foo.stp
function foo()
{
printf("Hello\n")
}
# stap -I tapset -e 'probe oneshot { foo() }'
Hello
====
However, if you try to do the same thing with a compile server, you get an
error:
====
# stap --use-server -I tapset -e 'probe oneshot { foo() }'
semantic error: unresolved function (similar: log, HZ, pn, pp, cpu): identifier
'foo' at <input>:1:17
source: probe oneshot { foo() }
^
WARNING: unused command line option $1/@1
Pass 2: analysis failed. [man error::pass2]
Passes: via server failed. Try again with another '-v' option.
====
--
You are receiving this mail because:
You are the assignee for the bug.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug translator/22278] the nss client code doesn't handle '-I DIR' well
2017-10-10 16:44 [Bug translator/22278] New: the nss client code doesn't handle '-I DIR' well dsmith at redhat dot com
@ 2017-10-10 16:47 ` fche at redhat dot com
2017-10-10 17:02 ` brolley at redhat dot com
` (4 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: fche at redhat dot com @ 2017-10-10 16:47 UTC (permalink / raw)
To: systemtap
https://sourceware.org/bugzilla/show_bug.cgi?id=22278
Frank Ch. Eigler <fche at redhat dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |fche at redhat dot com
--- Comment #1 from Frank Ch. Eigler <fche at redhat dot com> ---
The http client/server pair should try to handle this case (maybe by zipping up
that -I path). Wouldn't worry about the old client/server.
--
You are receiving this mail because:
You are the assignee for the bug.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug translator/22278] the nss client code doesn't handle '-I DIR' well
2017-10-10 16:44 [Bug translator/22278] New: the nss client code doesn't handle '-I DIR' well dsmith at redhat dot com
2017-10-10 16:47 ` [Bug translator/22278] " fche at redhat dot com
@ 2017-10-10 17:02 ` brolley at redhat dot com
2017-10-10 17:32 ` dsmith at redhat dot com
` (3 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: brolley at redhat dot com @ 2017-10-10 17:02 UTC (permalink / raw)
To: systemtap
https://sourceware.org/bugzilla/show_bug.cgi?id=22278
Dave Brolley <brolley at redhat dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |brolley at redhat dot com
--- Comment #2 from Dave Brolley <brolley at redhat dot com> ---
The old client/server should already handle this, by zipping up the -I
directories, exactly as you have suggested.
--
You are receiving this mail because:
You are the assignee for the bug.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug translator/22278] the nss client code doesn't handle '-I DIR' well
2017-10-10 16:44 [Bug translator/22278] New: the nss client code doesn't handle '-I DIR' well dsmith at redhat dot com
2017-10-10 16:47 ` [Bug translator/22278] " fche at redhat dot com
2017-10-10 17:02 ` brolley at redhat dot com
@ 2017-10-10 17:32 ` dsmith at redhat dot com
2017-10-10 17:38 ` dsmith at redhat dot com
` (2 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: dsmith at redhat dot com @ 2017-10-10 17:32 UTC (permalink / raw)
To: systemtap
https://sourceware.org/bugzilla/show_bug.cgi?id=22278
--- Comment #3 from David Smith <dsmith at redhat dot com> ---
(In reply to Frank Ch. Eigler from comment #1)
> The http client/server pair should try to handle this case (maybe by zipping
> up that -I path). Wouldn't worry about the old client/server.
The nss client/server does handle this case, it just doesn't handle it well. It
does copy the tapset directory over, but mangles the command line. If I'm
reading the "client" directory correctly, with --use-server the stap command
line that goes over to the server ends up looking like:
# stap tapset/home/dsmith/tapset -I tapset -v -e 'probe oneshot { foo() }'
which is clearly wrong.
The http client/server shares code/ideas with the nss client/server, and I'm
sure the same problem exists in the http client/server. I was wondering how the
nss client/server handled this when I found the problem.
--
You are receiving this mail because:
You are the assignee for the bug.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug translator/22278] the nss client code doesn't handle '-I DIR' well
2017-10-10 16:44 [Bug translator/22278] New: the nss client code doesn't handle '-I DIR' well dsmith at redhat dot com
` (2 preceding siblings ...)
2017-10-10 17:32 ` dsmith at redhat dot com
@ 2017-10-10 17:38 ` dsmith at redhat dot com
2017-10-10 18:29 ` brolley at redhat dot com
2017-10-17 13:50 ` dsmith at redhat dot com
5 siblings, 0 replies; 7+ messages in thread
From: dsmith at redhat dot com @ 2017-10-10 17:38 UTC (permalink / raw)
To: systemtap
https://sourceware.org/bugzilla/show_bug.cgi?id=22278
--- Comment #4 from David Smith <dsmith at redhat dot com> ---
(In reply to Dave Brolley from comment #2)
> The old client/server should already handle this, by zipping up the -I
> directories, exactly as you have suggested.
Right, but nss_client_backend::include_file_or_directory() is always adding an
included file/diretory to the command line, which isn't correct.
(It is certainly possible I introduced this bug when I generalized the client
code to be able to include the http code.)
--
You are receiving this mail because:
You are the assignee for the bug.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug translator/22278] the nss client code doesn't handle '-I DIR' well
2017-10-10 16:44 [Bug translator/22278] New: the nss client code doesn't handle '-I DIR' well dsmith at redhat dot com
` (3 preceding siblings ...)
2017-10-10 17:38 ` dsmith at redhat dot com
@ 2017-10-10 18:29 ` brolley at redhat dot com
2017-10-17 13:50 ` dsmith at redhat dot com
5 siblings, 0 replies; 7+ messages in thread
From: brolley at redhat dot com @ 2017-10-10 18:29 UTC (permalink / raw)
To: systemtap
https://sourceware.org/bugzilla/show_bug.cgi?id=22278
--- Comment #5 from Dave Brolley <brolley at redhat dot com> ---
IIRC, the modified command line was deliberate and it is supposed to be
suitable for finding the unzipped -I directories on the server side.
--
You are receiving this mail because:
You are the assignee for the bug.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Bug translator/22278] the nss client code doesn't handle '-I DIR' well
2017-10-10 16:44 [Bug translator/22278] New: the nss client code doesn't handle '-I DIR' well dsmith at redhat dot com
` (4 preceding siblings ...)
2017-10-10 18:29 ` brolley at redhat dot com
@ 2017-10-17 13:50 ` dsmith at redhat dot com
5 siblings, 0 replies; 7+ messages in thread
From: dsmith at redhat dot com @ 2017-10-17 13:50 UTC (permalink / raw)
To: systemtap
https://sourceware.org/bugzilla/show_bug.cgi?id=22278
David Smith <dsmith at redhat dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--- Comment #6 from David Smith <dsmith at redhat dot com> ---
This worked in 3.1. Commit 586c8666a, which split up the client functionality,
caused this bug. Test added so that we'll hopefully notice if this breaks
again. Fixed in commit 504364331
--
You are receiving this mail because:
You are the assignee for the bug.
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2017-10-17 13:50 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-10 16:44 [Bug translator/22278] New: the nss client code doesn't handle '-I DIR' well dsmith at redhat dot com
2017-10-10 16:47 ` [Bug translator/22278] " fche at redhat dot com
2017-10-10 17:02 ` brolley at redhat dot com
2017-10-10 17:32 ` dsmith at redhat dot com
2017-10-10 17:38 ` dsmith at redhat dot com
2017-10-10 18:29 ` brolley at redhat dot com
2017-10-17 13:50 ` dsmith at redhat dot com
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).