dosc: add long options to skill.1

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: add long options to kill.1

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

skill: fix compiler warnings

Pretty much all of the warnings where caused by some of the previous
patches, and they became visible with gcc when compiling with;

CFLAGS="-O -g -ggdb -Wall -Wextra -pedantic -Wstrict-overflow
-Wformat -Wswitch-default -Wswitch-enum -Wsync-nand
-Wredundant-decls -Wuninitialized"

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

skill: add long options to usage output

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

skill: support usage output to stdout & stderr

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

skill: use symbolic exit values

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

skill: use strtosig() for -l option argument

Re-enable signal number to name string, and other way around,
conversion after it broke at the time when long options where
introduced.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

libproc-ng: add strtosig() function

The function will convert a signal number string to a signal name, or
vice a verse.  Return value is string, which is an function user is
expected to free after use.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

skill: add long options

This commit broke `-l <sig>' conversion, which needs to be fixed
before kill can be claimed to work similar enough way as it did
earlier.  The fix require libprocps change, because the current
print_given_signals() simply does not work in this case.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

skill: warning printout for non-existing processes

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

skill: add long options & new debugging option

Done for skillsnice_parse(), meaning the kill needs functionality
will be done later.

This commit also introduces new debugging option.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

skill: parse kill signals separately

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

skill: use strtol_or_err to nice argument parsing

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

lib: add strtol into utility library

The utility library is for functions which are shared in commands,
but that does not belong to libproc-ng.  The first function is a
wrapper for strtol that performs error checking, and exists if such
happen.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

skill: remove run time configuration global variables

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

skill: move usage functions near each other

Done only for convenience.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

skill: clean up which functionality is being called

The skill.c is three different commands depending on how the
executable is named. This patch makes evaluation of the command name
cleaner.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

skill: reindent the file

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

build-sys: use Makevars --add-comments= to limit .pot comments

By not limiting --add-comments= via an argument, xgettext is far
too aggressive (one might say stupid) when extracting comments.
It doesn't even limit extraction to a single preceeding comment.

Here is an example showing a program source excerpt and the
resulting .pot excerpt:

 --- program source ----------------------------------------
 close (0); dup (in_fd[0]);  /* set the stdin to the in pipe */
 close (1); dup (out_fd[1]); /* set the stdout to the out pipe */
 close (2); dup (out_fd[1]); /* set the stderr to the out pipe */
 execvp (args[0], args);     /* exec gdb */
 perror (_("exec failed"));

 --- resulting .pot ----------------------------------------
 #. set the stdin to the in pipe
 #. set the stdout to the out pipe
 #. set the stderr to the out pipe
 #. exec gdb
 #: ps/stacktrace.c:28 ps/stacktrace.c:63
 msgid "exec failed"
 msgstr ""

build-sys: we don't need wide-ncurses, what were we thinking?

. 1 program uses curses (top)
. 2 programs use ncurses (slabtop, watch)
. all 3 include non-wide <curses.h> or <ncurses.h>
. those 3 do not currently need wide support
. but anticipating nls, we link against libncursesw

This patch ensures an environment consistent with current
and future ncurses needs.

build-sys: added config.h include to nls.h (for PACKAGE str)

build-sys: added tabs to AC_CONFIG_FILES for consistency

build-sys: alphabeticized strverscmp in AC_CHECK_FUNCS

build-sys: added some missing files to AC_CHECK_HEADERS

slabtop: added remaining missing nls support

free: added remaining missing nls support

misc: trailing space/tab cleanup following rebase

build-sys: corrected/simplified ncurses support detection

pmap: correct 'rh analysis #2' patch and contrib document

build-sys: verify gettext is present in autogen.sh

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

build-sys: add missing files to distribution

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

build-sys: remove non-existing files from Makefile.am

The f23390043bece9f2d4870e5b3a187896e2c7d23f removed few files,
which still exist in Makefile.am making `make dist' to fail. This
patch fixes the isue.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: add rh analysis #2 information to contrib

Reference information.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

build-sys: enable xgettext TRANSLATORS comments

This commit will enable pass comments to translators near gettext
directive to .pot file. For example;

