Commit graph

21 commits

Author SHA1 Message Date
Will Andrews
6e23ca1d3e Fix off-by-one error that was causing an infinite loop in some very
weird cases.  AAARGGH!!!

Discovered by:	will, sobomax, alex
Fix by:		Dima Dorfman <dima@unixfreak.org>
2001-01-16 23:11:38 +00:00
Will Andrews
759fa23f72 Fix two bugs: pass $passenv to make *always*; remember that the initial
join() in the autofill algorithm doesn't need a space.

Bugged by:	alex
2001-01-07 07:25:18 +00:00
Will Andrews
2f7c6bd9f5 Fix breakage when someone uses -t option: pass DISTDIR environment variable
to the extra testing's make procs.  Fix _stupid_ mistake where -n option
had no effect on hindering ``cvs add'' and CVSROOT/modules updating.

Cosmetics; remove extra space in generated commit log message and fix the
usage section's style in regards to options that take arguments.  Add
missing -g option's mention.

Submitted by:	assar (DISTDIR, CVSROOT/modules updating routine bugs)
2001-01-01 23:12:31 +00:00
Will Andrews
efc8971fd6 Several changes. First, depend on another environment variable rather
than CVSROOT (now ADDPCVSROOT) to avoid conflicts for people who normally
set CVSROOT to something besides freefall (like myself).  Second, allow
autofill to work for non-PR additions by checking for a -l argument of -1.
Third, remove the bogus $portname check in autofill that screws up if you
are adding something with a PKGNAMEPREFIX (i.e. p5-).
2001-01-01 05:50:33 +00:00
Will Andrews
8de8dbb8e6 Here's my holiday present to the ports team: new -l option allows you to
let portlint fill out your commit log message.  It takes data from your
port's Makefile and pkg-comment to generate this, then displays the
resulting file and gives a chance to fix any problems.

Also, change -c option to move the file to the temp directory.  Since -c
and -l option conflict, -l will take precedence.  Move -c option down to
where directories are parsed.

Reviewed by:	Jeremy Shaffner <jeremy@external.org>
2000-12-31 11:24:13 +00:00
Will Andrews
51dc79714d Add new option: -c file : Pass a file containing the log message for the
new port.  Slight hack used to obtain absolute path for the file.  I'll
probably change that to make it copy the file to the tmpdir instead.  For
now, this provides a stepping stone to use to begin automating commit log
messages - with, of course, a chance to edit the file again.
2000-12-30 02:20:13 +00:00
Will Andrews
6d4b5c5dad Add -g option to disable committing to CVSROOT/modules, useful if someone
is going to commit a large number of ports and doesn't want to bloat
the repo any more than necessary.  Later on I'll add support to
modulesupdate to do multiple modules at the same time.  I should check that
script in CVS, but nobody's repocopied non-ports-specific tools to src yet
to my knowledge so I can't do that.
2000-12-19 08:16:06 +00:00
Will Andrews
514c1d0973 Functional modification: Make -n only apply to cvs commit, NOT *all*
cvs ops.  Many committers found it difficult to debug while using addport
with the -n argument.  I'm inclined to agree with the sentiment.

Submitted by:	sada, deischen, others
2000-12-08 19:54:54 +00:00
Will Andrews
9cd0d7f466 Add standard 2-clause BSD license to protect my (and mharo's) copyright. 2000-11-04 12:19:43 +00:00
Will Andrews
fc010c8c8b Overhaul addport again. Should be quite functional now.
1) Re-add functionality removed in revision 1.11, but only as an option.
   This time, it checks out its own copy of ports/Mk and points PORTSDIR
   at the correct location.  To use this, use the -a option.  It will not
   work if it can't find portlint in your path.
2) Add option -f to disable fetching the distfile.
3) Add option -m to disable checking out a copy of Mk for 1).  For those
   that have a current Mk in PORTSDIR and don't want to check out a fresh
   copy for whatever reason.
