PR3823: use our own autosprintf()

* configure.ac, Makefile.am: Forget about libasprintf.
* main.cxx, session.cxx, session.h: Likewise.
* util.h, util.cxx: Define workalike autosprintf(), with a fresh
  new look, and with no added memory leaks!  WOW!

PR3823: regenerate po/en.po and en.gmo

Using msginit.

PR3823: remove macos-specific configury goo

PR3823: redo autoconfigury with gettextize

* Makefile.am, configure.ac: Use autoconf/automake gettext support.
* config.rpath, config.sub, ABOUT.NLS: New files.
* m4/*.  Many new files, copied by gettextize.
* po/*.  Many new files, mostly from gettextize.
* po/LINGUAS, po/POTFILES.in, po/systemtap.pot, po/en.po, po/en.gmo: New files.

Silence the closing of the ssh control master

* remote.cxx (ssh_remote::close_control_master): Tell stap_system to
  redirect stdout and stderr to /dev/null, so we don't see the insistent
  "Exit request sent" message.

Build the staprun command as an argument vector

* buildrun.cxx (make_run_command): Build the command as a vector.
* util.cxx (cmdstr_join): New, join a vector as quoted strings.
* remote.cxx (direct::start): Use make_run_command directly.
  (ssh_remote::start): Pass cmdstr_join(make_run_command(...)) to
  ssh, so all args survive intact to the remote side.

Don't quote the module path in vectored args

* buildrun.cxx (compile_pass): Remove the quotes from M=path, as they
  are treated literally by Kbuild.

Remove the flat-string stap_system altogether

* util.cxx (stap_system): Remove the flat-string version.  Add
  booleans to the vectored version for whether stdout/err should
  be redirected to /dev/null.
* buildrun.cxx (run_make_cmd): Take the cmd as a vector, and use the
  same nulling flags to pass along to stap_system.
  (compile_pass): Build a vectored make_cmd.
  (verify_uprobes_uptodate): Ditto.
  (make_uprobes): Ditto.
  (make_tracequery): Ditto.
  (make_typequery_kmod): Ditto.
  (make_typequery_umod): Build a vectored gcc command.
* csclient.cxx (compile_server_client::package_request): Manually wrap
  this compound command with an sh -c vector.  Add quoting to the client
  tmpdir and zipfile, just in case.

Convert more stap_system calls to vector form

* buildrun.cxx (kernel_built_uprobes): Use a vectored command.
  (copy_uprobes_symbols): Ditto.
* csclient.cxx (compile_server_client::unpack_response): Ditto.
* main.cxx (remove_temp_dir): Ditto.

The logger pmda now asks for a logfile to monitor.

* pcp/src/pmdas/logger/logger.h: New file.
* pcp/src/pmdas/logger/loggerMain.c: New custom main loop.
* pcp/src/pmdas/logger/logger.c (main): Added option for logfile to
  monitor and calls new custom main loop.
* pcp/src/pmdas/logger/Install: Asks for logfile name to monitor.
* pcp/src/pmdas/logger/GNUmakefile: Added loggerMain.c and logger.h.

Allow capitals in @cast headers

The conversion to regexp_match lost the REG_ICASE, so we need to
manually allow capitals in the regular expression.

* buildrun.cxx (make_typequery): Add A-Z to the character class.

Start using arg vectors for stap_system and friends

Instead of worrying about quoting properly in every command string, we
can rest easier by launching processes with a proper argv[].  All of the
stap_system-like functions are converted, but a flat-string stap_system
is also kept around for now while callers are converted.

* util.cxx (stap_spawn): Take a vector<string> for command arguments,
  convert that to an argv[], and use that for posix_spawnp.  Note the
  conversion to spawnp as well so PATH-searching is still performed.
  (stap_system, stap_system_read): Convert to vectors, but keep a flat
  version of the former for compatibility.
  (probe stap_system__start): Move to stap_spawn, to probe all flavors.
  (probe stap_system__complete): Likewise, move to stap_waitpid, to
  probe when any process is finished; add a pid to track who's done.
* remote.cxx (direct, ssh_remote): Convert all calls to vector form.

Use posix_spawn_file_actions for stap_system_read

The file actions can handle the dup2 necessary to hook a pipe onto the
child's stdout.  It could do the necessary closing too, but we'll just
use FD_CLOEXEC for that.

* util.cxx (stap_spawn): Take file_actions for the posix_spawn.
  (pipe_child_fd): New, open a pipe and set the dup2 action for it.
  (stap_system_read): Let stap_spawn and pipe_child_fd do the hard part.

Close the ssh ControlMaster after uname failure

Since ssh_remote::get_uname is called from ctors, any exception won't
call the dtor to close the connection.

* remote.cxx (ssh_remote::init): Open the master, try uname, and close
  the master on exception.
  (ssh_remote::ssh_remote): Both ctors, use init().

Read the pipe directly in stap_system_read

* util.cxx (stap_system_read): Use gnu's stdio_filebuf to read directly
  from the pipe file descriptor.

The logger pmda now builds outside of the pcp source tree.

* pcp/.gitignore: New file.
* pcp/config.guess: Ditto.
* pcp/config.sub: Ditto.
* pcp/configure.in: Ditto.
* pcp/install-sh: Ditto.
* pcp/src/include/.gitignore: Ditto.
* pcp/src/include/builddefs.in: Ditto.
* pcp/src/include/buildrules: Ditto.
* pcp/src/pmdas/logger/.gitignore: Updated.
* pcp/src/pmdas/logger/GNUmakefile: Modified to build either in the pcp
  source tree or separately.
* pcp/src/pmdas/logger/Install: Updated.
* pcp/src/pmdas/logger/percontext.c: Updated include file paths.

More improvements to the client.exp test harness.

Changes to account for unanticipated, but correct, output.

Initial version of pcp logger pmda.

Fix RHBZ#676474 by removing some incorrect asmlinkage() markers.

* tapset/nd_syscalls.stp: Remove asmlinkage() for do_fork, do_execve,
  compat_do_execve.

Test harness improvements for testing using a compile server.

- Always generate and authorize new server cert when a server starts.
- Only consider online servers when testing for trust in client.exp.

Don't report IPv6 entries returned by avahi, for now.

Fix a README typo of CONFIG_MODULE_UNLOAD

(reported by "wil_" on #systemtap)

Fix the indentation of systemtap_session::usage

Clean up exit paths

We need to strive not to leave files/processes around, so exit() calls
are reduced to only places where nothings been allocated yet.  Errors
are rarely so fatal to require immediate exit.

Use a ControlMaster for ssh remotes

We can start the ssh remote with a ControlMaster connection, using a
unique socket in the tmpdir, and then each subsequent remote command can
reuse that connection.

* remote.cxx (ssh_remote::open_control_master): Connect to the remote
  host with a ControlPath that later commands can reuse.  The relevant
  options are saved in member ssh_opts.
  (ssh_remote::close_control_master): Ask the control process to exit.
  (ssh_remote::get_uname, ::start, ::finish): Start using ssh_opts, so
  the ControlPath is always specified.

Precreate the main tmpdir

Create the primary temp dir early on, so we have a place to store
temporary data for remotes (like control socket paths).

* main.cxx (main): Create the tmpdir before remotes are started.
  (create_temp_dir, remove_temp_dir): Tolerate repeat calls.

Use a proper session copier when cloning

* session.cxx (systemtap_session): Define copy and assignment
  constructors without implementation, to avoid implicit behavior.
  Add a new copy constructor with a new kernel/arch.
  (systemtap_session::clone): Use the new copier.

Rework setup_server to ensure that servers get shut down on error.

Also reqorganize in order to save more output from server smoke tests.

accept and ignore %l (long) option in *printf family

* staptree.cxx (print_format::string_to_components): Parse 'l' type modifier.
* int1.stp: Test '#' flag and 'l' type modifier.
* int1.exp: Likewise.

Merge branch 'master' of ssh://sourceware.org/git/systemtap

Conflicts:
main.cxx

PR 11844 - --use-server-on-error option

- Now defaults to 'off'
- Errors no longer automatically result in retry using the server.
  This is done only for certain whitelisted errors.
  systemtap_session::set_try_server() is called for these errors.

Fix two more 'temporary' typos

Fix typo in message (temorary -> temporary) which broke compile server.

stap-server-connect uses this message to locate the temp directory.

Add NEWS item for process.mark $$parms

Support $$parms for sdt.

* tapsets.cxx (sdt_uprobe_var_expanding_visitor): Add
visit_target_symbol_arg visit_target_symbol_context
(visit_target_symbol_context): Handle $$name, $$provider, $$parms
(visit_target_symbol_arg): Handle symbol
* sdt_misc.exp:  Test $$parms.

PR 12428 - Document Compile Server/Client Setup and Admin

- New README.unprivileged
- Update info about unprivileged mode in stap.1 and in stap-server.8
- Don't bother autoconfing paths in stap-server.8. Refer to stappaths
  instead.

Add gettext-devel build requirement to spec file.

Avoid using $env(HOSTNAME)

When running crontab jobs the systemtap.server/server.exp tests
would not run because $env(HOSTNAME) is unset. HOSTNAME is
set by the bash shell and is not available in crontab run tests.
An equivalent [info hostname] is used in its place.

Revert accidental gettext clobbering

Wrapped lines, fixed some white space issues.

Some comments with the original strings have been left in for
ease of comparison. They will be removed for the final I18N commit.

gettext changes in main.cxx and session files

to invoke run $ xgettext -c -k_ main.cxx session.*
not every string has been set up for gettext translation, at this
point it is just to make sure it doesn't break stap.  The more
complicated strings will be I18N in time.

Autoconf groundwork for I18N of systemtap

TODO: localedir needs to be a variable and not hard-coded
further changes to files for .po file output are upcoming.

Cleanup to improve sdt on ppc.

* tapsets.cxx (sdt_uprobe_var_expanding_visitor::visit_target_symbol):
Use variable shift amount.  Don't cast unless probe type 3.
* sdt_misc.exp (type_flags): Don't use sup_flags
* sdt_types.c: Check __SIZEOF_SIZE_T__ for long long tests.

Assert that TMPDIR, if set, is a boring path

We don't want any excitement in the paths we must use.
e.g. Kbuild doesn't cope well with spaces in the M="/module path"

* session.cxx (systemtap_session::check_options): Restrict the
  characters allowed in TMPDIR to a universally-safe set.
* testsuite/parseko/cmdline22.stp: New, check that TMPDIR spaces are
  not allowed, even if that path really exists.

Merge branch 'master' of ssh://sourceware.org/git/systemtap

Ensure systemtap.stress/conversions.exp generates individual errors and warnings

PR12168 eliminated duplicated error messages and changed the count of
ERROR and WARNING messages. Adding the "-vv" option turns off the
duplication eliminate and allows an accurate count of the number
of times ERROR and WARNING messages occur.

PR 11844 - --use-server-on-error option

- Mechanism is in session.cxx, session.h.
- Code in main.cxx:passes_0_4 calls set_try_server upon compilation
  errors in phases 0-4.
- set_try_server (systemtap_session:dont_try_server) can be set for
  individual errors which do not warrant a retry (e.g. syntax errors)
  to override.

Make sure NSS and NSPR are properly initialized and shutdown.

- rpmReadConfigFiles initializes NSS, but does not shut it down.
  We now call rpmFreeCrypto if available or call NSS_Shutdown directly otherwise
  in missing_rpm_enlist
- In some versions of NSPR, PR_Init must be called once and only once before NSS_Init or
  NSS_InitReadOnly is called. In later versions, it is unnecessary and is ignored.
  systemtap_session::NSPR_Initialized is now static so that it is
  shared by all active session objects.

Remove unused variables and calls which initialized them.

Fixed BZ559599 by making target_set.stp use syscall probes for ia64.

* tapset/target_set.stp: Dwarfless probes don't work on ia64, so fall back
  to the 'syscall' probes (as we do for kernels < 2.6.18).

Fixed BZ661424 by checking for $prev_p first in __scheduler.ctxswitch.kp.

* tapset/scheduler.stp: Checks for '$prev_p' before checking for '$prev'
  so that i386 RHEL5 (which has both arguments) works correctly.

BZ673085: amplify documentation on initialization of probe alias variables

Avoid use of limits include in sdt.  Sign extend signed HI/SI values.

* sdt.h (limits): Remove.
(__sdt_type): Predefine type signness.
* tapsets.cxx (sdt_uprobe_var_expanding_visitor::visit_target_symbol):
Sign extend signed HI/SI values.
* sdt_asm.stp (rN): Add rNd tests

stap-report: also add a dummy "stap -vv -p4 ..." run

stap-report: dump more state

$XDG_DATA_* since that can interfere, and /proc*/*/kprobes*, for completeness

