On May 11, 2009, at 4:29 PM, Ate Douma wrote:
David Jencks wrote:
On May 11, 2009, at 3:42 PM, Ate Douma wrote:
<snip/>
The release plan I have come up to be in alignment with the
(refreshed) ASF release requirements is the following:
1) first do a Nexus release to a staged repository (this will
create the release tags for both projects)
2) do a manual checkout of the release tag src and build src
dist archives myself, sign them and upload to a public folder at
people.apache.org, like people.apache.org/~ate/portlet_spec/ [...]
or
for instance use the maven assembly plugin doing (all) the above
automatically???
I actually have never needed to use the assembly plugin yet so
I'm not sure what/how much it can do...
Maybe David or Carsten know more about it, but if I have to find
out myself, I might just resort back to doing it manual this time
3) send out a VOTE email concerning *both* the Nexus provided
artifacts from the temporary staged repository *and* the
uploaded src dist archives
4) .... process the VOTE results and (hopefully) have the
release completed.
IMO this is too complicated to be workable.
Well, at least it should work as we already know now Nexus works
fine, and the manual bit is just the same as how we've done it in
the past.
I think we should add the assembly plugin to each top level pom
to build the source distro, and continue using the release plugin
for the entire release process. IIUC this approach will get into
the next apache pom with some additional automation. I'll see if
I can come up with appropriate assembly plugin configuration.
Sure, if you find a way to further automate this, I'm all for it,
as long as it works :)
What is unclear to me however, how/where this source distro is
going to be "published" to the maven repository.
I assume that requires an additional artifact (e.g. pom.xml)
definition, right?
I set this up. IIUC the new project distro artifacts will be
deployed along with all the other artifacts when we run mvn deploy -
Papache-release, for instance when the release:perform goal does
this.
Cool, thanks David, I will try this out right away (locally of
course).
My original commit might not work, there seems to be a problem in the
assembly plugin 2.2-beta-3. I switched to 2.2-beta-2 which does seem
to work.
As noted in another thread I think the changes we'd like in portals-
pom are piling up and we should get them all in and release portals-
pom 1.2 and the specs in one vote (or simultaneous votes). I'd
move the pluginManagement assembly plugin configuration to the
parent.
OK, +1
Just as David Taylor just asked on another thread, can you explain
how you want the ianal plugin to be run on all builds?
just include it in the <build><plugins> element in the portals-pom.
To re-answer David's question, it checks that the required legal files
are included in every artifact.
If I may ask, would you be willing to do the required changed for
the portals-pom-1.2-SNAPSHOT so we can all assess what these changes
will amount to?
I did this and also updated the spec poms to use the snapshot parent.
rev 773745 in case you want to revert it.
Only part not discussed previously is that I renamed prepare-release
profile to rat. I don't know what problems occurred when running it
as apache-release, but I don't think people usually include rat in the
release profile. I'm not sure why or why not.... I'm fine with asking
RMs to run -Prat before release:prepare.
thanks
david jencks
Anything I could be missing in the above plan?
- checking that pluto and jetspeed work with our spec jars
Sure, we can (and should) do that, but I have no doubt it will as
we didn't change a single bit within the code base itself.
Its just a change of dependency IMO, nothing more.
I'd still like to see it working before voting :-)
I'm already working on it :)
I'm in the process of updating Pluto trunk to use the new
dependencies and will test both manually and run the JSR-286 TCK
against it.
If that works well, I'll repeat it for Jetspeed-2.
And if all passes, I'll commit the updates for these dependencies.
The area where we did make changes, the OSGi bundle configuration,
won't affect Pluto or Jetspeed-2 as they don't use that information.
- checking that our osgi metadata is accurate enough to work.
That I have to leave up to you and especially Carsten to validate.
I know Carsten is using OSGi a lot, so I presume he should be able
to quickly test this out.
For the the usage of these spec jars within the Apache Portals
projects, the dependency replacement is what matters to me the
most for this initial release. If the OSGi bundle configuration
turns out to need more tweaking in the future, a follow up release
is easily done.
agreed.
thanks
david jencks
Regards,
Ate
thanks
david jencks
Regards,
Ate
David Jencks wrote:
On May 11, 2009, at 11:20 AM, Carsten Ziegeler wrote:
Ate Douma wrote:
David Jencks wrote:
If we can get it to produce slightly more reasonable output
by the end
of tomorrow I think we should use it. For instance it has
pointed out
that the packages in Export-Packages should have uses
clauses, at
least for the servlet package needed. If no answer from
feiix arrives
by tomorrow then I agree we should use the manual manifest
setup.....
but informed by the bundle plugin output.
Sounds good to me.
I hope Carsten can chime in on this tomorrow too as he's the
one
bringing in the OSGi configuration in the first place ;)
I'm slowly recovering from switching to my new laptop :)
Ok I think we don't need to use the maven bundle plugin as the
manifest
headers are constant in our case; but there is nothing wrong
with using
the plugin, so we can leave it as is. But in this case, I
suggest to
remove the import package statement and change the export
statement to just:
<Export-Package>javax.portlet.filter;version=2.0.0,
javax.portlet;version=2.0.0
</Export-Package>
The plugin is doing the rest (adding the uses, creating the
correct
import etc.)
OK.... I had to include some info on imports in order to get
the servlet version to be 2.4, the sun api jar isn't a bundle
with package versions. I simplified the poms as much as I
could...
many thanks
david jencks
Carsten
--
Carsten Ziegeler
[email protected]
