Everything
php
Learn Community Releases The PHP Project
Home » Project » Newsgroups » php.internals » Commit messages »

Commit messages

php.internals

Andi Gutmans

21 years ago
It seems that everytime I commit to Zend CVS the commit message isn't sent out. For other's it works. It might be due to the warnings I'm getting. Does anyone have an idea what the problem could be? Checking in zend_compile.c; /repository/ZendEngine2/zend_compile.c,v <-- zend_compile.c new revision: 1.597; previous revision: 1.596 done Use of uninitialized value at /repository/CVSROOT/loginfo.pl line 68. Use of uninitialized value at /repository/CVSROOT/loginfo.pl line 71. Use of uninitialized value at /repository/CVSROOT/loginfo.pl line 83. Thanks, Andi

Derick Rethans

21 years ago
On Tue, 12 Oct 2004, Andi Gutmans wrote:
> It seems that everytime I commit to Zend CVS the commit message isn't sent > out. For other's it works. > It might be due to the warnings I'm getting. Does anyone have an idea what > the problem could be?
Sascha was doing something with it, and committed a fix to the script for it. Perhaps he can clarify? Derick
-- Derick Rethans http://derickrethans.nl | http://ez.no | http://xdebug.org

Sascha Schumann

21 years ago
That was an assert, not a fix. It appears to be a change in how cvs invokes the commitinfo/loginfo scripts. I will dig into the issue later, unless someone beats me to it. - Sascha On Wed, 13 Oct 2004, Derick Rethans wrote:

Derick Rethans

21 years ago
On Wed, 13 Oct 2004, Sascha Schumann wrote:
> That was an assert, not a fix. > > It appears to be a change in how cvs invokes the > commitinfo/loginfo scripts. I will dig into the issue later, > unless someone beats me to it.
I know they changed that in later CVS versions. But for my script it keeps working fine, although I get the following message: cvs commit: warning: commitinfo line contains no format strings: "/usr/local/bin/php -q /repository/CVSROOT/commitinfo.php" Appending defaults (" %r/%p %s"), but please be aware that this usage is deprecated. /repository/xdebug/README,v <-- README new revision: 1.8; previous revision: 1.7 cvs commit: warning: Set to use deprecated info format strings. Establish compatibility with the new info file format strings (add a temporary '1' in all info files after each '%' which doesn't represent a literal percent) and set UseNewInfoFmtStrings=yes in CVSROOT/config. After that, convert individual command lines and scripts to handle the new format at your leisure. Perhaps the FreeBSD folks did something already so that things screw up now. Derick
-- Derick Rethans http://derickrethans.nl | http://ez.no | http://xdebug.org