GCC 4.6 unused-variable warnings

We set a couple of variables that later code doesn't use.  GCC 4.6
warns about them, which our C*FLAGS=-Werror turned into a build failure.
Worked around these by adding a (void) var; to add a dummy use.

adding #ifdef CLONE_STOPPED to stop module compilation error

Audit some minus/dash/hyphen use in manpages

Groff treats "-" as a hyphen, U+2010.  When the intention is to produce
a literal ASCII minus sign, U+002D, it must be "\-".  This matters for
command-line options and examples to be copyable from the manpage.

I've updated cases that should obviously be escaped.  At a glance, I
think we have other cases that could be normal hyphens, not escaped, but
that's less of an issue.

Add the cache options from commit d105f66 to stap.1

Add support for real URI syntax in remote targets

We still only support SSH for now, but this opens the door for other
protocols.  All of the following should work:

  --remote host
  --remote user@host
  --remote ssh://host/
  --remote ssh://user@host/

URI paths, queries, and fragments are all parsed, but don't have any
meaning yet, so they're treated as an error.  I can imagine using this
for protocol-specific options though, e.g. ssh://host/?Compression=yes

* remote.cxx (class uri_decoder): New, breaks down a URI string.
  (ssh_remote::ssh_remote): Add a constructor using a uri_decoder.
  (remote::create): Redirect based on the scheme of URI-looking remote
  targets.  Assume all others are still plain [user@]host for ssh.