/* TRANSLATORS: this is visible in .pot file */
printf(_("to be translated"));

Signed-off-by: Jim Warner <james.warner@comcast.net>

sysctl: warn when separators are repeated

For some reason sysctl earlier allowed quite strange separators.

% sysctl kernel./.pty.nr
kernel./.pty.nr = 6
% sysctl kernel///pty//////////nr
kernel...pty..........nr = 6

This commit does not disallow that sort of constructs, but will warn
about them. In future disallowing these might be reasonable thing to
do.

% sysctl kernel./.pty.nr
sysctl: separators should not be repeated: ./.pty.nr
kernel./.pty.nr = 6

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

sysctl: use libc error printing facilities

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

vmstat: use libc error printing facilities

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

pgrep: use libc error printing facilities

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

libproc-ng & ps: fix warning reported by smatch

proc/ksym.c +279 read_file(54) info: redundant null check on buf calling free()
proc/ksym.c +330 parse_ksyms(41) info: redundant null check on ksyms_data calling free()
proc/ksym.c +332 parse_ksyms(43) info: redundant null check on ksyms_index calling free()
proc/ksym.c +451 sysmap_mmap(110) info: redundant null check on sysmap_index calling free()
proc/procps.h:74:5: warning: undefined preprocessor identifier 'SHARED'
proc/slab.c +145 parse_slabinfo20(24) error: potential null derefence 'prev'.
proc/slab.c +222 parse_slabinfo11(21) error: potential null derefence 'prev'.
ps/stacktrace.c +131 debug(6) error: snprintf() is printing too much 99 vs 16

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

misc: compiler warning fixes

Fix few compiler warnings. Some of these warnings appeared multiple
times, and the listing bellow is more about which sort of errors
where fixed.

devname.c:87:12: warning: comparison of integers of different signs: 'int' and 'unsigned long'
output.c:389:36: warning: passing 'char **const' to parameter of type 'const char *const restrict *' discards qualifiers in nested pointer types
output.c:611:31: warning: comparison of integers of different signs: 'const unsigned long' and 'int'
stacktrace.c:33:37: warning: unused parameter 'signum'

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

includes: add xalloc.h to unify memory allocations

The xalloc.h provides necessary error checking.

Signed-off-by: Jim Warner <james.warner@comcast.net>
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ps: add gettext support

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

watch: add gettext support

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

w: add gettext support

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

vmstat: add gettext support

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

uptime: add gettext support

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

tload: add gettext support

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

sysctl: add gettext support

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

slabtop: add gettext support

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

skill: add gettext support

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

pwdx: add gettext support

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

pmap: add gettext support

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

pgrep: add gettext support

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

free: add gettext support

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

build-sys: add c.h with fundamental C definitions

This file contains various preprocessor directed definitions, and
other definitions, which most programs need.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

build-sys: add NLS support

Add GNU gettext internationalization and localization support.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

sysctl: implement --pattern option

Useful for e.g network hook scripts together with --system to only apply
sysctls for a specific network interface.

Signed-off-by: Ludwig Nussel <ludwig.nussel@suse.de>
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

sysctl: add --system switch

instead of requiring distributions to construct a loop around sysctl
in boot scripts just scan a set of default directories if the --system
switch is used.

Config files are applied in alphabetic order of their base name.
Each base name is only applied once according to the directory
preference. /etc/sysctl.conf is always applied last.

Signed-off-by: Ludwig Nussel <ludwig.nussel@suse.de>
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ps: new usage function

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: clean up kill and skill man pages

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

skill: new usage function

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: mention long options in man vmstat.8

This commit includes generic manual clean ups as well.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

vmstat: add long options & new usage function

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: fix free.1 groff syntax

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: mention long options in man pgrep.1

This commit includes generic manual clean ups as well.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

w: new usage & fix coding style

Coding style fixed and more readable help output.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

include/devname.h: add missing header

The devname.h requires readproc.h header which has dev_t. Unless
header is included the includes in program files must be in
correct order. I also added FIXME item to remind that for uid &
gid value int not correct data type but uid_t or gid_t from
pwd.h.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

tload: new usage & fix coding style

Coding style fixed and more readable help output.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

