Kristis,

Attached is the logfile from a "good" small log message commit and a "bad"
big log message commit using your patch to the Connection.pm file.

Nothing was logged by Connection.pm in the failed case.

Regards,
Mike

-----Original Message-----
From: Kristis Makris [mailto:[EMAIL PROTECTED]
Sent: Thursday, January 17, 2008 1:40 PM
To: Brown, Mike
Cc: '[email protected]'
Subject: RE: [scmbug-users] very large commit comments break scmbug +
bugz illa


On Thu, 2008-01-17 at 12:58 -0600, Brown, Mike wrote:
> Krisitis,
> 
> I found that the logfile was not being written because of a permissions
> conflict.  I fixed the permissions issue and also put in the logging you
> suggested with some additional log messages.  I've attached 5 files for
your
> review:
> 
> a) Subversion.pm with logging
> b) Glue.pm with logging
> c) logfile generated with a small commit message (this worked)
> d) logfile generated with the large commit message (this failed)
> e) text file containing large commit message

I'm glad it was just a permissions issue. The debugging information you
provided is great.

I'm wondering if the freeze has something to do with the socket opened
to the daemon. Can you add the following debugging patch too ?

http://bugzilla.mkgnu.net/show_bug.cgi?id=1093#c12

We need to add officially add logging in the glue, really.


<< SMALL LOG MESSAGE ON SUBVERSION COMMIT >>
Glue.pm: Will read configuration file 
'/data/svn/test/hooks/etc/scmbug/glue.conf'
Glue.pm: Read configuration file '/data/svn/test/hooks/etc/scmbug/glue.conf'
Glue.pm: Will check the configuration
Glue.pm: Checked the configuration. Will now prepare an activity.
Subversion.pm: Will read the original log message
Subversion.pm: Read the original log message
Subversion.pm: ill begin examining what changed...
Subversion.pm: Examining line 1 for file 'trunk/test.txt'...
Subversion.pm: after last_revision
Subversion.pm: last revision is N/A
Subversion.pm: after affected files
Subversion.pm: after product name
Subversion.pm: check for tags/tagname/filename
Subversion.pm: done with prepare_activity_verify_or_commit
Glue.pm: Prepared an activity
Glue.pm: Detected branch name
Glue.pm: Checked policy adherence
Connection.pm: process_activity begins.
Connection.pm: will process an activity.
Connection.pm: prepared a request.
Connection.pm: submit_request begins.
Connection.pm: will instantiate a socket.
Connection.pm: will set the autoflush flag.
Connection.pm: will read the banner from the daemon.
Glue.pm: Processed the activity
Glue.pm: Process sub completed
Glue.pm: Will read configuration file 
'/data/svn/test/hooks/etc/scmbug/glue.conf'
Glue.pm: Read configuration file '/data/svn/test/hooks/etc/scmbug/glue.conf'
Glue.pm: Will check the configuration
Glue.pm: Checked the configuration. Will now prepare an activity.
Subversion.pm: Will read the original log message
Subversion.pm: Read the original log message
Subversion.pm: ill begin examining what changed...
Subversion.pm: Examining line 1 for file 'trunk/test.txt'...
Subversion.pm: after last_revision
Subversion.pm: Finding the previous version of this file...
Subversion.pm: Found it
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: Examining the remaining history...
Subversion.pm: after affected files
Subversion.pm: after product name
Subversion.pm: done with prepare_activity_verify_or_commit
Glue.pm: Prepared an activity
Glue.pm: Detected branch name
Glue.pm: Checked policy adherence
Connection.pm: process_activity begins.
Connection.pm: will process an activity.
Connection.pm: prepared a request.
Connection.pm: submit_request begins.
Connection.pm: will instantiate a socket.
Connection.pm: will set the autoflush flag.
Connection.pm: will read the banner from the daemon.
Glue.pm: Processed the activity
Glue.pm: Process sub completed

<< BIG  LOG MESSAGE ON SUBVERSION COMMIT >>


Glue.pm: Will read configuration file 
'/data/svn/test/hooks/etc/scmbug/glue.conf'
Glue.pm: Read configuration file '/data/svn/test/hooks/etc/scmbug/glue.conf'
Glue.pm: Will check the configuration
Glue.pm: Checked the configuration. Will now prepare an activity.
Subversion.pm: Will read the original log message
Subversion.pm: Read the original log message
Subversion.pm: ill begin examining what changed...
Subversion.pm: Examining line 1 for file 'trunk/test.txt'...
Subversion.pm: after last_revision
Subversion.pm: last revision is N/A
Subversion.pm: after affected files
Subversion.pm: after product name
Subversion.pm: check for tags/tagname/filename
Subversion.pm: done with prepare_activity_verify_or_commit
_______________________________________________
scmbug-users mailing list
[email protected]
http://lists.mkgnu.net/cgi-bin/mailman/listinfo/scmbug-users

Reply via email to