On 23/04/2009, Shanti Subramanyam <shanti.subraman...@sun.com> wrote: > sebb wrote: > > > On 23/04/2009, Shanti Subramanyam - PAE <shanti.subraman...@sun.com> > wrote: > > > > > > > On 04/22/09 10:35, sebb wrote: > > > > > > > > > > > > > On 16/04/2009, Shanti Subramanyam - PAE <shanti.subraman...@sun.com> > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > On 04/15/09 18:15, sebb wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > On 16/04/2009, Shanti Subramanyam <shanti.subraman...@sun.com> > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Thank you very much for your very prompt review. > > > > > > > Answers to your questions below. > > > > > > > > > > > > > > Shanti > > > > > > > > > > > > > > sebb wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > The OlioDriver.jar file contains a smaller OlioDriver.jar > file. > > > > > > > > This is very confusing; one of the jars should be renamed. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > This is the format of the Faban (http://faban.sunsource.net) > Driver > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > jar > > > > > > > > > > > > > > > - > > > > > > > > > > > > > > > > > > > > > > > > > > > > the user never has to do anything with it as the tool will > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > automatically > > > > > > > > > > > > > > > > > > > > > > > > > > > > unjar and put everything in it's right place. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > The binary olio file contains several .patch and .diff files. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > These > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > don't seem correct for a binary file. What is their purpose? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > These files are part of the 3rd party plugin > fixture_replacement2. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Since we > > > > > > > > > > > > > > > > > > > > > > > > > > > > tend to include 3rd party code as is (for easier upgrade), I > think > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > it > > > > > > > > > > > > > > > might > > > > > > > > > > > > > > > > > > > > > > > > > > > > be better to leave them as they are. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > But why are they in the binary rails jar, rather than in just the > > > > > > > > > > > > > > > > > > > > > > > > source > > > > > > > > > > > > > > > jar? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > There is no difference between the source and binary code for rails > (or > > > > > > > > > > > > > > > > > php > > > > > > > > > > > > > > > for that matter) since these are interpreted languages. The only > > > > > > > > > > > > > > > > > difference > > > > > > > > > > > > > > > between the source and binary packages of Olio is the geocoder and > > > > > > > > > > > > > > > > > workload > > > > > > > > > > > > > > > - these are written in Java so the binary packages have the > > > > > > > > > > > > > > > > > OlioDriver.jar > > > > > > > > > > > > > > > and geocoder.jar and the source packages have the corresponding > source > > > > > > > > > > > > > > > > > dirs. > > > > > > > > > > That does not explain why the diffs and patch files are present in the > > > > binary jar, because as far as I can tell they are not intended to be > > > > interpreted by rails. > > > > > > > > Are they actually *needed* at run-time? > > > > > > > > > > > > > > > > > > > I really don't know. As I said, there is no concept of 'binary' for > > > scripting languages - the source is the binary. I will try and > investigate > > > this matter for the next release if it's not a major problem to leave > them > > > here for now. > > > > > > > > > > > > > I understand that there is no binary for scripting languages, and > > therefore .js and .rb files appear in both source and binary archives. > > > > However, the .diff and .patch files are scripts for a patch program, > > which is used to modify source files. Are such patches really applied > > at run-time? Seems rather wasteful to me if so. > > > > But I agree that could be fixed later. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > The source PHP file contains several jars; I would expect > these to > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > be > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > in the binary archive only. It also contains the file > "event.pdf" > > > > > > > > which does not seem to belong in the archive (or indeed in > SVN). > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > These are third-party jars. They are included as a convenience > to > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > make > > > > > > > > > > > > > > > it > > > > > > > > > > > > > > > > > > > > > > > > > > > > easier to build and run the source. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > What about the event.pdf file? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > This is a resource file used by the web application. If you notice > > > > > > > > > > > > > > > > > there > > > > > > > > > > > > > > > are several image files as well - these are all static files used by > the > > > > > > > > > > > > > > > > > web > > > > > > > > > > > > > > > app. > > > > > > > > > > > > > > > > > > > What about the license for the event.pdf file? Is that also AL > licensed? > > > > > > > > > > > > > > > > > > > All the resource files are created as part of the apache project, so > they > > > are all Apache licensed. > > > > > > > > > > According to its properties, the event.pdf file was created in 1999 > > and last modified in 2004. AFAIK, that is well before the Apache > > project started. > > The file event.jpg was created in 2006, which was also before the > > project, and is a picture of 5 real people. Hopefully they have given > > permission for their photos to be published. > > > > > > > This project was started a long time ago at Sun - all of the code including > these files was then donated to apache. > > > Neither appears to be present in the binary file, so I'm not sure how > > the web application can use them. > > > > > > > They do exist in the binary package - they are part of OlioDriver.jar.
Ah, OK, I see them now. > > Note that there are other jpg files in the same directory with much > > the same contents: > > event_thumb.jpg, person.jpg and person_thumb.jpg. > > > > AFAICT, these files don't belong in SVN or in any of the archives. > > > > > > > These files are required at run-time. The driver uses these resource files > to upload content to the web application - please see the source code under > 'workload/.../driver'. OK. > > My vote is -1 based on the above. > > > > > > > I guess I don't really understand what the objection to having these files > are. The files just seemed very odd, as if they did not relate to the project at all. I wanted to make sure that they really belonged. > If you're concerned about the photo itself (it was actually just taken > by one of the engineers), I can replace it with a picture of me although you > may actually prefer the current image :-) So long as the subjects of the picture are happy for their images to be published, then fine. > But seriously, without these > files, we will have to somehow go out and manufacture image/pdf files at > install time - uploading files is a big part of a web2.0 workload these days > (and for sites like flickr the major workload). > > Does anyone else see a serious issue with these files ? > I withdraw my -1 vote. > > > > > > > > > > > > > > > > > Shanti > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Thanks > > > > > > > > > Shanti > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On 04/15/09 09:23, Shanti Subramanyam - PAE wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > The Olio community has voted and approved this first > binary > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > release of > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Olio. We are now asking for a Vote of the Incubator PMC to > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > publish > > > > > > > > > > > > > > > this > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > release. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > The release includes both the PHP and Rails versions of > Olio. > > > > > > > > > > > > > > > > > > > > The release artificats and RAT reports are available here > : > > > > > > > > > > > http://people.apache.org/~shanti/olio_0.1/ > > > > > > > > > > > > > > > > > > > > The mail thread and Vote results from Olio community : > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > http://mail-archives.apache.org/mod_mbox/incubator-olio-dev/200904.mbox/ajax/%3c49dd6496.9010...@sun.com%3e > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Thanks > > > > > > > > > > Shanti > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > --------------------------------------------------------------------- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > To unsubscribe, e-mail: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > general-unsubscr...@incubator.apache.org > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > For additional commands, e-mail: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > general-h...@incubator.apache.org > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > --------------------------------------------------------------------- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > To unsubscribe, e-mail: > > > > > > > > > general-unsubscr...@incubator.apache.org > > > > > > > > > For additional commands, e-mail: > > > > > > > > > general-h...@incubator.apache.org > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > --------------------------------------------------------------------- > > > > > > > > > > To unsubscribe, e-mail: > > > > > > > > > > > general-unsubscr...@incubator.apache.org > > > > > > > > > > For additional commands, e-mail: > > > > > > > > > > > general-h...@incubator.apache.org > > > > > > > > > > > > > > > > > > --------------------------------------------------------------------- > > > To unsubscribe, e-mail: > > > general-unsubscr...@incubator.apache.org > > > For additional commands, e-mail: > > > general-h...@incubator.apache.org > > > > > > > > > > > > > > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: > general-unsubscr...@incubator.apache.org > > For additional commands, e-mail: > general-h...@incubator.apache.org > > > > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: > general-unsubscr...@incubator.apache.org > For additional commands, e-mail: > general-h...@incubator.apache.org > > --------------------------------------------------------------------- To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org For additional commands, e-mail: general-h...@incubator.apache.org