Community
Participate
Working Groups
We need to update the demo install script to output a message to the console that it is for Tomcat only and that usage of another web server would require manual installation of the demo. (I decided to break this off from bug 252368 since it requires different skills and can be done in parallel) This would require a rebuild of the candidate driver, but I think this is important enough to do since we don't claim that Tomcat is a prereq.
The InstallGuide.txt needs to be updated at the same time to refer to the correct location of the Installation Guide. Should probably rename the file as readme.txt and include the text "Please refer to the COSMOS Installation Guide for configuration instructions, available on the COSMOS downloads page: http://www.eclipse.org/cosmos/downloads". Remember that this change must be made on both the HEAD stream and the cosmos 1.0 branch.
Created attachment 116406 [details] readme.txt
Created attachment 116407 [details] configDemo.sh
Created attachment 116408 [details] configDemo.bat
Created attachment 116409 [details] DemoConfig.java
David, I think its better to add a method to DemoConfig.java with SOP statement for the Tomcat info, instead of updating configDemo.bat and configDemo.sh with comments.I have renamed the InstallGuide.txt as readme.txt with appropriate content as mentioned below in your comment. Please find attached DemoConfig.java,readme.txt,configDemo.bat,configDemo.sh. Please checkin either DemoConfig.java or configDemo.bat,configDemo.sh which ever seems appropriate.
(In reply to comment #6) > David, > > I think its better to add a method to DemoConfig.java with SOP statement for > the Tomcat info, instead of updating configDemo.bat and configDemo.sh with > comments. I agree. I accepted the changes to DemoConfig.java. Nice work! Don't forget the following two things when you make fixes in the future: * Use Team > Create Patch... to make a patch file that a committer can apply to their workspace, instead of attaching multiple files to the defect * Add CA to the copyright whenever you change a file that hasn't previously been modified by CA. Checked into CVS HEAD and Cosmos 1.0 branch.
Verified on the latest build - COSMOS-1.0.0-200811041640, it's found to be fixed and working fine.