4) Support CVSROOT environment variable.
5) Note environment variables recognized by addport.
6) Fix breakage trying to use -d with multiple directories (not tested).
7) Actually use $interactive variable.
8) Update usage().
9) Note original RCSID for my shell script; take over as MAINTAINER.

Tested by:	gshapiro (this revision was used in adding sendmail)
2000-10-14 05:06:09 +00:00
Will Andrews
ae9e3e2171 Since people don't seem to appreciate the fact that addport can do
automatic checking on their ports to be added, I'm removing it and its
overriding -v option.  Let the improper port additions continue, and screw
anyone who doesn't care about the repo.

Propelled by:	obrien, msmith
2000-10-11 00:14:00 +00:00
Will Andrews
65d5ba399c Use -t flag with portlint.
Submitted by:	trevor
2000-10-04 03:58:52 +00:00
Will Andrews
6e07bc037c Fix bogon where replies were not chomp()'d.
Submitted by:	knu
2000-10-01 15:37:22 +00:00
Will Andrews
ad8939a4f6 Make -n work again. Add new option -s to allow people to specify a
$DISTDIR to put distfiles in, if they can't touch $PORTSDIR/distfiles.
If this is freefall, default to $tmpdir, which is relatively secure since
most users on it are trusted.  Reset FETCH_BEFORE_ARGS to "-btA" instead
of "-btsA", so the distfile is actually fetched.  Add a method to allow
the name of the module to be changed if -i is used.  Update to use the
modulesupdate that's in /usr/local/bin.

Addport should now work properly on freefall, without -v option.

Submitted by:	roger (-s, $tmpdir by default on freefall)
Thanks to:	peter (copying modulesupdate to /usr/local/bin)
2000-08-19 08:08:47 +00:00
Will Andrews
727770ca84 Properly check -d argument, and return usage() if it's not there. Also use
Sys::Hostname with hostname() instead of `hostname`.

Hostname stolen from:	CVSROOT/log_accum.pl
2000-08-11 21:44:18 +00:00
Will Andrews
4367619221 Minor cleanups. Print usage() if no arguments are passed. Get rid of
unused $pwd variable.  Remove extra cruft.  Fix comments.

usage() problem submitted by:	knu
2000-08-11 14:31:28 +00:00
Will Andrews
b1a6849d31 Major update to addport. Many *MANY* bugfixes. Addport should now work
with a directory specified as ``.'' or containing a ``/''.  A new option -i
was added to allow people more control over the port's position in the CVS
repository.  A new feature was added to allow multiple ports to be added in
the same execution of addport.  Addport will now add a port to the category
Makefile properly if it is the first port in a category.

A major reorganization of addport was also performed - now the functions
are at the *bottom* instead of scattered around.  This should allow easier
understanding of the process addport goes through.  I've also added -w to
perl's exec args, and moved to a hash for getopts().

usage() was extended to help explain addport's functionality better.

Bugs submitted by:	nbm
Some help from:		sheldonh
2000-08-09 19:27:29 +00:00
Will Andrews
9f764ffc42 Fix special case where people use ``.'' for the argument to -d. I'm not
very good with perl yet, so anyone who can propose a better way to do
this (with s/// or m// or something using regex) that might also include
the case where the argument contains slashes (i.e. games/somegame).  But
anyways, this should catch folks who use ``.''.  :->

Submitted by:	obrien, sada
2000-08-06 02:51:30 +00:00
Will Andrews
ed1447026c Allow existing CVS_RSH environment var to take precedent.
Submitted by:	sobomax
2000-07-03 19:58:41 +00:00
Will Andrews
bbdaf83188 Fix addport so that it will work on freefall. Remove old cruft left
around from older versions of addport.  Make addport a little more helpful
by actually displaying usage() if no args are passed.
2000-06-30 23:33:35 +00:00
Michael Haro
8ec2364637 Future replacement to easy-import for adding ports to the tree.
This still needs work, but I want to get this out there so people
can comment on it.  This is really only of use to committers.
2000-05-01 19:47:14 +00:00