Audit possible string buffer overruns.

* runtime/transport/procfs.c (_stp_register_ctl_channel_fs): Change
  sprintf() call to snprintf() call.
  (_stp_unregister_ctl_channel_fs): Ditto.
* runtime/transport/ring_buffer.c (_stp_transport_data_fs_init): Ditto.
* runtime/unwind.c (_stp_eh_enc_name): Ditto.
* runtime/io.c: Make sure STP_LOG_BUF_LEN isn't too short.
* runtime/staprun/staprun_funcs.c: Add comments about strcpy/strcat/sprintf
  appropriateness.
  (check_signature): Fix string length calculation.
* runtime/staprun/common.c: Add comments about strcpy/strcat/sprintf
  appropriateness.
* runtime/staprun/modverify.c: Ditto.
* runtime/staprun/relay.c: Ditto.
* runtime/staprun/relay_old.c: Ditto.
* runtime/staprun/staprun_funcs.c: Ditto.

Redistribute the Target Variable information

Redistribute the information about target variables in the manual. Also
added index entries for the information and edited the text.

Fixed possible string buffer overruns in several embedded-C functions.

* tapset/socket.stp (sock_flags_num2str): Use strlcat() instead of
  strcat() to guard against buffer overruns.  Instead of creating the
  string in a temporary variable and then copying it to THIS->__retvalue,
  just create the string in THIS->__retvalue.
  (msg_flags_num2str): Ditto.
* tapset/inet_sock.stp (daddr_to_string): Use snprintf() instead of
  sprintf() to guard against buffer overruns.
* tapset/signal.stp (sigset_mask_str): Ditto.
* tapset/s390/nd_syscalls.stp (get_32mmap_args): Use strlcat()/snprintf()
  instead of strcat()/sprintf() to guard against buffer overruns.
* tapset/s390/syscalls.stp: Removed unused function get_32mmap_args().
* tapset/aux_syscalls.stp: Minor string-related fixes.
* tapset/string.stp: Ditto.

PR 11906: stap-server prints more info about each server acted upon.

PR 11906: Include the user running each server in the status output.

PR 11197: Elaborate on security prophylactic settings for stap-server.

stap-server.8.in now elaborates on how to achieve the recommended
security settings and points out which ones are the default.

Fixed BZ624657 by fixing timing issues in the initscript.

* initscript/systemtap.in (start_script): Make sure we wait until the pid
  file has been written before trying to use it.
  (stop_script): Check to see if the module has been really unloaded
  before declaring success.

PR 10922: Client/Server Testsuite problems when avahi (mDNS) is bkocked by the firewall.

The test suite now communicates with the server directly using
--use-serber=$HOSTNAME:PORT when avahi is blocked or not working.

PR12141: mention in README the elfutils release that fixes problem

Add configure --disable-sdt-probes

This new option turns off SDT markers in our own binaries.  This doesn't
change the ability to probe SDT in other applications.