PR 11862: remove obsolete stap-client script and friends

Rework and improve warnings for missing NSS and avahi headers, libs and tools

PR 11862: remove obsolete stap-client script and friends

- Integrate stap-client-connect.c into csclient.cxx
- Remove stap-client-connect.c

PR 11862: remove obsolete stap-client script and friends

- Remove deprecated scripts
- Remove parts of stap-env used only by the removed scripts
- Remove man pages for removed scripts
- Remove parts of stap-client-connect which allowed it to be built
  as a standalone tool (used by stap-client).
- Update spec file. systemtap-client package removed.

Improve sdt asm test coverage.

* sdt_asm.S:  Improve coverage of tested registers and values.
* sdt_asm.stp: Likewise.

PR12413, Provide TID information in kprocess.create and kprocess.release

PR12412: Make the text agree with the example (should be reads)

Allow sdt to avoid application defined min/max macros.

* sdt.h:  Surround limits inclusion with push_macro and pop_macro.

PR12419: fix x86 uprobes decoding of "repnz ret"

BZ669082: support powerpc prior to 2.6.15, resolving conflict with PR4186

* buildrun.cxx (compile_pass): Omit passing of ARCH=foo for
  ancient powerpc.  This enables native ARCH=ppc64 builds,
  but sacrifices ppc-target cross-architecture cross-compilation.
  (make_tracequery_kmod, make_typequery_kmod): Ditto.

