Sylvain Wallez wrote:
Nicola Ken Barozzi wrote:
Sylvain Wallez wrote:

[...]

So I propose that we _remove all the conditional compilation stuff_ and write mock classes where needed to ensure that _all_ source files are always compiled.
[...]

Thoughts ?
I agree. +1

In fact I added the mocks dir in the blocks to be able to set them there. IMHO that's all that is should take, because all optional stuff is to be in blocks, and in the near future in "/environment/**".
This means of course that we remove all conditional compileation as you say.
I also had to add a mocks directory in scratchpad and in src (needed for sun.tools.javac.Main, and JaxenProcessorImpl for which I didn't want to make a new block).
Ah, ok, gotcha.

What is needed in your opinion to make it fully? I can help you make adjustments if necessary :-)
I think I did the main part of the work : add mock classes where needed and tweak the build to handle them. It's also working in the 2.0 branch.

The remaining work will be some cleanup in the build.xml files : I admit I don't clearly understand all the details of block build :-/
Ask, and it will be replied :-)
If you put in the mocks, I will cleanup that file. So each does what he understands best :-)

Basically what it does is to get the dependencies from the project-info.xml file (the Gump descriptor), resolves dependencies, and builds the blocks one after the other, buy invoking the same build on each in turn.

Try this: do a build and look in build/cocoon/work/blocks-build.xml
That's the generated buildfile that compiles all the blocks.

One more proposal, also : what JDK are using Cocoon devs ? I personally use 1.4, and so would like to make JDBC 3.0 methods uncommented by default (so that the IDE is happy) and let the build comment them out if needed.


--
Nicola Ken Barozzi                   [EMAIL PROTECTED]
            - verba volant, scripta manent -
   (discussions get forgotten, just code remains)
---------------------------------------------------------------------


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, email: [EMAIL PROTECTED]

Reply via email to