| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
| |
* There was too much space (three lines) between the diffs in commit mails.
This reduces it to one.
Submitted by: fanf
|
|
|
|
| |
imports.
|
|
|
|
| |
Submitted by: Jun Kuriyama <kuriyama@imgsrc.co.jp>
|
|
|
|
| |
lines of diff in a commit email.
|
|
|
|
| |
actually was.
|
|
|
|
| |
Submitted by: jesper
|
|
|
|
| |
its own subroutine ready for the diff code to use also.
|
| |
|
|
|
|
| |
the configuration file.
|
| |
|
|
|
|
| |
them.
|
|
|
|
| |
to the configuration file creating a new config variable $COMMITTER.
|
|
|
|
|
|
|
|
| |
to determine the location now-a-days.
Move the name of the exclude file into the configuration file.
Partly Submitted by: alfred
|
|
|
|
| |
Submitted by: alfred
|
|
|
|
| |
Submitted by: Jon Parise <jon@csh.rit.edu>
|
|
|
|
|
|
|
| |
stays correctly lined up after indentation.
Requested by: Dan Langille <dan@langille.org>
Submitted by: tobez
|
|
|
|
| |
Submitted by: tobez
|
|
|
|
| |
Partly Submitted by: tobez
|
|
|
|
| |
dependant upon the order that the committer specified them.
|
|
|
|
| |
for FreeBSD).
|
|
|
|
|
|
| |
Sendmail automatically adds one but other MTA's, like postfix,
don't. This should make for a more consistant email across different
platforms.
|
|
|
|
|
|
|
| |
cvs gives them the chance to enter different log messages for each
area. If they do then it's really separate commits and so treat
it as such, mailing separate emails and writing separate logs for
each one.
|
|
|
|
|
| |
that used to be used. Move the bit of code that gets them back
into the bit-rotted commented out code block.
|
|
|
|
| |
Requested by: will
|
|
|
|
|
|
| |
anonymous subroutine in the cfg.pm file if it is defined. This
allows the administrator to add cvsweb URL's to the email, for
instance, at time of generation.
|
|
|
|
|
|
|
| |
All log file entries are now separated by three blank lines;
previously only commits were, and directory creates weren't.
Email messages now aren't terminated by two trailing blank lines.
|
|
|
|
|
| |
better with the commitlogs/, and correct the way that its new line
is done.
|
|
|
|
| |
committed to into the config file.
|
|
|
|
|
|
|
| |
not safe to rely on CVS_USER just yet. It doesn't get cleared in
normal mode, so it would be possible for a local user to pretend
that someone else made a commit in the email (even though the
repository itself was correct.)
|
|
|
|
|
|
|
|
|
|
| |
both in the access controls and in the log message generation.
Use the CVS_USER environment variable if it's defined to obtain
the true username. This is set by later versions of cvs (it's in
1.11.1p1), and gets around the problem of using pserver username
aliases in CVSROOT/passwd. With this fix the committer's true
username is checked and reported instead of the system one.
|
|
|
|
| |
Bug spotted by: alex
|
|
|
|
| |
internal to CVS and shouldn't be mentioned here.
|
|
|
|
|
| |
delta for a removed file for instance, take into account branch
points.
|
|
|
|
|
|
| |
the commit mail.
Requested by: rwatson + others.
|
|
|
|
| |
it done within each individual script.
|
|
|
|
|
| |
the name of the last directory in a multidirectory commit. Move
the definition of this to cfg.pm so that it's defined in one place.
|
|
|
|
| |
commit_prep.pl
|
|
|
|
| |
directory creation.
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
commitlogs. This was controlled by tweaking the $RCSIDINFO variable,
where 0 = off, 1 = mail only, and 2 = both mail and commitlog. It
looks like '1' was broken and so not used by anyone, and '0' isn't
so useful (I'll re-add support if anyone moans enough ;)
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
notification service driven from 'MFC after:' headers in the commit
message.
This commit adds 'MFC after:' into the rcstemplate and associated
code to make sure that if it's used it's used correctly.
Based upon code submitted by Maxim Sobolev <sobomax@FreeBSD.org>.
Submitted by: sobomax
|
|
|
|
|
| |
They've never been logged here before by the looks of things, which
seems a little strange.
|
|
|
|
| |
Tidy up a comment.
|
|
|
|
|
|
| |
from my test environment).
Several dunce caps to: joe
|
| |
|
|
|
|
|
| |
potential bug where the contents of the log message could conflict
with the parsing process.
|
|
|
|
| |
use of @log_lines in a different context.
|
|
|
|
| |
for later use.
|
| |
|
| |
|
|
|
|
| |
change_summary_X subroutines.
|
|
|
|
|
| |
loads of 'my' declarations. It won't be long before this will run with
'strict' switched on.
|
|
|
|
| |
This allows it to be switched off if necessary.
|
|
|
|
| |
the style guide.
|
|
|
|
|
|
|
|
| |
NEVER NEVER list cvs-committers@freebsd.org in these scripts by default!
At least, not without checking the hostname of the machine first!
Pointy-hat to: joe
|
| |
|
| |
|
|
|
|
| |
Remove the 'append_to_logfile' subroutine as it's not used.
|
|
|
|
|
|
|
| |
* local -> my.
* Don't chdir to /tmp.
* Use $TMPDIR to file the temporary files.
* Don't delete the temporary files if we're in debug mode.
|
|
|
|
| |
Problem spotted by: bfumerola
|
| |
|
|
|
|
| |
the process id.)
|
| |
|
|
|
|
| |
along with the prefix that we use for the temporary filenames.
|
|
|
|
|
| |
uppercase:
$debug => $DEBUG and $rcsidinfo => $RCSIDINFO
|
|
|
|
| |
now that the header is generic.
|
|
|
|
|
| |
mail a configuration variable at the top of the script. (The crypto
repository used this, but other projects may find it handy also.)
|
|
|
|
|
| |
Move the mail command and email recipients to the top of the script where
they are easier to modify.
|
|
|
|
|
|
| |
messages and commit mails so that the values always fit the columns.
Reviewed by: markm
|
|
|
|
|
|
|
|
|
|
| |
were applied to; useful for filtering by procmail, etc.
The additional header is of the form:
X-FreeBSD-CVS-Branch: HEAD,RELENG_3,RELENG_4
Requested by: andreas & asmodai
|
| |
|
|
|
|
|
|
| |
from seeing any more than the 'lines:' in the rcs header itself. It used
to scan the entire log message and could "find" another entry if another
log message, complete with full headers, was pasted into the middle.
|
|
|
|
|
|
|
|
| |
(Actually, Mark removed a bit of commented code whice I've left in since
we might use something like it for using cvs-$branch commit messages
one day)
Submitted by: markm
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
are not active until after the commit that chages them)
|
|
|
|
|
| |
various things to inject the commit messages. This should give us back
the 'From: User Name <user@freebsd.org>' headers we're used to.
|
| |
|
|
|
|
|
| |
using bulk_mailer to inject the mail, the address list is ignored now,
this broke the cvs-committers list)
|
|
|
|
|
|
| |
comment-out, and add comments that mlist_map is actually still used
(the commitlogs specifically) so people don't keep offering to remove
it. :-)
|
| |
|
| |
|
|
|
|
| |
This is a workaround until the postmaster fix his scripts.
|
|
|
|
| |
addresses
|
| |
|
|
|
|
|
|
| |
Reviewed by:
Submitted by:
Obtained from:
|
|
|
|
|
|
|
|
|
|
|
| |
- suppress duplicate copies of mail to mailing lists when directories
are crossed. I removed the use of dbm files for simplicity and used
flat text files, but I kinda forgot about the implicit unique keys, so
work around it now.
- Don't log the action header if it's unchanged from the last directory.
It could still possibly be duplicated, but I'm pretty sure the branch
tag hashing will stop that from happening. (I hope :-)
- add an extra 5 chars to the directory column
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
problem...
No seriously, this is a major revamp of the log scripts in preperation
for an update to a later version of cvs (ie: 1.9.8)
Among the changes:
- lots of dead code and useless stuff ripped out
- lots of things rewritten and/or redesigned
- support new message format that's passed from cvs to the log
script so that commits to individual files on branches works.
- perl5 now *required* due to above (hashes of arrays @{$var${tag}})
- commit message format slightly changed due to per-file branches
- new mailing lists (well, from last year) and commitlogs supported,
eg: cvs-contrib, cvs-tools, etc
- more space for filenames on each line before wrapping.
- more intelligent line wrapping, alignment should be better.
- no more insanely long Subject: lines.
There are more changes in the pipeline, including being able to do a
partial commit message when cvs is aborted in the middle of a commit,
but since the diff of the changes is now much larger than the file
itself, and we need the new cvs support, commit this now.
It's had a fair bit of testing in a private copy of the cvs tree, but
it's still not all that unlikely that there won't be things I've missed.
|
|
|
|
| |
PR:
|
|
|
|
|
| |
much use anyway and just makes people reluctant to use 'cvs add' even
though it's more appropriate than 'cvs import' in several cases.
|
|
|
|
| |
for imports..
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
code, as well as simplifying some admin procedures.
The main change is to stop multi-directory commits only having the subject
info of the last directory, and not being sent to all the lists, and not
being logged to the appropriate commitlogs.
updateCVSROOT is gone (yay!), it's functionality is replaced by a cvs-1.6
feature - the 'checkoutlist' file, which is designed explicitly for this.
This will stop the easy-import users from getting told the mailing list
is out of date when a committer has been added but had missed out on being
put on the cvs-committers mailing list.
Update the comments and info in the *info files with the cvs-1.6 headers,
this fixes a few typos, clarifys some things, corrects some misleading
comments, etc.
Some stub files added to indicate how new cvs-1.6+ functionality is used,
but left commented out (taginfo, cvswrappers, etc)
All hard-coded references to /home/ncvs are changed back to $CVSROOT
(as suggested to me by bde) now that cvs-1.6+ makes a special effort
to ensure $CVSROOT is always set. This allows the $CVSROOT files to be
used elsewhere (eg: for testing) with much less pain.
mkavail is history as well, it was often being forgotten. cvs-committers
now takes the user list from /etc/group:ncvs via a trivial perl script.
This is so that we still have the ability to temporarily disable commit
access via 'avail' without cutting off the cvs-committers mailing list.
Since /etc/group editing is still required, this is not a new admin
overhead. If we ever change so that membership of "ncvs" is not required
(eg: /usr/bin/cvs becomes setgid and access control is via avail) then
we are going to have to do it some other way. (I have ideas)
I have tested this on my machines, but it is possible that I have missed
something. As usual, please yell if I've botched it.
|
|
|
|
|
| |
the perl construct "local($dir) = @_; " - perl warns about it, but
it doesn't seem to work if I directly assigned $dir....
|
|
|
|
|
|
|
|
|
| |
I've been using this code in this approximate form for about 6 months now.
I'm keen to hear how useful this is/isn't for people.
It gives an unambiguous indication of how big the commit is, and the
revision numbers are collected into one place to make it easier to find and
follow all the parts of a commit over multiple files.
|
|
|
|
|
| |
Also, try a simple sanity check in "commitcheck" to try and head off
accidental commits on thud over NFS...
|
|
|
|
| |
off newlines. :-O
|
|
|
|
|
|
|
|
|
|
| |
set it. I'm still at a loss to explain why getlogin and `logname`
(which make a supposedly secure system call) are returning somebody else's
username when cvs (a non privileged process) is run on the end of a rsh.
If I do: rsh freefall 'print getlogin' it always seems to work...
(If this doesn't work after this commit, I might commandeer Jeffrey Hsu's
login.. :-) This commit may say 'hsu' though.)
|
|
|
|
|
| |
that the mail process always gets it right and that's a subshell of the perl
script log_accum.pl, so it's worth giving this a go.
|
|
|
|
|
| |
Remove debugging output again.. (which proved that getlogin returns the
wrong value sometimes..)
|
|
|
|
| |
I can see how 'hsu' is getting attributed with some of my commits.
|
|
|
|
|
|
|
|
| |
Reviewed by:
Submitted by:
Obtained from:
Are blank, after this commit, cvs will remove them from the Log:
section.
|
|
|
|
|
|
|
|
|
|
| |
Branch: detection logic into this. It's only 8 new lines, but due
to the learning curve it took me 3 hours to write them :-(.
Also fixed the $FreeBSD$ keyword to be $Id$ since it is turned off.
Next.. auto deletion of blank ``Reviewed, Submitted and Obtained" lines.
That should only take an hour more :-)
|
|
|
|
|
|
| |
/usr/local.
Submitted by: asami
|
|
|
|
|
|
|
|
|
| |
to get the commit timestamp. This makes the time stamps comparable.
I'm rather in favour for using gmtime() instead of localtime(), but i
guess i cannot convince all you american guys. :)
I'll let up the RCS checkout to our CVS master...
|
|
|
|
|
|
|
|
|
|
|
| |
automagically each and any time it's changed. Change log_accum.pl
to send to the new cvs-committers mailing list that uses this
file. This should cut down on the size of our monster cvs mail
headers considerably! :-)
Many thanks to Gary Clark II for these changes.
Submitted by: gclarkii
|
|
|
|
| |
CVS:
|
|
|
|
| |
Submitted by:
|
|
|