Hi Albert comments inline. 2009/3/6 Albert Lee <[email protected]>
> Mike, just a few comments > > 1) In the release note, the " Release Notes - OpenJPA - Version 1.2.1." > should be hightlighted. We can do this in future releases. I'm not sure it's worth re-spinning just for a formatting change. > 2) The link below can remove the suffix "&view=markup" so that it goes > right > to display the content. +1, the revision was just a copy from the link in viewcvs. A better link is here : http://svn.apache.org/viewvc/openjpa/tags/1.2.1/openjpa-project/RELEASE-NOTES.html?view=markup <http://svn.apache.org/viewvc/openjpa/tags/1.2.1/openjpa-project/RELEASE-NOTES.html?revision=749830&view=markup> > 3) The binary.zip is 18 Mb, which is 7 Mb less than the version of the > previous release. Looks like the javadoc and source java files are removed > from the openjpa.121.jar in the binary zip file. That's intentional see OPENJPA-713. We could include the sources jar and javadoc as separate artifacts in *binary.zip if the community thinks it's useful. Maybe it's worth creating a javadoc zip as a separate download? 4) The doc links to the TOC of the manul, I wonder if we can link to the > framed version like > http://openjpa.apache.org/builds/latest/docs/manual/main.html? There is no main.html for 1.2.1 at the moment (or it didn't get built I'll look into it). Thanks for doing the release. > Albert Lee. > > 2009/3/6 Michael Dick <[email protected]> > > > Hi Milosz, > > > > I messed up and didn't put the right revision number in my link. Try this > > one : > > > > > > > http://svn.apache.org/viewvc/openjpa/tags/1.2.1/openjpa-project/RELEASE-NOTES.html?revision=749830&view=markup > > > > Thanks for catching it. > > > > -mike > > > > On Fri, Mar 6, 2009 at 5:21 AM, Miłosz Tylenda <[email protected]> wrote: > > > > > Mike, > > > > > > Are the release notes, you gave a link to, the final version? Its > > contents > > > are from 1.1.0 release. On the other hand, the release notes in the > > binary > > > distribution seem to be up to date. > > > > > > Greetings, > > > Milosz > > > > > > > > > > > > > > > -- > Albert Lee. >
