repaired partial change from 'class' to 'pkg'.  added use strict/warnings

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1764 9efc2488-bf62-4759-914b-345cdb29e865

1. Add comments to be recognized by doxygen or other such
documentation generators.

2. Change the values returned by buffer_fadd(), buffer_add(),
buffer_add_n(), and buffer_add_char() to make them more
consistent.  In practice we never pay any attention to the
return values anyway.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1763 9efc2488-bf62-4759-914b-345cdb29e865

changed 'my' globals to 'our' globals for testing purposes

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1762 9efc2488-bf62-4759-914b-345cdb29e865

first tests

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1761 9efc2488-bf62-4759-914b-345cdb29e865

more cleanup and documentation in progress

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1760 9efc2488-bf62-4759-914b-345cdb29e865

yanking LogServer test file

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1759 9efc2488-bf62-4759-914b-345cdb29e865

removing use of LogServer

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1758 9efc2488-bf62-4759-914b-345cdb29e865

removing _json_hint_to_class

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1757 9efc2488-bf62-4759-914b-345cdb29e865

shuffling around, adding POD stubs

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1756 9efc2488-bf62-4759-914b-345cdb29e865

removing OSRF::Utils::LogServer at berick's suggestion

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1755 9efc2488-bf62-4759-914b-345cdb29e865

rearranging things a bit, docs. no actual code changes.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1754 9efc2488-bf62-4759-914b-345cdb29e865

new test files for Utils breakout

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1753 9efc2488-bf62-4759-914b-345cdb29e865

removed $VERSION construct which caused undef warnings when not under svn, and only occurred in 2 modules

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1752 9efc2488-bf62-4759-914b-345cdb29e865

method prettyJSON doesnt exist anymore

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1751 9efc2488-bf62-4759-914b-345cdb29e865

ditto

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1750 9efc2488-bf62-4759-914b-345cdb29e865

that last version change was rather poorly timed, as it turns out

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1749 9efc2488-bf62-4759-914b-345cdb29e865

local doc updates for recent changes. wondering how much point there is in keeping these around, long-term

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1748 9efc2488-bf62-4759-914b-345cdb29e865

preëmptive version bump

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1747 9efc2488-bf62-4759-914b-345cdb29e865

'make check' works as promised. perl tests moved to 'check' target

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1746 9efc2488-bf62-4759-914b-345cdb29e865

updating MANIFEST for Module::Build-ness

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1744 9efc2488-bf62-4759-914b-345cdb29e865

added separate install fail diagnostic message

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1743 9efc2488-bf62-4759-914b-345cdb29e865

removing files no longer needed

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1742 9efc2488-bf62-4759-914b-345cdb29e865

'make' will now just plain make again without proceeding to install.
Added Build to files swept by 'make clean'

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1741 9efc2488-bf62-4759-914b-345cdb29e865

Going back to dumping a script (unbreaking non-root make vs. make install)

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1740 9efc2488-bf62-4759-914b-345cdb29e865

Added DISTCLEANFILES to make "make distclean" work better

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1739 9efc2488-bf62-4759-914b-345cdb29e865

AC_CONFIG_MACRO_DIR added at the behest of libtoolize

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1738 9efc2488-bf62-4759-914b-345cdb29e865

Added distclean-local: target and DISTCLEANFILES to make 'make distclean' work superawesomely
Added ACLOCAL_AMFLAGS at the behest of libtoolize

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1737 9efc2488-bf62-4759-914b-345cdb29e865

Build.PL built; build/test/install all selfcontained and automatic now

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1736 9efc2488-bf62-4759-914b-345cdb29e865

M::B changes

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1735 9efc2488-bf62-4759-914b-345cdb29e865

M::B changes

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1734 9efc2488-bf62-4759-914b-345cdb29e865

Extended the JSON_INIT_CLEAR macro to avoid segfaults.

Scenario: converting a JSON_BOOL, with a value of true, to a JSON_HASH or
JSON_ARRAY.  The true value (in a union with an osrfHash* and an osrfList*)
was being interpreted as a non_NULL pointer and deferenced.  Oops.

With this change, we clear the boolean value (by nullifying one of the
unioned pointers) whenever changing from a JSON_BOOL to anything else.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1733 9efc2488-bf62-4759-914b-345cdb29e865

Performance tweak to jsonIterator.

Instead of storing a malloc'd copy of the key of a JSON_HASH entry,
just store a const pointer to the key string stored in the
internal osrfHash.  That way we don't have to do a malloc and
free every time we bump the iterator.

