Re: [Trinidad2] InternalView VDL/TRINIDAD-1735

2010-03-02 Thread Max Starets
Oops - thanks for noticing this, Matthias. I copied the license header from the wrong file (was working on a Mojarra patch at the same time ;-) ). I will correct it right now. Max Matthias Wessendorf wrote: here is some background on not allowed licenses:

Re: [Trinidad2] InternalView VDL/TRINIDAD-1735

2010-03-02 Thread Max Starets
Matthias, And one more thing - no Sun RI code was copied. Max Matthias Wessendorf wrote: -1 on the patch. We really can't use license header from LGPL2code...: -InternalViewHandlingStrategy -ViewDeclarationLanguageFactoryImpl Nor can't we copy code from the SUN RI to Apache. -Matthias

Re: [Trinidad2] InternalView VDL/TRINIDAD-1735

2010-03-02 Thread Matthias Wessendorf
that would qualify the patch as -1 On Tue, Mar 2, 2010 at 4:14 PM, Max Starets max.star...@oracle.com wrote: Matthias, And one more thing - no Sun RI code was copied. Max Matthias Wessendorf wrote: -1 on the patch. We really can't use license header from LGPL2code...:

Re: [Trinidad2] InternalView VDL/TRINIDAD-1735

2010-03-02 Thread Matthias Wessendorf
which means we are good to go, since nothing was copied ;-) +1 On Tue, Mar 2, 2010 at 4:16 PM, Matthias Wessendorf mat...@apache.org wrote: that would qualify the patch as -1 On Tue, Mar 2, 2010 at 4:14 PM, Max Starets max.star...@oracle.com wrote: Matthias, And one more thing - no Sun RI

[Trinidad2] InternalView VDL/TRINIDAD-1735

2010-03-01 Thread Max Starets
Hello Everyone, While I was working on a solution for TRINIDAD-1735, I realized that the clean solution would involve moving all the code dealing with InternalViews from the ViewHandlerImpl to the new ViewDeclarationLanguageFactoryImpl. We also would expose a special VDL for the internal

Re: [Trinidad2] InternalView VDL/TRINIDAD-1735

2010-03-01 Thread Gary Kind
Will this affect the Controller team? Gary Kind Max Starets wrote: Hello Everyone, While I was working on a solution for TRINIDAD-1735, I realized that the clean solution would involve moving all the code dealing with InternalViews from the ViewHandlerImpl to the new

Re: [Trinidad2] InternalView VDL/TRINIDAD-1735

2010-03-01 Thread Max Starets
Gary, No, this just sn internal reorganization, and there will be no API changes. Max On Mar 1, 2010, at 7:04 PM, Gary Kind gary.k...@oracle.com wrote: Will this affect the Controller team? Gary Kind Max Starets wrote: Hello Everyone, While I was working on a solution for

Re: [Trinidad2] InternalView VDL/TRINIDAD-1735

2010-03-01 Thread Matthias Wessendorf
-1 on the patch. We really can't use license header from LGPL2code...: -InternalViewHandlingStrategy -ViewDeclarationLanguageFactoryImpl Nor can't we copy code from the SUN RI to Apache. -Matthias On Tue, Mar 2, 2010 at 1:40 AM, Max Starets max.star...@oracle.com wrote: Gary, No, this just

Re: [Trinidad2] InternalView VDL/TRINIDAD-1735

2010-03-01 Thread Matthias Wessendorf
here is some background on not allowed licenses: http://www.apache.org/legal/resolved.html#category-x -Matthias On Tue, Mar 2, 2010 at 7:19 AM, Matthias Wessendorf mat...@apache.org wrote: -1 on the patch. We really can't use license header from LGPL2code...: -InternalViewHandlingStrategy