+1! regards,
Martin On 9/23/05, Manfred Geiler <[EMAIL PROTECTED]> wrote: > +1 > > Manfred > > > 2005/9/23, Sean Schofield <[EMAIL PROTECTED]>: > > Apparently there is a problem with faces-config.xml in myfaces-all.jar > > of the current release. All of this confusion seems to be coming from > > the fact that sandbox is in myfaces-all.jar in the nighlty but not the > > release. We have the -Dskip.sandbox option and a bunch of other hacks > > in the build to make everything work the way it is now. > > > > I propose that we not include the sandbox stuff in the myfaces-all.jar > > anymore. I was always against this and I think the resulting > > confusion and series of hacks outweighs the argument of those that are > > lazy and don't want to include two jars in their ongoing projects. > > > > Sandbox is untested, undocumented, unvoted and unreleased code. It > > deserves its own jar with its own tld. Its already excluded from the > > release build (which I believe is correct) but the myfaces-all.jar in > > the nightly should mirror whats in the release. > > > > So the proposal is that dist-all generates a separate sandbox.jar with > > its own faces-config.xml and its own sanbox.tld. > > > > I propose we do this *before* any patch release. Also this will not > > affect SVN. It will be a build change only. > > > > sean > > > -- http://www.irian.at Your JSF powerhouse - JSF Trainings in English and German