This change also requires the addition of a couple of const qualifiers
in the client code.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1732 9efc2488-bf62-4759-914b-345cdb29e865

set the 'from' address in outbound jabber messages

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1731 9efc2488-bf62-4759-914b-345cdb29e865

set the 'from' address in outbound xmpp messages

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1730 9efc2488-bf62-4759-914b-345cdb29e865

reduce unnecessary calls to fcntl() while reading from the socket

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1729 9efc2488-bf62-4759-914b-345cdb29e865

check NULL-ness on hash the key before calling find_item

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1727 9efc2488-bf62-4759-914b-345cdb29e865

This is a performance tweak to the osrfHashGet function, a widely
used utility function.

The old version accepted a variable number of arguments: a pointer
to an osrfHash, a string optionally containing printf-style
format specifiers, and addtional parameters as needed to fill in
the blanks.

In practice, none of the code ever uses the printf-style formatting.
We always pass exactly two parameters.  We burn CPU cycles scanning
the string for format specifiers and never find any.

I eliminated the unused variable parameters and turned osrfHashGet()
into a simple two-parameter function.  Just in case anybody ever
wants it, I also cloned the original version into a new function
named osrfHashGetFmt, which accepts a variable number of arguments
as before.

Note that, since the signature of the function is changing,
it is necessary to recompile any open-ils programs that
call it, namely:

oils_dataloader.c
oils_event.c
oils_idl-core.c
oils_cstore.c
dump_idl.c

The Makefiles apparently don't recognize this dependency.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1726 9efc2488-bf62-4759-914b-345cdb29e865

Add two new commands, "open" and "close", to open and close ongoing
connections to specified services.  (Otherwise a connection is created
and destroyed for every request.)

Main purpose: to be able to use srfsh for database operations that require
transactions.

Warning: typically the <keepalive> value will need to be increased for
the service in question in opensrf.xml, or else the server will close
the connection before you can type your request.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1725 9efc2488-bf62-4759-914b-345cdb29e865

Fix a bug whereby, if there was only one <service> entry for the
public router in opensrf_core.xml, the service would fail to register.

Root cause: The code was expecting to see a list of services in a
JSON_ARRAY.  But if there's only one service, it's represented as a
JSON_STRING.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1724 9efc2488-bf62-4759-914b-345cdb29e865

until we make the router / C processes write their own PID file, put a short sleep in after startup to give each time to show up in the 'ps' output.  this should help prevent lingering router processes after service stop

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1720 9efc2488-bf62-4759-914b-345cdb29e865

call cleanup instead of running the death callback directory when server session expires

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1719 9efc2488-bf62-4759-914b-345cdb29e865

added session callback support with example method.  added method-not-found handling.  added network socket flush for clearing stale inbound data.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1718 9efc2488-bf62-4759-914b-345cdb29e865

added a session_data disctionary to ServerSession class for storing per-session data.  fixed bug where no params caused serer-side confusion.  added exmample of session_data usage to example app

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1717 9efc2488-bf62-4759-914b-345cdb29e865

1. If an input line ends in a backslash, remove the backslash
and append the following line as a continuation.

2. Disable the readline library's special treatment of
horizontal tabs, which by default trigger file name
completion (which is not useful for srfsh).

3. In calls to strtok(): accept horizontal tabs as
delimiters, since they are no longer suppressed
by readline().

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1716 9efc2488-bf62-4759-914b-345cdb29e865

teach interval_to_seconds about HH:MM:SS format time intervals

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1715 9efc2488-bf62-4759-914b-345cdb29e865

teach interval_to_seconds about negative intervals

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1714 9efc2488-bf62-4759-914b-345cdb29e865

flush_socket now flushes data through the xml stream parser to prevent the destorying the validity of the stream.  now discarding RESULT messages passed to server sessions.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1712 9efc2488-bf62-4759-914b-345cdb29e865

don't try to 'kill' an empty pid / prevents warnings

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1711 9efc2488-bf62-4759-914b-345cdb29e865

define OSRF_HOSTNAME globally to prevent 'unbound variable' error

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1710 9efc2488-bf62-4759-914b-345cdb29e865

Use autotools to set the default values in opensrf-perl.pl to what was passed in to ./configure

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1709 9efc2488-bf62-4759-914b-345cdb29e865