* configure.ac: Add --disable-sdt-probes.
* runtime/staprun/configure.ac: Ditto.
* includes/stap-probe.h: Define PROBE[N] depending on ENABLE_SDT_PROBES.
* *.cxx, runtime/staprun/*.[ch]: Use the indirect PROBE macros instead
  of STAP_PROBE, so SDT can be disabled.
* (everything else): Regenerated with autoreconf.
  (The change is bigger due to my newer autotools on F14 -- sorry...)

Adapt to old kernels lacking asm/tsc.h

Some old x86_64 kernels do have an export for tsc_khz, but don't need
(or even have) asm/tsc.h to define it.

Add user_{int8,int16,uint16,int32,uint32,int64}.

* sdt.exp (extra_flags): Expand list of tested options.  Use instead
of having a separate c++ section.
(type_pedantics): New.

* sdt_misc.exp (extra_type_flags): Expand list of tested options.
(has_long_long, type_xfail, type_pedantics): New.

* sdt_types.c (NO_LONG_LONG): New.

Updated news regarding uprobes.ko and the compile-server/client.

PR10651 / RHBZ653286: mod_timer vs. del_timer_sync races

It appears possible for del_timer_sync (from outside) and mod_timer
(from within a timer callback) to race.  Defeat this race by ensuring
that the timer callback checks an atomic_t flag before rescheduling
itself with mod_timer.

* runtime/transport/relay_v2.c (transport_state): Turn into an atomic_t.
  Update users.
  (__stp_relay_wakeup_timer): Observe flag.
  (_stp_transport_data_fs_{start,stop}): Update flag before timer manipulations.
* runtime/transport/ring_buffer.c: Ditto for corresponding functions.

rhbz656699: use %ghost for /var/run/{systemtap,stap-server}

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

testsuite: fragment buildid.exp test driver, fixing objdir!=srcdir case

* testsuite/systemtap.base/buildid.exp: Evict embedded files into ...
* testsuite/systemtap.base/buildid.{c,hex,stp}: ... these new files.

build: don't include a <linux/version.h> in userspace builds

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

Conflicts:
runtime/staprun/common.c
session.cxx
session.h

Improved handling of uprobes.ko by compile-server and client.

- Server now returns uprobes.ko to client when required.
- New optional argument to staprun's -u option to specify the location
  of uprobes module.
- Client uses new argument of -u to specify uprobes.ko returned by
  the server.
- staprun still prefers already-loaded uprobes, built-in uprobes and
  kernel-built uprobes over the one returned by the server.
- staprun verifies the signature of the uprobes module for unprivileged
  users.

Rejigger sdt.h macros for more nit cases.

PR12138: Convert the user_int functions to kread

This uses our more-centralized mechanism for reading uncertain memory,
which also has the ability to read 8-byte values on i386 (not atomically).

* tapset/uconversions.stp: New, move the user_* functions here.  The new
  STP_GET_USER[_WARN] standardize all integer accesses through kread.
* tapset/conversions.stp: This is only for kernel functions now.
* doc/SystemTap_Tapset_Reference/tapsets.tmpl: Include uconversions.
* testsuite/buildok/conversions.stp: Build the user_intNN too.
* testsuite/buildok/conversions-embedded.stp: Ditto.

if we say sdt_types is untested, then don't test it...

PR12139: revamp sys/sdt.h macros' argument handling

* includes/sys/sdt.h: Revamp size and signedness calculations on probe
arguments for C and C++.

PR12139: test case for pointer to an incomplete type in probe argument

* testsuite/systemtap.base/sdt_types.c: Test a pointer to an incomplete type.
* testsuite/systemtap.base/sdt_types.stp: Use that new probe.
* testsuite/systemtap.base/sdt_misc.exp: Expect that.

Fix last commit.

Fix runcheck for dtrace cases

* Makefile.am (runcheck): Depend on ...
(testsuite/dtrace): ... this new target.
* Makefile.in: Regenerated.

Fix runcheck for dtrace cases

* Makefile.am (runcheck): Depend on ...
(testsuite/dtrace): ... this new target.
* Makefile.in: Regenerated.

Test sdt_types.c with c++.

sdt_misc.exp (type_language,type_language_mssgs,extra_type_mssgs) New.
Loop over for c++ testing.  Improve error messages.
sdt_types.stp:  Improve error messages.

Revert "PR12254: make ioblock.* probes tolerate 0-byte I/O (e.g. blkdev_issue_flush)"

This reverts commit d213fd5908edce8c730f08524eae15bc26d62187.
It's not necessary, as commit 3963dacdfe handles this already.

PR12254: make ioblock.* probes tolerate 0-byte I/O (e.g. blkdev_issue_flush)

* ioblock.stp (__bio_ino): Make all callers conditional on
  bi_vcnt != 0.

Test sdt_types.c with various gcc options.

sdt_misc.c (extra_type_flags): New. Test sdt_types.c with misc compile options.
Move sdt_misc.c, sdt_misc.stp, sdt_misc_.d to separate files.
sdt_types.c:  Make -pedantic friendly.  Add constants probe.
sdt_types.stp (constants): New.

work around rhel6 module signing nits in this builder script too

testing: add new script to generate   kernel.function("*")  equivalent module

releng: workaround for more rhel6 module-signing nits

* buildrun.cxx (make_typequery_kmod, make_tracequery): Also
  set CONFIG_MODULE_SIG=n in the destination makefile, on
  account of RHBZ 655231.

Make lex_cast work numerically with [u]int8_t

These were getting treated by istream like char types, which would
always fail our generic template since eof isn't reached.  There's not
much point to using lex_cast<char> anyway, since str[0] will do the
same, so I've specialized lex_cast with [u]int8_t to read numerically.

The only place we use this so far is in SDT V3 parsing constants.  SDT
doesn't yet emit 1-byte arguments, but it may in the future.

* util.h (lex_cast<int8_t>): Read via int16_t to stay numeric.
  (lex_cast<uint8_t>): Same, with uint16_t.

compat: adapt to old kernels' dislike of 'bool' type in runtime code

* runtime/sym.c (_stp_build_id_check): Take int instead of bool.  Update
  callers.

testsuite: unembed test sources

Pleading guilty to
* youthful indiscretions
* providing unadmirable examples

Paying penance by
* separating teeny little non-tcl sources from tcl code
* swearing to do better in the future

Updated stapusr/stapdev requirements in stap.1.

sdt_types: remove manual type-casting

For now, remove the int() and short() typecasting, as it seems to be
working without.  We should try as much as possible to fix these issues
in the translator and/or sdt.h before masking it in the test.

Cast the proper sign and size of SDT constant args

* tapsets.cxx (sdt_uprobe_var_expanding_visitor::visit_target_symbol):
  For the constant case, we still need to do some typecasting, as gcc
  doesn't necessarily match the sign of the constant to the type.

Cast the proper sign and size of SDT register args

* tapsets.cxx (sdt_uprobe_var_expanding_visitor::visit_target_symbol):
  For the REGISTER case, insert a typecast to deal with truncation and
  sign-extension of the argument as needed.

sdt_types: Use direct array args rather than &array when possible

sdt_types: add a PASS statement for bitfields_small_var

Don't autoconf systemtap_session::NSPR_Initialized

The 1-byte saving of an autoconfed bool isn't worth the headache of
mismatched definitions, even if we've resolved that.

Remove dead code from systemtap_session

These two functions are dupes of code in main.cxx.

* session.cxx (parse_kernel_config): Removed.
  (parse_kernel_exports): Removed.

Make sure we all use the same systemtap_session

In particular, if some sources don't include config.h, then they'll be
missing HAVE_NSS, which changes the layout of systemtap_session.

* session.h: #include "config.h", so everyone gets HAVE_NSS.

Add user_{int8,int16,uint16,int32,uint32,int64}.

* conversions.stp (STP_GET_USER): New.  Use it to define...
(user_{int{8,16,32,64},uint{16,32}}) New. Names derive from types.h.
(user_char,user_short,user_int): Use STP_GET_USER.

* tapsets.cxx (uprobe_var_expanding_visitor::visit_target_symbol):
Use user_intN tapsets.

* sdt_types.c (unsigned_short_int_var,unsigned_int_var): New

* sdt_types.stp (unsigned_short_int_var,unsigned_int_var): New

CVE-2010-4170, CVE-2010-4171: staprun module loading/unloading security fixes

We would like to thank Tavis Ormandy for reporting this issue.

* runtime/staprun/staprun.c (enable_uprobes): Don't run /sbin/modprobe
  directly, since it takes more inputs than we have tried to sanitize.
  (remove_module): Call init_ctl_channel on putative stap module name,
  to check that it's our own stap module.
  (init_staprun): Do remove/retry via remove_module rather than
  underchecked delete_module(2).
* runtime/staprun/ctl.c (init_ctl_channel): Check ownership of
  .ctl files, to preclude manipulation of some other stapusr member's modules.
* runtime/staprun/Makefile.am, systemtap.spec: Install staprun as
  mode 04110, group stapusr.
* README.security, runtime/staprun/staprun.8: Note new stapdev/stapusr
  joint requirements.

Use output_exportconf for finding tsc_khz

* buildrun.cxx (compile_pass): Switch tsc_khz to output_exportconf.
* runtime/autoconf-tsc-khz.c: Delete this version-based check.

Add a fallback for kernels not exporting add_timer_on

Commit 3fd1c49 regressed for kernels which don't EXPORT add_timer_on.

* buildrun.cxx (compile_pass): Test for add_timer_on's export.
* runtime/time.c (_stp_init_time, __stp_init_time): Restore the
  IPI-add_timer as a fallback.

PR11735: Hash ldd/vdso unwindsym_modules too

* translate.cxx (prepare_symbol_data): Do add_unwindsym_ldd/vdso here.
  (prepare_translate_pass): Call above, and placeholder for the future.
* main.cxx (passes_0_4): Call prepare_translate_pass before checking the
  cache, so we have more complete unwindsym_modules.

PR11735: Hash unwindsym_modules as full paths

* hash.cxx (find_script_hash): Use hash::add_path instead of plain add.

uprobes: Clean up after vfork-exec

When a vfork'ed thread execs, we don't want to remove the probes from
the vfork parent, but we still need to clean up the thread associations.
Otherwise, the newly-execed process won't have the right accounting for
its SSOL area.

* runtime/uprobes2/uprobes.c (uprobe_report_exec): Clean up thread info
  when a vfork'ed task execs.
* runtime/uprobes/uprobes.c (uprobe_report_exec): Ditto.

tapset: add buildok test for inode_name()

tapset: new inode_name tapset function

Remove unfinished sentence from stap.1

The problem was introduced in commit 98aab489.

PR12164: Emit kprobes-sdt goo only when needed

This stuff is only used for apps compiled with EXPERIMENTAL_KPROBE_SDT,
so we don't need it most of the time.

* runtime/kprobes-common.c (stap_kprobe_process_found,
  stap_kprobe_mmap_found): Only create #ifdef KPROBES_TASK_FINDER.
* tapsets.cxx (dwarf_derived_probe::join_group): Only enable_task_finder
  in the session if a semaphore is present.
  (dwarf_derived_probe_group::enroll): Remember if any has_semaphores.
  (dwarf_derived_probe_group::emit_module_decls): Predicate sdt and
  task_finder code on has_semaphores.
  (dwarf_derived_probe_group::emit_module_init): Ditto.
  (dwarf_derived_probe_group::emit_module_exit): Ditto.

runtime: tolerate overlong section/symbol names better

* runtime/staprun/staprun.c (send_a_relocation): Don't
  complain about overlong names, except if very verbose.
  Triple-check null termination of surviving strings.

releng: tweak build ordering to get code first, docs last

* Makefile.am (SUBDIRS): Build . and runtime/staprun first.
* Makefile.in: Regenerated.

regen misc. Makefile.in's due to earlier addition of support_section_question

runtime: bump up module/symbol name length limits

Due to a report that kernel modules written with C++ inside sometimes
grow extra-long sections:

* runtime/transport/transport_msgs.h: Double
  STP_MODULE_NAME_LEN and STP_SYMBOL_NAME_LEN to 128.

Keep running buildid.exp if a test fails.

* buildid.exp:  Only quit if a compile or link fails.

Make the arg of dwflpp::function_scope_matches a reference

Test user mode build id checking.

* buildid.exp: New.

PR11811: news blurbage

PR11811: warn on missing unwind info with 'stap -d FOO'

* unwind.c (unwind_frame): Demote common warnings to dbug_unwind messages.
  (unwind): Print a warning for unwind operations foiled by modules that
  did not have pre-uploaded unwind data.
* buildrun.cxx (compile_pass): Check for exported __module_text_address.

fix accidental prefix '/' in abbreviated module name in backtraces

* runtime/sym.c (_stp_snprint_addr): Use strrchr correctly.

PR12168: eliminate duplicate run-time warning messages

* runtime/staprun/mainloop.c (stp_main_loop STP_OOB_DATA):
  Rewrite WARNING handling.  Include optional duplicate-elimination
  with size-limited glibc tsearch() tree.
* testsuite/systemtap.base/warnings2.exp: New test.
* NEWS: Mention this.

PR12168: extend staprun with -w (suppress warnings) option.

* buildrun.cxx: Pass -w to staprun.
* runtime/staprun/common.c: Parse new -w.  Document old -u.
* runtime/staprun/staprun.8: Ditto.

PR11811 prep: save full mmpath for found vmas

Extend the print_*backtrace symbol names with the full path
of the user-space ELF module, not just the basename.  This
requires saving a char[] copy in the __stp_tf_vma_entry
structure instead of a char* pointing into a dentry.d_name.
-DTASK_FINDER_VMA_ENTRY_PATHLEN governs copy length.

* runtime/task_finder_vma.c (__stp_tf_vma_entry): Change
  char*name -> char[]path field.  Update manipulating
  functions accordingly.
* runtime/vma.c (_stp_vma_mmap_cb): Pass mmpath to tf-vma table
  if available.
* testsuite/systemtap.context/usymbols.exp: Update for expected
  full path in output string.
* NEWS: Mention change.
* runtime/sym.h (_STP_SYM_MODULE_BASENAME): New flag.
  (_STP_SYM_SIMPLE): Include it.
* runtime/sym.c (_stp_snprint_addr): Process it.

fix -DDEBUG_UNWIND compile warning via printf %ld & casting

tweak tz_ctime() parametrization to satisfy kerneldoc

Add build id support for user modules.

* sym.c (_stp_build_id_check): New.  Moved the build id check code from
 _stp_module_check and changed...
(_stp_module_check): to use it.
(_stp_usermodule_check): New.  This is called from the
task manager callback stap_uprobe_change_plus for the process found and
mmap found cases.

* translate.cxx (dump_unwindsyms): Don't stop if it isn't a kernel module.
Set build_id_vaddr for user modules.  Output module->build_id_offset as an
absolute address.

* uprobes-common.c (stap_uprobe_change_plus): Call _stp_usermodule_check.

PR12195: pass user-space timezone at script startup

* runtime/transport/transpot_msgs.h: Declare STP_TZINFO message type.
* runtime/staprun/staprun.c (send_tzinfo): New function to send time zone
  at startup.
* runtime/transport/control.c (_stp_ctl_write_cmd): Allow this message.
* runtime/transport/transport.c (_stp_handle_tzinfo): New function to
  process this message and declare various globals.
* tapset/tzinfo.stp: New tapset to access those globals.
* testsuite/buildok/timestamp-embedded.stp: Extend test.
* NEWS: Mention tz_ctime().
* doc/SystemTap_Tapset_Reference/tapsets.tmpl: Extract docs from new tapset.

Make pfiles kneel before the frame-size overlord

In socket_optname, there's an optname_entries mapping array.  On 64-bit,
each item is 24 bytes, so with 14 entries that's 336 bytes.  That
doesn't need to be on the stack -- static const it is.

Add make targets "uprobes" and "install-uprobes"

This builds uprobes.ko in $srcdir and $DESTDIR, respectively,
because I'm too lazy to type it out all the time.

PR10821: Remove large stack variables from the unwinder

We tend to use the percpu CONTEXT as a pre-allocated stack, so add the
large unwinder variable there too.

* runtime/unwind.c (unwind, unwind_frame): Use an unwind_context
  parameter for the larger data needed.
* runtime/stack-$arch.c (__stp_stack_print): Add the unwind_context
  parameter, and use it for the x86 versions with a dwarf unwinder.
* runtime/stack.c (_stp_stack_print, _stp_stack_sprint): Add the
  unwind_context parameter and pass it along.
* translate.cxx (c_unparser::emit_common_header): Add unwind_context the
  the global CONTEXT when we NEED_UNWIND_DATA.
* tapset/context-unwind.stp (*backtrace): Use CONTEXT->uwcontext.
* tapset/ucontext-unwind.stp (*backtrace): Use CONTEXT->uwcontext.
* buildrun.cxx (compile_pass): Tighten the frame warning to 256 bytes.

Updated memory.stp, rpc.stp, task.stp, and vfs.stp for 2.6.36.

* tapset/memory.stp (vm.oom_kill): Added oom_kill_process().
* tapset/rpc.stp (sunrpc.clnt.call_sync): Updated for 2.6.36.
  (sunrpc.clnt.call_async): Ditto.
* tapset/task.stp (task_gid): Updated for 2.6.36 by defining and using
  task_gid().
  (task_egid): Updated for 2.6.36 by defining and using task_egid().
* tapset/vfs.stp (_vfs.block_prepare_write): Update for 2.6.36 by using
  __block_write_begin() as an alternative to __block_prepare_write().
  (_vfs.block_write_begin): Updated for 2.6.36.

Test that uprobes supports "rep ret"

We've made a special case in uprobes to support this sequence, since it
does occur in normal gcc output, so make sure of that.  At the same
time, test that we reject other prefixes like "repnz ret", even though
we might be able to support them after more analysis.

uprobes_i386: backport check_legacy_prefix from uprobes2

Without this, we can't give "special" prefixed instructions the
treatment they deserve. (e.g. rep-ret)

building: add placeholder for "make -j<N>" runs to build substuff

uprobes: Fix post_ssout handling of "rep ret" on x86

That odd sequence is apparently used to coerce better behavior from the
branch predictor on AMD K8.  GCC does this, so we need to be prepared to
deal with it.

In uprobe_post_ssout, most instructions just need a relative %ip fixup
after single-stepping out-of-line.  A few are special though, either
because their new %ip is not relative, or because their return address
on the stack needs that relative SSOL adjustment, or even both.  This
oddball "rep ret" is such a case, but we were missing the proper fixup
due to the unexpected prefix.

This patch moves that ip-fixup decoding to the validation phase in
setup_uprobe_post_ssout, saving the flags UPFIX_RETURN and UPFIX_ABS_IP
as needed.  In this same step, we reject probing these instructions if
they have a prefix, as we haven't determined the correct behavior for
them.  For "rep ret" we make an explicit exception, since we know that
behaves just like a bare "ret".

Edits to the User-space chapter.

Restore legacy domain (localhost) in server cert for stap-client.

Improve gettimeofday stability

These are a few tweaks to improve the stability of our gettimeofday
startup and shutdown.  Hopefully this helps with PR10651 and PR12182,
but that's not been confirmed yet.

* runtime/runtime.h: Don't include time.c here.
* translate.cxx (c_unparser::emit_common_header): Include time.c here,
  only if NEEDed (so we also ensure init/kill_time will be called).
  (c_unparser::emit_module_init): Change to STAP_SESSION_STOPPED for the
  final cleanup efforts, especially for time.c to wrap up.
  (c_unparser::emit_module_exit): Ditto.
* runtime/time.c (__stp_time_timer_callback): Use the session_state
  instead of stp_timer_reregister, and continue until STOPPED.
  (__stp_init_time): Don't add_timer from the IPI.
  (_stp_init_time): Instead, add_timer_on each cpu here.

Add comment to latencytap.stp on where the translations came from.

Add information about uprobe module in beginners guide

fix memory double-free in module_cache

The kmodule.stp test case could evoke a double-free of module-info
structs, since these were shared pointers, and not managed by GC.
Oh yeah, what GC, in the year 2010.

* dwflpp.cxx (dwflpp dtor): Don't delete shared mod_info ptr.
  (module_cache dtor): New function.
* dwflpp.h: Declare.
* tapsets.cxx (delete_session_module_cache): New function.
  (several::build_no_more): Call it.
  (dwarf_build_no_more): Use delete_map<> template.

Improved fork/vfork tests for RHEL5.

* testsuite/systemtap.clone/dtrace_fork_exec.exp: Improve output string to
  handle parent/child execution order differences.
* testsuite/systemtap.clone/dtrace_vfork_parent.c: Added include files to
  avoid compile error on RHEL5.
* testsuite/systemtap.clone/dtrace_vfork_parent.c: Ditto.

Improve handling of vfork'ed processes for uprobes version 1.

* runtime/uprobes/uprobes.c (insert_bkpt): Don't log EEXIST errors (since
  systemtap inserts duplicate probes).
  (uprobe_report_exec): Only cleanup if this is the last thread.  Without
  this change, vfork'ed processes doing an exec weren't handled correctly.

PR11376: reinforce doc blurbage for process(PID).statement(ADDR).absolute

PR11376: a quickie test case for process(PID).statement(NUM).absolute

PR11376: fix generated probe definition struct for process(PID).statement(ADDR).absolute*

Add Chapter about user-space probing to the Beginner's guide.

PR12176 String literal at end of file crashes parser.

* parse.cxx (parser::parse_literal): Only inspect next token if there is one.
* testsuite/parseok/end_string.stp: New testcase.

examples: Print PID of peer for unix sockets

This information is very useful for debugging but is not normally
available to userland since /proc/net/unix does not list it.

PR12169: fix tokenize()

* tapset/tokenize.stp: New file for tokenize().  Use per-context statics.
* doc/SystemTap_Tapset_Reference/tapsets.tmpl: Pull it into new docs.
* tapset/string.stp: Remove old tokenize() code.
* translate.cxx (emit_common_header): Add STAP_NEED_CONTEXT_TOKENIZE bits.
  (translate_pass): Emit global %{ %} embeds before context definition.

PR12169: avoid unprivileged use of tokenize() until fixed

Revert "git-version: use git status --porcelain to detect tree dirtiness"

This reverts commit 3c29b815f282a0629b85af1758b256bf790bf371.

Meh, old git 1.5* doesn't support this.