PR12411 cont'd: use enum type decl

* tapsets.cxx (sdt_uprobe_var_expanding_visitor): Make regwidths
  an enum type decl, not an anonymous-enum-typed variable.  Move
  DRI macro #define/#undef just around its use.

RHBZ669751: old ia64 kernels lack task_pt_regs()

... so define it in terms of ia64_task_regs()

Switch to using a register width table instead of heuristic code.

       * tapsets.cxx (sdt_uprobe_var_expanding_visitor): Add DRI and regwidths.
       Change dwarf_regs to be a regname/regwidth pair.  Change initializer.
       * (sdt_uprobe_var_expanding_visitor::visit_target_symbol):  Use it.

Fixed BZ669740 by making target_set.stp use syscall probes in kernels < 2.6.18.

* tapset/target_set.stp: Dwarfless probes don't work in kernels < 2.6.18.
  So, fall back to 'syscall' probes for those kernels (instead of
  'nd_syscall' probes.

PR 11445: Check that assert_is_my_proc check is within all uprobes/utrace callback entry points

Add comment explaining why a check for is_myproc() is not generated as a
safety net for process.end probes and their variants.

systemtap.apps infrastructure fixes.

mysql.exp (stap-mysql.sh): Use push/pop_macro to avoid mysql max/min macros.
xulrunner.exp (stap-xul.sh): Make JAVA_HOME platform independent.

Cleanup warnings caused by CFLAGS correction.

Merge branch 'master' of ssh://sources.redhat.com/git/systemtap

Conflicts:
configure
testsuite/configure

Configuration problems.

- CFLAGS was being cleared in configure.ac (oldCFLAGS typo)
- CXXFLAGS was not always being set (CXFXLAGS typo)
- pick up $(AM_CFLAGS) and @PIECFLAGS@ in all C compilations in Makefile.am

bump version numbers to 1.5, for future release cycle

datestamp release

README: build instructions suggest elfutils version 0.151

PR12399: make sdt.h semaphores visibility("hidden")

NEWS: sdt.h v3

PR 10657: Document changes in NEWS and in stap.1.

Fix myproc-unprivileged in embedded_expr

The macro assert_is_myproc() forms a statement, so we need to wrap it
in ({}) to use it in an expression list.

* translate.cxx (c_unparser::visit_embedded_expr): Wrap stmt as expr.

PR 10657 - semi-unprivileged tapset embedded-C functions.

Including the comment /* myproc-unprivileged */ within an embedded C
tapset function or embedded C expression makes it available for use
by unprivileged users and also automatically adds a call to
assert_is_myproc() within the function or expression (i.e. it is
equivalent to including the comment /* unprivileged */ and
manually adding the the call to assert_is_myproc).

All instances within the tapset source tree have been updated.

PR11756 - adding warnings for deprecated declarations
Added the --version-check  flag, this will check if systemtap
constructors being used are version dependent. If so, a warning
detailing what syntax element is displayed.

PR12115: defeat uninitialized-variable warnings

PR12115: defeat uninitialized-variable warning

NEWS: tweak --remote blurb wording

AUTHORS bump

Fix for undefined STAPCONF_PERF_STRUCTPID

Fix paragraph indentation of ssl options

Disable auto-unpriv when --remote is used

PR12115: Document --remote in NEWS, stap.1, and --help

Merge branch 'master' into remote

Integrate sdt-compat.h into testsuite sdt.h

Makefile.am: Use sdt_types.h instead of sdt-compat.h
Makefile.in: Likewise.
tapsets.cxx: Likewise.
testsuite/sys/sdt.h: Integrate sdt-compat.h.
sdt-compat.h: Remove.
sdt_types.h: New.

PR11203: Provide default init functions for initscript/systemtap.

Used on plattforms where /etc/rc.d/init.d/functions is not available.

PR11203: Provide default init functions for stap-server.

Used on plattforms where /etc/rc.d/init.d/functions is not available.

tapset compat: irq tracepoints to use @cast() for $action