Use autotools a bit more to our advantage:
  * In osrf_ctl.sh, use the configured location for osrf_config by default
  * Differentiate between prefix and exec_prefix to respect configure options
  * Make configure output slightly more consistent

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1708 9efc2488-bf62-4759-914b-345cdb29e865

try finding osrf_config in the users PATH first to prevent the use of 'find' when possible (find + NFS = watching paint dry)

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1706 9efc2488-bf62-4759-914b-345cdb29e865

opensrf.py can now read the opensrf.settings file for managing apps.  added start/stop_all actions.  support for service name syslog ident

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1705 9efc2488-bf62-4759-914b-345cdb29e865

until full settings support is added, comment that out for now.  added example keepalive

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1704 9efc2488-bf62-4759-914b-345cdb29e865

added final bits for stateful sessions (drone keepalive)

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1703 9efc2488-bf62-4759-914b-345cdb29e865

no need to wrap each method description in an object container with class 'method'

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1702 9efc2488-bf62-4759-914b-345cdb29e865

activate the config options, need to that if you're going to use a cache server that's not == localhost:11211

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1700 9efc2488-bf62-4759-914b-345cdb29e865

go ahead and log the cache keys when we log the cache data

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1699 9efc2488-bf62-4759-914b-345cdb29e865

don't set stream state to disconnected on a non-stream error.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1697 9efc2488-bf62-4759-914b-345cdb29e865

pass unknown string as second param to osrfLog to avoid literal % parsing in vsnprintf

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1695 9efc2488-bf62-4759-914b-345cdb29e865

Performance tweak for osrfLog* functions: don't pass the message
through VA_LIST_TO_STRING unless you're actually going to use it.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1694 9efc2488-bf62-4759-914b-345cdb29e865

OpenSRF r1692 + OpenILS r12809 broke the Evergreen build; this fixes it

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1693 9efc2488-bf62-4759-914b-345cdb29e865

New JSON parser -- simpler, faster, and more robust in the
face of syntax errors

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1692 9efc2488-bf62-4759-914b-345cdb29e865

capture osrfServerError responses

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1691 9efc2488-bf62-4759-914b-345cdb29e865

In osrfConfigGetValue(): plugged a memory leak
reported by Steven Chan.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1690 9efc2488-bf62-4759-914b-345cdb29e865

1. Create a new osrfListExtract function, which removes an item
from an osrfList without destroying it, and returns a pointer to
the item thus removed.

2. Create a new jsonObjectExtractIndex, which removes a
specified entry in a JSON_ARRAY, and returns a pointer
to it, without destroying it.

3. In osrf_json.h: Corrected an inaccurate comment about
jsonObjectRemoveIndex().  Contrary to the original comment, this
function does not shift other objects down to fill the gap.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1689 9efc2488-bf62-4759-914b-345cdb29e865

In osrf_system_shutdown(): provide a return value for an early
return (since the function is of type int).

It's not clear whether these are the right semantics, since we
never capture the return value anyway.  But if we're going
to return anything at all, it makes sense to return something
different in the case of an early return.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1688 9efc2488-bf62-4759-914b-345cdb29e865

Plug a memory leak reported by Steven Chan.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1687 9efc2488-bf62-4759-914b-345cdb29e865

hash keys have to be escaped like regular strings

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1682 9efc2488-bf62-4759-914b-345cdb29e865

Patch from John Craig fixing a problem with the pre-reqs makefile

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1681 9efc2488-bf62-4759-914b-345cdb29e865

make OpenSRF 'global' so IE won't clobber it

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1675 9efc2488-bf62-4759-914b-345cdb29e865

try the various xhr loading techniques

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1673 9efc2488-bf62-4759-914b-345cdb29e865

IE mad.  IE throw tantrum.  Bill give IE a Baby Ruth

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1672 9efc2488-bf62-4759-914b-345cdb29e865

in IE, localeMatch might be null

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1671 9efc2488-bf62-4759-914b-345cdb29e865

don't try to close the socket if it's not there

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1669 9efc2488-bf62-4759-914b-345cdb29e865

Non-Mozilla browsers want an explict dojo.declare() for a given namespace.
Partially addresses http://svn.open-ils.org/trac/ILS/ticket/40

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1668 9efc2488-bf62-4759-914b-345cdb29e865

gotta protect the JSONObject2Perl() when decoding arrays as well

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1666 9efc2488-bf62-4759-914b-345cdb29e865

