I just reviewed it and it looks fine. I have also updated the AMQ-978 and made available my patch. I will let Leon choose the patch.
I appreciate your help in this regard. On another subject: As you suggested, I checked out 4.1 snapshot and attempt to build it maven 2.0.4, but I keep having build error on POM. here is the error I'm getting(let me know if I miss something): [ERROR] BUILD ERROR [INFO] ------------------------------------------------------------------------ [INFO] Error building POM (may not be this project's POM). Project ID: directory-protocols:kerberos-protocol Reason: Error getting POM for 'directory-protocols:kerberos-protocol' from the repository: Error transferring file directory-protocols:kerberos-protocol:pom:0.5 from the specified remote repositories: central (http://repo1.maven.org/maven2), apache.snapshots (http://people.apache.org/repo/m2-snapshot-repository), apache.incubating.releases (http://people.apache.org/repo/m2-incubating-repository) [INFO] ------------------------------------------------------------------------ [INFO] For more information, run Maven with the -e switch [INFO] ------------------------------------------------------------------------ [INFO] Total time: 9 minutes 31 seconds [INFO] Finished at: Mon Oct 23 15:10:43 CDT 2006 [INFO] Final Memory: 7M/13M [INFO] ------------------------------------------------------------------------ Hiram Chirino wrote: > > Did you ever review the patch that I attached to: > http://issues.apache.org/activemq/browse/AMQ-978 > > > On 10/23/06, sileshi <[EMAIL PROTECTED]> wrote: >> >> >> I have added support for "content-length" header field to handle >> Bytes Messages from Java JMS or Perl Stom clients. >> 1. Net::Stomp.pm >> - Rewrote the receive_frame() method to read the header first, >> and based on the "content-length" if any, read the body. >> - Applied binmode to socket to avoid problems on Windows >> >> 2. Net::Stomp::Frame >> - Rewrote parse() method to fix problems I run into. >> this also makes it robust using the power of regular expression. >> >> I have attached the two files below: >> >> http://www.nabble.com/file/3810/Stomp.pm Stomp.pm >> >> http://www.nabble.com/file/3811/Frame.pm Frame.pm >> -- >> View this message in context: >> http://www.nabble.com/Net%3A%3AStomp-0.31-Patch-to-Support-Bytes-Message-tf2496525.html#a6959274 >> Sent from the ActiveMQ - User mailing list archive at Nabble.com. >> >> > > > -- > Regards, > Hiram > > Blog: http://hiramchirino.com > > -- View this message in context: http://www.nabble.com/Net%3A%3AStomp-0.31-Patch-to-Support-Bytes-Message-tf2496525.html#a6962294 Sent from the ActiveMQ - User mailing list archive at Nabble.com.
