I have just applied your patch. At some point, once we get things more stable, we should decide witch way to go, and maybe promote the das_lite to just c++ das. For now this should be OK.
BTW, I liked Pete's comments : "keep DASing ;-)" -- Luciano Resende http://people.apache.org/~lresende On 3/15/07, Adriano Crestani <[EMAIL PROTECTED]> wrote:
I think you got the point when you said to remove the files from the project, I don't know how I hadn't thought about it. Because the main reason to create a new project in VS solution is to avoid the compilation errors from those many classes that are not completed implemented. But also because I wanted to keep the previous implementation, for example, of the class ReadCommandImpl that was reimplemented in das_lite project. With diferent versions of classes we can define what is already implemented completely(das_lite project) and what remains to be implemented(das_runtime project). I think we could try another approach where we could move the classes from the das_runtime to a trunk or something like and the classes that we are actually implementing from the das_lite(cpp/das/runtime/das_lite/src/) project to the das_runtime(cpp/das/runtime/core/src) project and continuing implementing from there. What do you think? Adriano Crestani On 3/15/07, Luciano Resende <[EMAIL PROTECTED]> wrote: > > Hi Adriano > > I was looking into your patch, and realized you had created a new > project > located at (cpp/das/runtime/das_lite/src/) instead of continuing to work > on > (cpp/das/runtime/core/src). I tought that from a previous quick discussion > > we had agreed that das_lite was more a initial simple scenario rather then > a > new project. > > I have couple questions : > - Could you please elaborate what your plans are here ? > - What's the main differences between C++ DAS and C++ DAS Lite ? > - Is this new project a small subset of files that were duplicated > from cpp/das/runtime/core/src ? If so, is there a way to have one project > but only compile/use the subset of files required, maybe just not adding > them to the VS project ? > > I'll wait after clarification on this subject before I apply the patch. > If anyone else have other thoughts/questions, please let me know. > > -- > Luciano Resende > http://people.apache.org/~lresende <http://people.apache.org/%7Elresende > > > ---------- Forwarded message ---------- > From: Adriano Crestani (JIRA) <[email protected]> > Date: Mar 15, 2007 10:39 AM > Subject: [jira] Updated: (TUSCANY-1140) Implementation of DAS Lite Command > classes > To: [email protected] > > > [ > > https://issues.apache.org/jira/browse/TUSCANY-1140?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel > ] > > Adriano Crestani updated TUSCANY-1140: > -------------------------------------- > > Attachment: tuscany1040.crestani.20070315.patch > > - Added a new project to vs solution titled das_lite > > - Implementation of ReadCommanImpl, BaseCommandImpl and CommandImpl > classes > > - Integration with sdo on ReadCommanImpl::buildGraph method is not working > > correctly > > - Also added other classes of das_lite library, but not completely > implemented yet > > > Implementation of DAS Lite Command classes > > ------------------------------------------ > > > > Key: TUSCANY-1140 > > URL: https://issues.apache.org/jira/browse/TUSCANY-1140 > > Project: Tuscany > > Issue Type: Sub-task > > Components: C++ DAS > > Affects Versions: Wish list > > Reporter: Adriano Crestani > > Assigned To: Adriano Crestani > > Fix For: Wish list > > > > Attachments: tuscany1040.crestani.20070315.patch > > > > > > Implementation of BaseCommandImpl, Command, CommandImpl and > ReadCommandImpl classes as described on the DAS Lite class diagram: > http://cwiki.apache.org/confluence/pages/viewpage.action?pageId=45093 > > -- > This message is automatically generated by JIRA. > - > You can reply to this email to add a comment to the issue online. > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] >