uptime: new usage & fix coding style

Coding style fixed and more readable help output.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

slabtop: new usage & fix coding style

Coding style fixed and more readable help output.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

pwdx.c: new usage & fix coding style

Coding style fixed and new help output.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

pgrep: add long options & new usage

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

free: do not mix declarations and code [smatch scan]

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: mention long options in man sysctl.8

The commit includes few generic manual clean ups as well.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: mention long options in man pmap.1

Also add range option to manual page. The commit includes few generic manual
clean ups as well.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: mention long options in man watch.1

This patch also changed a little the manual page structure, and wording.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: mention long options in man w.1

The commit includes few generic manual clean ups as well.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: mention long options in man tload.1

The commit includes few generic manual clean ups as well.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: mention long options in man uptime.1

The commit includes few generic manual clean ups as well.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: small nroff fixes to man slabtop.1

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: mention long options in man pwdx.1

The commit includes few generic manual clean ups as well.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

sysctl: options handling & usage output

The sysctl now uses getopt_long and has help screen which be user
friendly. Rest of the modernization is left later, since this is
a command is used in scripts, and changing for instance error
printing to use warn & warnx could break stuff.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

pmap: new usage & fix coding style

Coding style fixed and more readable help output.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

watch: new usage & fix coding style

Coding style fixed and more readable help output.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

Removed xalloc type functions from library

These xalloc functions are a private function for the library. If using
programs need them, then they should make their own error reporting or
use a common file.

Added news item for top search capability

top: update man document for new search capability

This commit documents the top search provisons as a
new section entitled:
   5d. SEARCHING in a window

It also adds minor unrelated content and tweaks some
miscellaneous formatting and spacing inconsistencies.

top: add a search capability

Fixed watch 8 bit so its optional

You can make watch 8bit clean by using the configure option
--enable-watch8bit

watch: support unicode

A patch from Debian.

Bug-Debian: http://bugs.debian.org/240989
Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/procps/+bug/318221
Backported-by: Sami Kerola <kerolasa@iki.fi>

watch: add precision wait time option -p

A patch from Debian.

Bug-Debian: http://bugs.debian.org/183486
Reviewed-by: Craig Small <csmall@debian.org>
Backported-by: Sami Kerola <kerolasa@iki.fi>

watch: interpret ANSI color code sequences

A patch from Debian.

Bug-Debian: http://bugs.debian.org/129334
Backported-by: Sami Kerola <kerolasa@iki.fi>

watch: add -exec and -beep flags and has better quoting

Additionally add -errexit flag (#183346).

A patch from Debian.

Bug-Debian: http://bugs.debian.org/410967
Bug-Debian: http://bugs.debian.org/183346
Reviewed-by: Craig Small <csmall@debian.org>
Backported-by: Sami Kerola <kerolasa@iki.fi>

w: use environment to set user and from/host column widths

A patch from Debian.

Bug-Debian: http://bugs.debian.org/396423
Bug-Debian: http://bugs.debian.org/341439
Backported-by: Sami Kerola <kerolasa@iki.fi>

w: use COLUMNS environment if TIOCGWINSZ fails

A patch from Debian.

Backported-by: Sami Kerola <kerolasa@iki.fi>

w: bassman emulation with -o option

A patch from Debian.

Bug-Debian: http://bugs.debian.org/414906
Backported-by: Sami Kerola <kerolasa@iki.fi>

vmstat: do not scale si/so just like bi/bo

Use strtoull insteadof strtoul for some counters.

A patch from Debian.

Bug-Debian: http://bugs.debian.org/558361
Bug-Debian: http://bugs.debian.org/558134
Backported-by: Sami Kerola <kerolasa@iki.fi>

libprocps-ng: sysinfo.c: truncate the vmstat figure to 32 bits

A patch from Debian.

Bug-Debian: http://bugs.debian.org/620557
Backported-by: Sami Kerola <kerolasa@iki.fi>

tload: remote unneeded optarg and optind variables

A patch from Debian.

Backported-by: Sami Kerola <kerolasa@iki.fi>

sysctl: fix up some option processing

A patch from Debian.

Backported-by: Sami Kerola <kerolasa@iki.fi>