summaryrefslogtreecommitdiffstats
path: root/usr.bin/xargs
Commit message (Collapse)AuthorAgeFilesLines
* Fix typos; each file has at least one s/seperat/separat/schweikh2002-08-111-2/+2
| | | | | | | | | | (I skipped those in contrib/, gnu/ and crypto/) While I was at it, fixed a lot more found by ispell that I could identify with certainty to be errors. All of these were in comments or text, not in actual code. Suggested by: bde MFC after: 3 days
* Declare environ as char **environ like in environ(7), not char *environ[].tjr2002-07-011-1/+1
| | | | | | | | This corrects a problem whereby xargs could not walk the environment table to count the amount of space it used, and treated it as if it were empty. This problem was introduced in rev 1.15. MFC after: 2 days
* Stylistic nit:jmallett2002-06-221-5/+5
| | | | | main()'s argv argument is char*[], and functions that may inherit that arg should use char*[] as well.
* Check for results of repeated calls to strnsubst(), as well as for thejmallett2002-06-221-6/+17
| | | | behaviour with NULL match string, as that has changed over time.
* When giving an example that relies on shell expansion/globbing, don't use ajmallett2002-06-221-1/+1
| | | | | | replstr for -J that will be interpereted by the shell. MFC after: 1 day
* Note that this appeared at least as early as PWB UNIX.jmallett2002-06-101-0/+4
| | | | Use the literal string 'PWB UNIX', as we still have no .At macro for it.
* Err, duh, free(3) doesn't set its argument to NULL... Fix realloc of a freedjmallett2002-06-041-1/+3
| | | | chunk.
* Restructure sanity checks so that -I and -J just override eachother, ratherjmallett2002-05-301-5/+6
| | | | | | | than triggering a usage(). Allow -R and -I to be specified in any order, and thus change how -R checks for -I not being given and triggering a usage(). Partially requested by: gad
* Allow the input line to be NULL, and teach strnsubst() that NULL means to usejmallett2002-05-302-3/+8
| | | | a nil-string.
* mdoc(7) police: punctuation, miscellaneous.ru2002-05-301-8/+8
|
* Simplify prerun() in the case where there are no arguments. Can't do -I or -Jjmallett2002-05-171-1/+1
| | | | | substitution if `utility' is not specified and we're using the buildin echo; It has no arguments for us to abuse.
* Fix a "tiny and squeeby and little" markup mistake.jmallett2002-05-141-1/+1
| | | | Submitted by: fenner
* xargs.c,v 1.33 broke the upgrade path from old versions of 4.x and 5.0.ru2002-05-102-1/+13
|
* -I and -J both set the same variable, replstr, to the string to replace withjmallett2002-05-061-0/+2
| | | | | a line of input, and both work differently, so prevent them from both being passed to xargs(1).
* Modify the -p implementation to use a user's locale, so they can respond tojmallett2002-05-052-13/+63
| | | | | | | | | | | | the prompt in their native language. Also make the prompt fit what POSIX asks for (?...). This should not affect use of -p with yes(1) [as every locale I know of matches 'y' as YESEXPR as well], but that's what -t is for anyway. -p is meant to be really used interactively. Submitted by: tjr, jmallett
* Fix a typo.jmallett2002-05-051-2/+3
| | | | | | Submitted by: Carl Schmidt <cschmidt@slackerbsd.org> Wrap an obscenely long line while I'm here.
* In an effort to make this utility easier to work with at a source level, movejmallett2002-05-051-141/+154
| | | | | | | | | | | | | | out the parse loop to a seperate function, and move local variables around as is needed. To keep the state of some variables and keep from having huge arg lists to functions, make a bunch global. Change use of err(3) in situations where malloc(3) will be setting errno to errx(3) since assuming malloc(3) sets errno is non-portable, and provides no more useful information in the context of FreeBSD. Submitted by: bde (err/errx), Carl Schmidt <cschmidt@slackerbsd.org> (some of the movement of the input loop to a function) Approved by: src/tools/regression/usr.bin/xargs
* Grouping changes.jmallett2002-05-041-2/+2
| | | | Submitted by: Carl Schmidt <cschmidt@slackerbsd.org>
* Don't use pointers as booleans.jmallett2002-05-041-3/+3
|
* Replace calloc(3) with malloc(3), and set the tail of thejmallett2002-05-041-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | argument list to NULL, to terminate the arguments passed to execvp(2). Thanks to: bde (for pointing out some missing parens) And with apologies to Ozzy Osbourne: On his way to dinner It took him by surprise When an email from bde Said his code was full of lies Style (indentation) And his bracing were all wrong He casted to size_t When he should have cast to long I don't mind Single letter identifiers Unwrapped Line Over 80 chars Far over 80 chars Who can we get to send diffs We need Bruce D. Evans Shows you comparason to his We need Bruce D. Evans again
* Fix a bug whereby we were getting ~0 and comparing it to maxsize, i.e. ifjmallett2002-05-031-1/+1
| | | | | | | s1 was 0 length, and replstr was 0 length, etc., we would end up subtracting one from zero and seeing if it was greater than the size_t (unsigned) var maxsize... This would cause us to return a string consisting of essentially only match, which is not the right behaviour if we have 0 length inpline.
* Move substitution out to a function, prerun(), and replace use of run() withjmallett2002-05-031-85/+78
| | | | | | | it. It handles everything (right now) that needs done before run(), except the -J case, because while that would be easy, I don't quite understand -J. Reviewed by: src/tools/regression/usr.bin/xargs
* Get rid of bogus holder for realloc(3).jmallett2002-05-031-6/+5
|
* Rewrite the loop that handles substitution in the -I case, and try to makejmallett2002-05-031-15/+75
| | | | | | | | | it easier to understand. Making it easy to understand isn't all that easy, so litter the code with comments so some other poor soul can come along some day and work on this if they see fit to do so. Avoid calling strlen(3) to check for a nil-string, when we can just check for *str=='\0'. Approved by: src/tools/regression/usr.bin/xargs/
* Rename `lflag' to `Lflag' to match the option's case.jmallett2002-05-031-6/+6
|
* Revert revision 1.22. I incorrectly modified the Berkeley identifiers.jmallett2002-05-031-3/+12
| | | | Requested by: bde, mike
* __COPYRIGHT() and __SCCSID().jmallett2002-05-021-12/+3
|
* An explicit cast to size_t for an inline integer, since the two are not thejmallett2002-05-021-2/+2
| | | | | | | same on Alpha and lint(1) pointed that out. lint(1) on the same architecture pointed out how silly a cast to (u_int) to malloc(3)'s argument was. Change that to size_t.
* A markup fix, and document -R as non-standard.jmallett2002-05-021-3/+5
|
* Make -J a proper flag internal to the code (rather than just check forjmallett2002-05-022-10/+30
| | | | | | | | use of replstr and lack of Iflag), and add -R, which when given with -I controls the number of arguments on which replacement will be done. Some people happen to think it's idiotic to limit to 5 arguments, so let the user override it if they like.
* *str is spelled 's1' inside the body of the code.jmallett2002-05-021-4/+2
| | | | | Make 'this' not a local variable, since it isn't necessary or complex enough to warrant such.
* Rework some of the -I support so I can't find a way to make xargs(1) core orjmallett2002-05-022-15/+14
| | | | | | | | corrupt memory. Simplifies the code in one or two places, also removes some code that looks like it was bogus or incomplete. Update strnsubst to have one or two extra conditions which maybe would make it more efficient, or at least more versatile. This is likely a no-op.
* In the case where we can't open /dev/tty, fall back to -t behaviour.jmallett2002-04-191-9/+7
| | | | Submitted by: fenner
* Bump WARNS to 4.jmallett2002-04-191-0/+1
|
* Compound two fprintf(3)s into 1 using string concatenation.jmallett2002-04-191-2/+1
|
* After 3 months...jmallett2002-04-194-44/+272
| | | | | | | | | | | | | | | | | Merge xargs(1) with that of xMach. Bring in xargs(1) changes to add -L and -I as per the Single Unix Specification version 3. Proper exit status numbers are implemented, and the manual page has been updated to reflect reality. The code has been ANSIfied, and a new file has been added to xargs(1) to do the substring substitution as SUSv3 requires. Traditional behaviour should not be affected, use of -J should be deprecated in favor of the more portable -I (though -J has been left, for now). Submitted by: me, tjr (the exit status stuff) Obtained from: xMach
* ANSIfy: Function declarations and prototypes, use of environ(7).jmallett2002-03-151-9/+8
| | | | | Reviewed by: mike Approved by: mike
* Change back to using vfork() now that execvp() is vfork()-safe. If execvp()des2002-03-091-7/+10
| | | | | | | | fails, errno is saved to a volatile variable that the parent later inspects. PR: bin/34898 Submitted by: Tim J. Robbins <tim@robbins.dropbear.id.au> MFC after: 1 week
* Leave the `copyright' variable exposed, since the main purpose of itmike2002-03-071-2/+2
| | | | | | is to put a copyright in the object file. Submitted by: bde
* Fix vendor ID (mostly obtained from Rev 1.1).mike2002-03-051-7/+8
|
* WARNS=2 fixes, remove register keyword, use __FBSDID()markm2001-12-111-10/+14
|
* Spelling police: extention -> extension.dd2001-07-051-1/+1
|
* Add a -J replstr option that allows the user to tell xargs to insertdd2001-05-152-6/+68
| | | | | | | | the data read from standard input at a specific point in the command line arguments rather than at the end. Submitted by: dd, gad Reviewed by: gad, brian
* Alphabetize command-line options in the synopsis.dd2001-05-021-1/+1
|
* mdoc(7) police: Fixed typo and markup in rev.1.11.ru2001-04-131-2/+2
|
* o The -s limit is ARG_MAX - 4K, not ARG_MAX - 2K.brian2001-04-101-4/+14
| | | | | | | | | | | | | | | | | | o Mention that the current environment is part of the -s calculation. o Add a BUGS section that warns against executing a program that increases the size of the argument list or the size of the environment. I have wondered for a while what the difference is between get a big list | xargs sudo command which fails and get a big list | sudo xargs command which succeeds. The answer is that in the first case, sudo expands the environment and pushes the amount of data passed into execve over the E2BIG threshold.
* Prepare for mdoc(7)NG.ru2000-12-271-2/+3
|
* Prepare for mdoc(7)NG.ru2000-12-191-1/+0
|
* mdoc(7) police: use the new features of the Nm macro.ru2000-11-201-1/+1
|
* Use .Xrcharnier2000-03-261-2/+2
|
OpenPOWER on IntegriCloud