not sure why this is an issue now (different perl/json:xs verions?), but
$obj->{$k} = JSONObject2Perl($bool)
results in 'Modification of a read-only value attempted', while
$obj->{$k} = $bool
does not, even though the JSON::XS::Boolean object is not altered by JSONObject2Perl().
just missing something?
patched

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1664 9efc2488-bf62-4759-914b-345cdb29e865

Remove old install target for CentOS - thanks Carl T. Miller!

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1662 9efc2488-bf62-4759-914b-345cdb29e865

Bug in CentOS prerequisite installer - thanks Carl T. Miller!

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1660 9efc2488-bf62-4759-914b-345cdb29e865

1. Added a missing #include

2. In jsonNewIterator(): initialize hashItr to NULL
if the object is not a JSON_HASH

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1659 9efc2488-bf62-4759-914b-345cdb29e865

let's avoid creating double-hyphened locales, too

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1654 9efc2488-bf62-4759-914b-345cdb29e865

dojo.locale is the preferred method of determining Dojo's current locale; avoids having to explicitly set djConfig.locale

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1653 9efc2488-bf62-4759-914b-345cdb29e865

So, djfiander says "fix the code so it's not case-sensitive", so this seems to fix it.
OpenSRF locale gets xx-YY, even though Dojo locale is xx-yy.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1651 9efc2488-bf62-4759-914b-345cdb29e865

eplace the OSRF_METHOD_VERIFY_CONTEXT macro with an
2 equivalent call to the osrfMethodVerifyContext function.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1650 9efc2488-bf62-4759-914b-345cdb29e865

Replace the OSRF_METHOD_VERIFY_CONTEXT macro with an
2 equivalent call to the osrfMethodVerifyContext function.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1649 9efc2488-bf62-4759-914b-345cdb29e865

Replace the OSRF_METHOD_VERIFY_CONTEXT macro with an
equivalent call to the osrfMethodVerifyContext function.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1648 9efc2488-bf62-4759-914b-345cdb29e865

This update mainly tightens the error handling.

1. If there aren't enough arguments on the command line, return EXIT_FAILURE
instead of zero.

2. Defer the call to set_proc_title() until we are done using the command
line arguments, so that we don't have to make copies of them.

3. Check the return value from osrfConfigInit().  Otherwise a NULL (caused,
e.g., by a missing config file) leads to a segfault.

4. If the config file doesn't define any routers to spawn, exit immediately
with an error message before entering the fork loop.

5. If a child process returns (due to an error) instead of entering the
normal endless loop, break out of the fork loop.  Otherwise the child
remains in the fork loop and spawns children of its own (unless it's the
last child to be spawned).  At best, that's just silly.

6. Append an newline to a message issued from setupRouter().  (It's not
clear why this message goes directly to stderr instead of to the usual
logging machinery, which at this point is directed to stderr anyway.)

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1647 9efc2488-bf62-4759-914b-345cdb29e865

added what amounts to -l (use localhost as fqdn) support for correctly reporting status to the terminal

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1645 9efc2488-bf62-4759-914b-345cdb29e865

Debian and Ubuntu both want libgrcypt11-dev explicitly specified

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1643 9efc2488-bf62-4759-914b-345cdb29e865

make config parsing more tolerant

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1639 9efc2488-bf62-4759-914b-345cdb29e865

forgot a noun

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1638 9efc2488-bf62-4759-914b-345cdb29e865

srfsh.xml tweaks

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1637 9efc2488-bf62-4759-914b-345cdb29e865

quote-o

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1636 9efc2488-bf62-4759-914b-345cdb29e865

repaired config data structure ref check for registering routers

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1633 9efc2488-bf62-4759-914b-345cdb29e865

removing perl special-casing from src/Makefile.am

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1631 9efc2488-bf62-4759-914b-345cdb29e865

adding src/perl to AC_CONFIG_FILES

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1630 9efc2488-bf62-4759-914b-345cdb29e865

initial checkin for perl Makefile.am

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1629 9efc2488-bf62-4759-914b-345cdb29e865

Add a new function osrfStringArrayTokenize.  It parses an input
string into tokens separated by a specified delimiter character,
much like strtok() or strtok_r(), and loads them into an
osrfStringArray.

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1627 9efc2488-bf62-4759-914b-345cdb29e865

Add a function osrfMethodVerifyContext() to do what the
existing OSRF_METHOD_VERIFY_CONTEXT macro does.  Use it in
_osrfAppRunSystemMethod() and osrfAppEcho().

git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@1626 9efc2488-bf62-4759-914b-345cdb29e865