Age | Commit message (Collapse) | Author |
|
There is some common code used by both sed & grep that should be put into
utility.c as per Mat Kraai's suggestions/patch on the mailing list.
Specifically, a common regex_compile() and a regex_subst() function need to be
made.
|
|
-Erik
|
|
-Erik
|
|
-Erik
|
|
-Erik
|
|
only function where it's used and it's always good to keep the namespace
clean. :-)
|
|
suggestion on the mailing list.
|
|
-Erik
|
|
Howdy,
Bug #1006 reports that
ln -s /tmp/foo .
does not work correctly. In fact, it appears that any instantiation of
ln -s FILE... DIRECTORY
does not work. The following patch adds support for this form, which
then fixes the particular instance noted in the bug report.
In the process, I needed the basename function. This appears in the
string.h provided by glibc, but not uC-libc. So I wrote my own to go in
utility.c, called get_last_path_component. I also modified the basename
utility to use this function.
At some point it might be desirous to use the basename from the library
if it exists, and otherwise compile our own. But I don't know how to do
this.
Matt
|
|
GNU tr complains on the following:
$ tr a ''
tr: when not truncating set1, string2 must be non-empty
BusyBox tr does not complain:
$ tr a ''
a
^D
0
It should result in an error, not in some spurious output. The attached
patch generates an error.
Matt
|
|
However, the case of
grep foo$ file
didn't work, due to a problem with the flags used in regular expression
compilation. The attached patch fixes this problem.
---patch-------
Index: grep.c
===================================================================
RCS file: /var/cvs/busybox/grep.c,v
retrieving revision 1.30
diff -u -r1.30 grep.c
--- grep.c 2000/07/04 22:17:01 1.30
+++ grep.c 2000/07/10 08:57:04
@@ -141,8 +141,10 @@
if (argv[optind] == NULL)
usage(grep_usage);
- /* compile the regular expression */
- reflags = REG_NOSUB; /* we're not going to mess with sub-expressions
*/
+ /* compile the regular expression
+ * we're not going to mess with sub-expressions, and we need to
+ * treat newlines right. */
+ reflags = REG_NOSUB | REG_NEWLINE;
if (ignore_case)
reflags |= REG_ICASE;
if ((ret = regcomp(®ex, argv[optind], reflags)) != 0) {
---patch-------
Thanks, Matt, it works great.
|
|
(except for mkfs_minix and fsck_minix -- and it doesn't yet link vs uclibc due
to missing stuff in the library).
-Erik
|
|
-Erik
|
|
GROWBY - 1, then it writes the null character just after the buffer. Yipe.
Fix thanks to Matt Kraai <kraai@alumni.carnegiemellon.edu> Thanks Matt!
-Erik
|
|
while debhelper 1.1.24 is available for slink. Just forget the dependancy --
it isn't needed.
-Erik
|
|
-Erik
|
|
-Erik
|
|
-Erik
|
|
-Erik
|
|
-Erik
|
|
-Erik
|
|
deg entries which break mount, df, and friends. Renamed WillThisGoIntoTheKernel
to avoid any confision.
-Erik
|
|
maintain it very well, so including it was not very appropriate. Those wanting
an fdisk are invited to grab a copy from util-linux.
-Erik
|
|
-Erik
|
|
-Erik
|
|
-Erik
|
|
-Erik
|
|
do so... Fix thanks to Matt Kraai <kraai@alumni.carnegiemellon.edu>
-Erik
|
|
-Erik
|
|
the remaining chapters -- Matt Kraai <kraai@alumni.carnegiemellon.edu>
-Erik
|
|
<kraai@alumni.carnegiemellon.edu>
-Erik
|
|
the CVS leaves lying about...
-Erik
|
|
-Erik
|
|
-Erik
|
|
thanks to Daniel Quinlan <quinlan@transmeta.com>
-Erik
|
|
since 0.44.
-Erik
|
|
Thanks Matt!
-Erik
|
|
busybox 0.28.
-Erik
|
|
-Erik
|
|
put a gap between the normal commands and the fancy commands. This makes it
cleaner to add normal commands. Patch from Jon McClintock <jonm@bluemug.com>.
-Erik
|
|
leave setkeycodes active, busybox will not link. Also fix a trivial
use-before-initialize warning. Both fixes from Jon McClintock
<jonm@bluemug.com>.
-Erik
|
|
-Erik
|
|
-Erik
|
|
thanks to simon wood <simon@mungewell.uklinux.net>
-Erik
|
|
-Erik
|
|
-Erik
|
|
-Erik
|
|
-Erik
|
|
will be removing this stuff from /proc/mounts real soon now I am assured.
-Erik
|
|
escape seq. code from tr and echo into utility.c. Fix thanks to
Matt Kraai <kraai@alumni.carnegiemellon.edu>.
* This should close Bug #1015. Please test.
-Erik
|