Hi Scott,
I looked at the download you provided and I have several comments and questions,
which is why I opened this separate DISCUSS thread to not pollute the vote
thread itself.
First of all, I assume the target for this artifact is the Maven central
repository, right?
For that purpose, I think we'll then need a 'proper' maven artifact build, which
means includes a maven project pom (dependency resolution, general project and
artifact meta-data), and preferably also a -sources.jar and -javadoc.jar
artifact for IDE support etc.
Maven based builds of course can produce such additional artifacts as a trivial
by-product, given a proper maven pom that is, but as Wookie currently is using
ivy (which I don't know nothing about) I don't know how such things are done the
ivy way.
I also noticed the binary jar has the sources embedded, which is kind of unusual
nowadays (I think). Not really wrong, but unexpected.
The LICENSE and NOTICE files are contained within the root of the jar while
these IMO should be provided under the META-INF/ folder.
I really think, as a minimum, a proper maven pom is required to even be able to
upload to the Maven repository, and such pom should provide proper dependency
resolutions. Can ivy 'generate' such a maven pom automatically?
If not, I really think the java connector module should be (also) build by maven
itself then, which also will make it much easier to 'stage' the artifact first
through repository.apache.org (Nexus) and after release automatically 'publish'
it to Maven central. If you need to do this manually I have no idea how that
(nowadays) works.
Ate Douma
As an side-note: I noticed the current wookie (main) download page only provides
the links to the binary distributions. I was kind of surprised by that, until I
found out that the sources download is provided on a separate page. Not sure how
this is viewed by others, but the common (ASF) practice is to promote the source
distributions as the primary artifacts and typically the sources and optionally
also the binary downloads are provided from the main download page.
On 10/15/2011 02:17 PM, Scott Wilson wrote:
In order to integrate Wookie and Rave we need a published non-snapshot version
of the Java Connector framework.
While we do have the 0.9.1 release coming soon, it would be good to publish the
0.9.0 version of the connector to the repository given the full 0.9.0 release
is already available, just so we can get Wookie-Rave integration into the next
Rave release.
The scope of the release is the following artifact:
http://people.apache.org/~scottbw/wookie-connector/
[ ] +1 approve
[ ] +0 no opinion
[ ] -1 disapprove
S
