summaryrefslogtreecommitdiffstats
path: root/usr.bin/make/make.c
Commit message (Collapse)AuthorAgeFilesLines
* - Partially revert r200417. config.h brings several definitions,stas2009-12-171-0/+1
| | | | | that appears to be actually used. Without config.h included cross-build of world failed (at least for ARM).
* Remove unnecessary includes.delphij2009-12-111-3/+0
| | | | Reviewed by: rodrigc
* Avoid infinite loops when remaking makefiles not onlyfjoe2009-04-071-5/+21
| | | | for Makefile targets but also for targets they depend on.
* Exit with error code 2 when run with -k (continue if errors) and build failed.fjoe2008-12-181-4/+2
|
* Improve logging when -dm is specified: if the node is consideredfjoe2007-04-201-8/+5
| | | | | out-of-date print not only "modified before source" message but also the path of youngest source.
* The caller of Var_Value() should not change the variable value. Makeharti2005-05-241-7/+7
| | | | | | this clear by constifying the return value. Obtained from: DragonFlyBSD
* Get rid of the third argument to Var_Value() the pointer it pointedharti2005-05-241-12/+4
| | | | | | to has always been set to NULL for some time now. Obtained from: DragonFlyBSD
* Move the definitions of the OP_* constants from make.h into GNode.hharti2005-05-101-0/+1
| | | | | | | | where they actually belong to. Move the definitions of the strings for special macros like "$*" from make.h to parse.h - they're used only in the parser. Submitted by: Max Okumoto <okumoto@ucsd.edu> (7.211)
* Style: indentation.harti2005-04-011-475/+473
|
* Replace another bunch of Lst_ForEachs by LST_FOREACHs and simplify code.harti2005-03-161-145/+99
|
* General whitespace cleanup: remove mixes of tabs and spaces, removeharti2005-02-041-1/+1
| | | | | | space after function names, remove spaces on emtpy lines. Submitted by: Max Okumoto <okumoto@ucsd.edu>
* Clean up include files and file including. Split nonints.h into piecesharti2005-02-011-5/+14
| | | | | | | | | | | | | that get included just where they are needed. All headers include the headers that they need to compile (just with an empty .c file). Sort includes alphabetically where apropriate and fix some duplicate commenting for struct Job, struct GNode and struct Shell by removing one version and inlining the comments into the structure declaration (the comments have been somewhat outdated). This patch does not contain functional changes (checked with md5). Submitted by: Max Okumoto <okumoto@ucsd.edu>
* Instead of dynamically allocating list heads allocated them staticallyharti2004-12-161-38/+35
| | | | | | | | now that their size is only two pointers. This eliminates a lot of calls to Lst_Init and from there to malloc together with many calls to Lst_Destroy (in places where the list is obviously empty). This also reduces the chance to leave a list uninitilized so we can remove more NULL pointer checks and probably eliminates a couple of memory leaks.
* Get rid of the sequential access feature of the lists. This was usedharti2004-12-081-46/+38
| | | | | | | | only in a couple of places and all of them except for one were easily converted to use Lst_First/Lst_Succ. The one place is compatibility mode in job.c where the it was used to advance to the next command on each invocation of JobStart. For this case add a pointer to the node to hold the currently executed command.
* Typedefs of pointers to structs are evil. Make Lst and LstNode typedef ofharti2004-12-071-7/+7
| | | | | | the structs itself not of pointers to them. This will simplify constification. Checked by: diff on the object files
* Make needs no circular lists so remove them from the list code.harti2004-12-071-1/+1
|
* Style: remove a lot of unnecessary casts, add some and spell the nullharti2004-12-011-29/+29
| | | | | | pointer constant as NULL. Checked by: diff -r on the object files before and after
* Stylification: missing spaces, extra space after function names, castsharti2004-11-301-84/+93
| | | | | | | | and the sizeof operator, missing empty lines, void casts, extra empty lines. Checked by: diff on make *.o lst.lib/*.o Submitted by: Max Okumoto <okumoto@soe.ucsd.edu> (partly)
* Add code to enforce the paralleism count (-j N) for the entire treephk2004-11-121-2/+2
| | | | | | | | | | | | | | | | | | | | of submakes spawned during processing. We create a fifo and stuff one character into it for each job we are allowed to run. The name of the fifo is passed to child processes in the MAKE_JOBS_FIFO environment variable. A make which finds this variable on startup will open the fifo and only spawn jobs when it managed to read a token from the fifo. When the job completes a token is writen back to the fifo. Slave make processes get one token for free: the one their parent make got in order to run them. This makes the make processes themselves invisible in the process counts. The net effect is that "make -j 12 -s buildworld" will start at most 12 jobs at the same time, instead of as previously up to 65 jobs would get started.
* Remove efree(), it isn't used consistently enough to even pretend that itjmallett2002-10-231-4/+4
| | | | | | might help on the systems it could possibly be used as a bandaid for. In fact, the only thing it's useful for is instrumenting free(3) calls, and in that capacity, it's better served as a local patch, than a public wrapper.
* Convert make(1) to use ANSI style function declarations. Variablejmallett2002-10-091-34/+15
| | | | | | | | | | | documentation already adequatedly existed in the description in most cases. Where it did not, it was added. If no documentation existed beforehand, then none was added. Some unused dummies for use in the traversal functions were marked as __unused during the conversion. Occasionally, local style fixes were applied to lines already being modified or influenced. Now make(1) should always build with WARNS=3.
* Code that has been #if0'd with the comment "WHY" since revision 1.1 canjmallett2002-10-021-8/+1
| | | | disappear.
* Make make(1) WARNS=6 clean except for const issues. This mostly involvesjmallett2002-09-281-3/+3
| | | | | | | | | | | renaming variables to not shadow libc functions or greater scope locals. Kinda makes one wonder if the extern ones weren't meant in some of these places :) The only thing I'd still like to do WRT this is possibly combine rstat and status in compat.c -- that should be fine, as I do not think the codepaths will want both around at once. Sponsored by: Bright Path Solutions
* Make the DEBUGF() macro portable by (ugh) adding a Debug() function, whichjmallett2002-09-181-17/+17
| | | | | is merely printf() but to stderr. This takes care of the caveat which lead to the use of a vararg macro -- getting everything to stderr.
* Move common use of if (DEBUG(FOO)) printf... to DEBUGF(FOO, ...), usingjmallett2002-09-171-49/+25
| | | | | | | | | variable length arguments to a macro. Bump version as this makes DEBUG statements *always* go to stderr rather than sometimes stdout. There are a few stragglers, which I will take care of as soon as I can. Mostly these relate to the need-for-death-of some of the remote job code. Nearby stylistic nits and XXX added/fixed where appropriate.
* Prefer BSDmakefile over makefile and Makefile.obrien2002-04-131-1/+1
| | | | | Submitted by: jmallett Obtained from: OpenBSD
* Update SCM ID method.obrien2002-04-131-3/+1
|
* De'register.obrien2002-04-131-13/+13
|
* remove __Pimp2002-03-221-6/+6
|
* There's also no point in #typedef'ing void/char pointers. Accordingly,will2000-12-021-27/+27
| | | | | | rip out ClientData/Address pointers and use standard types. Obtained from: OpenBSD
* There's no reason to use fancy forms of NULL. Replace all instanceswill2000-12-021-9/+9
| | | | | | of NIL, NILLST, NILLGNODE, etc. with NULL. Obtained from: OpenBSD
* Use __RCSID()wsanchez2000-07-091-6/+4
|
* Unduplicate IDs from comments, do $Id -> $FreeBSD$ (submitted-by: bde)hoek1999-09-111-2/+0
|
* $Id$ -> $FreeBSD$peter1999-08-281-2/+2
|
* Merge style- and trivial- only changes from OpenBSD (dated 1999/07/29-19:55+1).hoek1999-08-171-13/+15
| | | | Obtained from: OpenBSD, sometimes indirected from NetBSD; myself
* Revert $FreeBSD$ to $Id$peter1997-02-221-1/+1
|
* Make the long-awaited change from $Id$ to $FreeBSD$jkh1997-01-141-1/+1
| | | | | | | | This will make a number of things easier in the future, as well as (finally!) avoiding the Id-smashing problem which has plagued developers for so long. Boy, I'm glad we're not using sup anymore. This update would have been insane otherwise.
* O' to be bitten by CVS. Cleanup after importsteve1996-10-081-0/+2
| | | | | | of Christos' version of make(1) and add Id's. Set straight by: Bruce Evans and Peter Wemm :)
* Merge in NetBSD's changes to make(1). Changes include:steve1996-10-061-3/+12
| | | | | | | | | | | | | | | | - Add the .PHONY, .PARALLEL, and .WAIT directives - Added the -B and -m commandline flags - misc. man page cleanups - numerous job-related enhancements - removed unused header file (bit.h) - add util.c for functions not found in other envs. - and a few coordinated whitespace changes Special thanks to Christos Zoulas <christos@netbsd.org> for help in the merge. A 'diff -ur' between Net and FreeBSD now only contains sccsid-related diffs. :) Obtained from: NetBSD, christos@netbsd.org, and me
* NetBSD fixes: declare more prototypes, move .OPTIONAL in alphabetache1995-06-181-0/+2
| | | | | | | | order, prevent suffix rule, if direct or children dependances exists, forget to free v->name in var.c My fixes: fix famous ^\t\n bug, correct free order in str_end Reviewed by: bde Obtained from: NetBSD, me
* Remove trailing whitespace.rgrimes1995-05-301-18/+18
|
* Bring in a number of changes from NetBSD's make, fixing quite a fewjkh1995-01-231-27/+67
| | | | | | | | | | | | problems in the process: 1. Quoting should work properly now. In particular, Chet's reported bash make problem has gone away. 2. A lot of memory that just wasn't being free'd after use is now freed. This should cause make to take up a LOT less memory when dealing with archive targets. 3. Give proper credit to Adam de Boor in a number of files. Obtained from: NetBSD (and Adam de Boor)
* BSD 4.4 Lite Usr.bin Sourcesrgrimes1994-05-271-0/+859
OpenPOWER on IntegriCloud