Tim Ellison wrote:
Geir Magnusson Jr wrote:
Tim Ellison wrote:
[ If you don't know/care about either OSGi manifests or Eclipse, then
don't worry, you can ignore this note -- just continue as you are. ]
I'm going through the module manifests and putting in some explicit
imports to satisfy the requirements of the tests within a module.
Since this means that a module will now 'import' (depend upon) a package
only because a test case uses it, I'm marking such imports in the
manifest as 'optional' -- this means that the same manifest can be used
in development and in the JARs that we deploy at runtime (which don't
contain the test code). The alternative would have been to move test
code out into a separate module but I'm NOT going there!
That makes sense.
Similarly, where a module manifest exports a package only to satisfy
somebody else's test code, it is done with a mandatory attribute to show
why it is being imported.
:/
What's the downside?
Clutter in the manifests.
It doesn't matter now, but in the future, should
we consider having the manifests assembled as part of the build based on
target? Talking out of my hat, it sounds like each breaks into two
parts, a 'manifest base' and 'manifest test' or -ish...
Component-based tools work on whole manifests, so unless we wanted to do
it all manually or roll our own then having bona fide manifests makes sense.
?
I was thinking we'd just assemble them at build time. IOW, when doing a
test build, combine the base and test fragments into the manifest, and
when doing a build for dist, leave off the test.
Downside is that you can't build/test/package in one shot, which I'll
admit is a biggie, and one I wouldn't accept either...
As above, breaking it into two parts can be done by making the impl and
tests separate components with separate manifests.
But as you noted, that sounds yecchy anyway.... or is it? it's actually
cleaner, and there is no need to re-modularize - just have a
<module>test.jar manifest along with the <module>.jar manifest?
geir
---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
