Al thx for your investigation... you're welcome to fix anything in the
release branch. :)

Martijn

On 3/28/07, Al Maw <[EMAIL PROTECTED]> wrote:
Martijn Dashorst wrote:
> I've tried to remove some issues from our previous round. The less
> issues to cope with, the better.
> - just one target directory still eludes me, the rest have been eliminated
> - jetty jars are now provided, and not put in the examples war
> - build artifacts now go into target/dist and target/m2-repo (making
> the build process self contained and more platform independent)
>
> So another round, another vote.
>
> [ ] Yes release the damned thing already
> [ ] Yes, this is the last time I vote on this... My + and 1 keys show
> signs of wear
> [ ] No, I found this issue and you'd better fix it, or ...

Yeah, +1 to release it to the incubator folk.

I can't verify the ASC sigs, 'cos I don't have your public PGP key.

Other than that, things I've checked are:
  - Tests all pass.
  - mvn install works with nothing in local repo already.
  - SHA1 and MD5 sums are OK.
  - Tarball and ZIP have exactly the same contents.
  - The links in the README, etc. text files work/exist.

I might write a script to automate as much of the above as possible.

Things I haven't checked:
  - That the JAR files in /lib all work.
  - That the wicket-examples WAR works.

Obvious those are quite important. :-)
If someone else can verify those, then I'm +1 on the rest of it.


Some minor quibbles, none of which stand in the way:

  - There's a src/dist folder that's empty.

  - I'm not sure we need the release.sh script in the actual zip/tgz.

  - It'd be nice to make the eclipse.sh script executable (for the
    tar.gz, at least). You can do that with a <fileMode>0755</fileMode>
    element in a <fileSet> element in the assembly descriptor, but it's
    hardly a big deal.

  - I think we should move the EclipseCodeFormat.xml from wicket to
    the top-level parent folder.

  - While checking all this, I noticed that the JIRA link on
    http://incubator.apache.org/projects/wicket.html doesn't work (it
    needs an http:// on the front).


Thanks Martijn!

Al



--
Learn Wicket at ApacheCon Europe: http://apachecon.com
Join the wicket community at irc.freenode.net: ##wicket
Wicket 1.2.5 will keep your server alive. Download Wicket now!
http://wicketframework.org

Reply via email to