Hi Nate!

Once again thank you for applying uploaded patches, I've checked the code and 
it looks fine. There are some files, however, that should be removed from the 
"activemq-core" project since they are no longer used. The files are...

activemq-core/src/gram/script
        GenerateCppMarshallingClasses.groovy
        GenerateCppMarshallingHeaders.groovy

activemq-core/src/gram/java/org/apache/activemq/openwire/tool
        OpenWireCppMarshallingClassesScript.java
        OpenWireCppMarshallingHeadersScript.java

Also, the above file names should be removed from the "openwire:generate" goal 
in maven.xml.

 <java classname="org.codehaus.gram.Gram" fork="true">
  <classpath refid="openwire.classpath" /> 
  <sysproperty key="openwire.version" value="${openwire.version}" /> 
  <arg value="src/main/java" /> 
  <arg value="src/gram/script/GenerateCppMarshallingClasses.groovy" />    <-- 
REMOVE!
  <arg value="src/gram/script/GenerateCppMarshallingHeaders.groovy" />    <-- 
REMOVE!
  <arg value="src/gram/script/GenerateJavaMarshalling.groovy" /> 
  <arg value="src/gram/script/GenerateJavaTests.groovy" /> 
  <arg value="src/gram/script/GenerateCSharpMarshalling.groovy" /> 
  <arg value="src/gram/script/GenerateCSharpClasses.groovy" /> 
  <arg value="src/gram/script/GenerateCppHeaders.groovy" /> 
  <arg value="src/gram/script/GenerateCppClasses.groovy" /> 
  <arg value="src/gram/script/GenerateCMarshalling.groovy" /> 
  </java>

Thanks! :)

Regards,
Mats

-----Original Message-----
From: Nathan Mittler (JIRA) [mailto:[EMAIL PROTECTED] 
Sent: den 27 maj 2006 16:02
To: [email protected]
Subject: [jira] Commented: (AMQ-656) Update of AMQ C++ client

    [ 
https://issues.apache.org/activemq/browse/AMQ-656?page=comments#action_36210 ] 

Nathan Mittler commented on AMQ-656:
------------------------------------

Mats,
The 060518 patch has been applied and I moved the gram files to the 
activemq-core project.  Let me know if everything is in order.

Regards,
Nate

> Update of AMQ C++ client
> ------------------------
>
>          Key: AMQ-656
>          URL: https://issues.apache.org/activemq/browse/AMQ-656
>      Project: ActiveMQ
>         Type: Improvement

>   Components: JMS client
>     Reporter: MF
>  Attachments: README.TXT, patch_060518.zip, source_060323.zip, 
> source_060324.zip, source_060404.zip, source_060406.zip, 
> source_060425.zip, source_060508.zip, source_060515.zip
>
>
> Attached is a new update of the C++ client, the zip-file contains the full 
> source since the update is a major overhaul.

--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   https://issues.apache.org/activemq/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira

Reply via email to