[BusyBox] [BusyBox 0000168]: pidof, make -s optional and add optional -o
bugs at busybox.net
bugs at busybox.net
Wed Mar 16 22:31:25 UTC 2005
A NOTE has been added to this issue.
======================================================================
<http://busybox.net/bugs/view.php?id=168>
======================================================================
Reported By: bernhardf
Assigned To: BusyBox
======================================================================
Project: BusyBox
Issue ID: 168
Category: New Features
Reproducibility: N/A
Severity: tweak
Priority: normal
Status: assigned
======================================================================
Date Submitted: 03-16-2005 14:11 PST
Last Modified: 03-16-2005 14:31 PST
======================================================================
Summary: pidof, make -s optional and add optional -o
Description:
For pidof, i want to make support for option -s (single shot) optional
instead of unconditional as it is now.
Also, this adds optional support for option -o (omit given pid).
Please consider reviewing and applying.
Thanks to Vladimir for pointing out alot of errors i managed to make.
======================================================================
----------------------------------------------------------------------
bernhardf - 03-16-05 14:31
----------------------------------------------------------------------
If -s and -o are not compiled into pidof.o.new, size stats are:
text data bss dec hex filename
262 0 0 262 106 pidof.o.oorig
236 0 0 236 ec pidof.o.new
and in the cleanup, i foobared again.. :-(
The cleanup is obviously depending on the OMIT flag.
#ifdef CONFIG_FEATURE_CLEAN_UP
#ifdef CONFIG_FEATURE_PIDOF_OMIT
Sorry for that.
Issue History
Date Modified Username Field Change
======================================================================
03-16-05 14:11 bernhardf New Issue
03-16-05 14:11 bernhardf File Added: pidof-optional_args.3.diff
03-16-05 14:31 bernhardf Note Added: 0000090
======================================================================
More information about the busybox
mailing list