+1
2005/9/23, Bill Dudney <[EMAIL PROTECTED]>: > Agreed, > > -bd- > > On Sep 23, 2005, at 11:20 AM, Sean Schofield wrote: > > > 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 > > > >
