Re: [TRINIDAD] - IMPORTANT - Custom Skin Developers
We should have been quicker in turning it into a public API as soon as we realized its value for skin developers. So I think that having a short grace period would be nice for our users because it was our mistake for not making it public sooner. - Jeanne Matt Cooper wrote: Hey guys, Here's my thinking... Everything that contains "trinidadinternal" in its name is not supported as an external API. By definition it is subject to change without notice. If something is internal and valuable as a supported API we are doing the right thing by exposing it as a public API. I am quite happy that we now have a public config param to disable content compression in the skins. However, I feel that supporting the old trinidadinternal config param will set a precedent for supporting things that are not considered to be ready or designed for public consumption and potentially misuse of the system. That said, I could be convinced to have a short grace period for this parameter since I think it has been recommended for use on the mailing lists though I think we (myself included) should be careful about making recommendations to use internal things going forward. Regards, Matt On 9/7/07, Martin Marinschek <[EMAIL PROTECTED]> wrote: That's what I suggested, and Adam and Matt disagreed on. At least I thought so ;) One more hint why I think this would be appropriate: it shouldn't have been a trinidadinternal parameter in the beginning, so I'd think we shouldn't force users to take action on fixing a mistake we've been making. However, I do also see the added value of a consistent set of configuration parameters which are the same everywhere, so I stop discussing right now ;) regards, Martin On 9/8/07, Andrew Robinson < [EMAIL PROTECTED]> wrote: > Why not keep the old one and the new one and just mark the old one as > deprecated for the next release? If both are set, the new one could > take precedence. > > On 9/7/07, Martin Marinschek <[EMAIL PROTECTED]> wrote: > > Hi Matt, Adam, > > > > well, it's your call - but there are quite a few people out there who > > will have used this parameter already. Have fun answering their mails > > ;)) > > > > regards, > > > > Martin > > > > On 9/5/07, Matt Cooper < [EMAIL PROTECTED]> wrote: > > > I agree with Adam. > > > > > > > > > On 9/4/07, Adam Winer < [EMAIL PROTECTED]> wrote: > > > > The old parameter was "trinidadinternal", and therefore was > > > > never officially supported. If we were changing from one > > > > supported name to another, I'd agree that we should keep the > > > > old name around, but supporting backwards compatibility > > > > for anything "trinidadinternal" is a precedent I don't think we > > > > should set... > > > > > > > > -- Adam > > > > > > > > > > > > > > > > > > > > On 8/31/07, Martin Marinschek < [EMAIL PROTECTED]> wrote: > > > > > Hi Danny, > > > > > > > > > > eventually we could additionally support also the old name for the > > > > > configuration parameter, and only phase this out in a major release? > > > > > > > > > > regards, > > > > > > > > > > Martin > > > > > > > > > > On 8/28/07, Danny Robinson < [EMAIL PROTECTED]> wrote: > > > > > > In the hope of pre-empting some emails, be aware that the trunk has > > > switched > > > > > > to now use the following config setting for disabling skin > > > compression. > > > > > > > > > > > > > > > org.apache.myfaces.trinidadinternal.DISABLE_CONTENT_COMPRESSION > > > > > > became > > > > > > > > > org.apache.myfaces.trinidad.DISABLE_CONTENT_COMPRESSION > > > > > > > > > > > > The docs and release notes in trunk also mention this. > > > > > > > > > > > > Thanks, > > > > > > > > > > > > Danny > > > > > > -- > > > > > > Chordiant Software Inc. > > > > > > www.chordiant.com > > > > > > > > > > > > > > > -- > > > > > > > > > > http://www.irian.at > > > > > > > > > > Your JSF powerhouse - > > > > > JSF Consulting, Development and > > > > > Courses in English and German > > > > > > > > > > Professional Support for Apache MyFaces > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > http://www.irian.at > > > > Your JSF powerhouse - > > JSF Consulting, Development and > > Courses in English and German > > > > Professional Support for Apache MyFaces > > > -- http://www.irian.at Your JSF powerhouse - JSF Consulting, Development and Courses in English and German Professional Support for Apache MyFaces
Re: [TRINIDAD] - IMPORTANT - Custom Skin Developers
Hey guys, Here's my thinking... Everything that contains "trinidadinternal" in its name is not supported as an external API. By definition it is subject to change without notice. If something is internal and valuable as a supported API we are doing the right thing by exposing it as a public API. I am quite happy that we now have a public config param to disable content compression in the skins. However, I feel that supporting the old trinidadinternal config param will set a precedent for supporting things that are not considered to be ready or designed for public consumption and potentially misuse of the system. That said, I could be convinced to have a short grace period for this parameter since I think it has been recommended for use on the mailing lists though I think we (myself included) should be careful about making recommendations to use internal things going forward. Regards, Matt On 9/7/07, Martin Marinschek <[EMAIL PROTECTED]> wrote: > > That's what I suggested, and Adam and Matt disagreed on. At least I > thought so ;) > > One more hint why I think this would be appropriate: it shouldn't have > been a trinidadinternal parameter in the beginning, so I'd think we > shouldn't force users to take action on fixing a mistake we've been > making. However, I do also see the added value of a consistent set of > configuration parameters which are the same everywhere, so I stop > discussing right now ;) > > regards, > > Martin > > On 9/8/07, Andrew Robinson <[EMAIL PROTECTED]> wrote: > > Why not keep the old one and the new one and just mark the old one as > > deprecated for the next release? If both are set, the new one could > > take precedence. > > > > On 9/7/07, Martin Marinschek <[EMAIL PROTECTED]> wrote: > > > Hi Matt, Adam, > > > > > > well, it's your call - but there are quite a few people out there who > > > will have used this parameter already. Have fun answering their mails > > > ;)) > > > > > > regards, > > > > > > Martin > > > > > > On 9/5/07, Matt Cooper <[EMAIL PROTECTED]> wrote: > > > > I agree with Adam. > > > > > > > > > > > > On 9/4/07, Adam Winer <[EMAIL PROTECTED]> wrote: > > > > > The old parameter was "trinidadinternal", and therefore was > > > > > never officially supported. If we were changing from one > > > > > supported name to another, I'd agree that we should keep the > > > > > old name around, but supporting backwards compatibility > > > > > for anything "trinidadinternal" is a precedent I don't think we > > > > > should set... > > > > > > > > > > -- Adam > > > > > > > > > > > > > > > > > > > > > > > > > On 8/31/07, Martin Marinschek < [EMAIL PROTECTED]> > wrote: > > > > > > Hi Danny, > > > > > > > > > > > > eventually we could additionally support also the old name for > the > > > > > > configuration parameter, and only phase this out in a major > release? > > > > > > > > > > > > regards, > > > > > > > > > > > > Martin > > > > > > > > > > > > On 8/28/07, Danny Robinson < [EMAIL PROTECTED]> wrote: > > > > > > > In the hope of pre-empting some emails, be aware that the > trunk has > > > > switched > > > > > > > to now use the following config setting for disabling skin > > > > compression. > > > > > > > > > > > > > > > > > > org.apache.myfaces.trinidadinternal.DISABLE_CONTENT_COMPRESSION > > > > > > > became > > > > > > > > > > > org.apache.myfaces.trinidad.DISABLE_CONTENT_COMPRESSION > > > > > > > > > > > > > > The docs and release notes in trunk also mention this. > > > > > > > > > > > > > > Thanks, > > > > > > > > > > > > > > Danny > > > > > > > -- > > > > > > > Chordiant Software Inc. > > > > > > > www.chordiant.com > > > > > > > > > > > > > > > > > > -- > > > > > > > > > > > > http://www.irian.at > > > > > > > > > > > > Your JSF powerhouse - > > > > > > JSF Consulting, Development and > > > > > > Courses in English and German > > > > > > > > > > > > Professional Support for Apache MyFaces > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > http://www.irian.at > > > > > > Your JSF powerhouse - > > > JSF Consulting, Development and > > > Courses in English and German > > > > > > Professional Support for Apache MyFaces > > > > > > > > -- > > http://www.irian.at > > Your JSF powerhouse - > JSF Consulting, Development and > Courses in English and German > > Professional Support for Apache MyFaces >
Re: [TRINIDAD] - IMPORTANT - Custom Skin Developers
That's what I suggested, and Adam and Matt disagreed on. At least I thought so ;) One more hint why I think this would be appropriate: it shouldn't have been a trinidadinternal parameter in the beginning, so I'd think we shouldn't force users to take action on fixing a mistake we've been making. However, I do also see the added value of a consistent set of configuration parameters which are the same everywhere, so I stop discussing right now ;) regards, Martin On 9/8/07, Andrew Robinson <[EMAIL PROTECTED]> wrote: > Why not keep the old one and the new one and just mark the old one as > deprecated for the next release? If both are set, the new one could > take precedence. > > On 9/7/07, Martin Marinschek <[EMAIL PROTECTED]> wrote: > > Hi Matt, Adam, > > > > well, it's your call - but there are quite a few people out there who > > will have used this parameter already. Have fun answering their mails > > ;)) > > > > regards, > > > > Martin > > > > On 9/5/07, Matt Cooper <[EMAIL PROTECTED]> wrote: > > > I agree with Adam. > > > > > > > > > On 9/4/07, Adam Winer <[EMAIL PROTECTED]> wrote: > > > > The old parameter was "trinidadinternal", and therefore was > > > > never officially supported. If we were changing from one > > > > supported name to another, I'd agree that we should keep the > > > > old name around, but supporting backwards compatibility > > > > for anything "trinidadinternal" is a precedent I don't think we > > > > should set... > > > > > > > > -- Adam > > > > > > > > > > > > > > > > > > > > On 8/31/07, Martin Marinschek < [EMAIL PROTECTED]> wrote: > > > > > Hi Danny, > > > > > > > > > > eventually we could additionally support also the old name for the > > > > > configuration parameter, and only phase this out in a major release? > > > > > > > > > > regards, > > > > > > > > > > Martin > > > > > > > > > > On 8/28/07, Danny Robinson < [EMAIL PROTECTED]> wrote: > > > > > > In the hope of pre-empting some emails, be aware that the trunk has > > > switched > > > > > > to now use the following config setting for disabling skin > > > compression. > > > > > > > > > > > > > > > org.apache.myfaces.trinidadinternal.DISABLE_CONTENT_COMPRESSION > > > > > > became > > > > > > > > > org.apache.myfaces.trinidad.DISABLE_CONTENT_COMPRESSION > > > > > > > > > > > > The docs and release notes in trunk also mention this. > > > > > > > > > > > > Thanks, > > > > > > > > > > > > Danny > > > > > > -- > > > > > > Chordiant Software Inc. > > > > > > www.chordiant.com > > > > > > > > > > > > > > > -- > > > > > > > > > > http://www.irian.at > > > > > > > > > > Your JSF powerhouse - > > > > > JSF Consulting, Development and > > > > > Courses in English and German > > > > > > > > > > Professional Support for Apache MyFaces > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > http://www.irian.at > > > > Your JSF powerhouse - > > JSF Consulting, Development and > > Courses in English and German > > > > Professional Support for Apache MyFaces > > > -- http://www.irian.at Your JSF powerhouse - JSF Consulting, Development and Courses in English and German Professional Support for Apache MyFaces
Re: [TRINIDAD] - IMPORTANT - Custom Skin Developers
Why not keep the old one and the new one and just mark the old one as deprecated for the next release? If both are set, the new one could take precedence. On 9/7/07, Martin Marinschek <[EMAIL PROTECTED]> wrote: > Hi Matt, Adam, > > well, it's your call - but there are quite a few people out there who > will have used this parameter already. Have fun answering their mails > ;)) > > regards, > > Martin > > On 9/5/07, Matt Cooper <[EMAIL PROTECTED]> wrote: > > I agree with Adam. > > > > > > On 9/4/07, Adam Winer <[EMAIL PROTECTED]> wrote: > > > The old parameter was "trinidadinternal", and therefore was > > > never officially supported. If we were changing from one > > > supported name to another, I'd agree that we should keep the > > > old name around, but supporting backwards compatibility > > > for anything "trinidadinternal" is a precedent I don't think we > > > should set... > > > > > > -- Adam > > > > > > > > > > > > > > > On 8/31/07, Martin Marinschek < [EMAIL PROTECTED]> wrote: > > > > Hi Danny, > > > > > > > > eventually we could additionally support also the old name for the > > > > configuration parameter, and only phase this out in a major release? > > > > > > > > regards, > > > > > > > > Martin > > > > > > > > On 8/28/07, Danny Robinson < [EMAIL PROTECTED]> wrote: > > > > > In the hope of pre-empting some emails, be aware that the trunk has > > switched > > > > > to now use the following config setting for disabling skin > > compression. > > > > > > > > > > > > org.apache.myfaces.trinidadinternal.DISABLE_CONTENT_COMPRESSION > > > > > became > > > > > > > org.apache.myfaces.trinidad.DISABLE_CONTENT_COMPRESSION > > > > > > > > > > The docs and release notes in trunk also mention this. > > > > > > > > > > Thanks, > > > > > > > > > > Danny > > > > > -- > > > > > Chordiant Software Inc. > > > > > www.chordiant.com > > > > > > > > > > > > -- > > > > > > > > http://www.irian.at > > > > > > > > Your JSF powerhouse - > > > > JSF Consulting, Development and > > > > Courses in English and German > > > > > > > > Professional Support for Apache MyFaces > > > > > > > > > > > > > > > > > -- > > http://www.irian.at > > Your JSF powerhouse - > JSF Consulting, Development and > Courses in English and German > > Professional Support for Apache MyFaces >
Re: [TRINIDAD] - IMPORTANT - Custom Skin Developers
Hi Matt, Adam, well, it's your call - but there are quite a few people out there who will have used this parameter already. Have fun answering their mails ;)) regards, Martin On 9/5/07, Matt Cooper <[EMAIL PROTECTED]> wrote: > I agree with Adam. > > > On 9/4/07, Adam Winer <[EMAIL PROTECTED]> wrote: > > The old parameter was "trinidadinternal", and therefore was > > never officially supported. If we were changing from one > > supported name to another, I'd agree that we should keep the > > old name around, but supporting backwards compatibility > > for anything "trinidadinternal" is a precedent I don't think we > > should set... > > > > -- Adam > > > > > > > > > > On 8/31/07, Martin Marinschek < [EMAIL PROTECTED]> wrote: > > > Hi Danny, > > > > > > eventually we could additionally support also the old name for the > > > configuration parameter, and only phase this out in a major release? > > > > > > regards, > > > > > > Martin > > > > > > On 8/28/07, Danny Robinson < [EMAIL PROTECTED]> wrote: > > > > In the hope of pre-empting some emails, be aware that the trunk has > switched > > > > to now use the following config setting for disabling skin > compression. > > > > > > > > > org.apache.myfaces.trinidadinternal.DISABLE_CONTENT_COMPRESSION > > > > became > > > > > org.apache.myfaces.trinidad.DISABLE_CONTENT_COMPRESSION > > > > > > > > The docs and release notes in trunk also mention this. > > > > > > > > Thanks, > > > > > > > > Danny > > > > -- > > > > Chordiant Software Inc. > > > > www.chordiant.com > > > > > > > > > -- > > > > > > http://www.irian.at > > > > > > Your JSF powerhouse - > > > JSF Consulting, Development and > > > Courses in English and German > > > > > > Professional Support for Apache MyFaces > > > > > > > > > -- http://www.irian.at Your JSF powerhouse - JSF Consulting, Development and Courses in English and German Professional Support for Apache MyFaces
Re: [TRINIDAD] - IMPORTANT - Custom Skin Developers
I agree with Adam. On 9/4/07, Adam Winer <[EMAIL PROTECTED]> wrote: > > The old parameter was "trinidadinternal", and therefore was > never officially supported. If we were changing from one > supported name to another, I'd agree that we should keep the > old name around, but supporting backwards compatibility > for anything "trinidadinternal" is a precedent I don't think we > should set... > > -- Adam > > > On 8/31/07, Martin Marinschek < [EMAIL PROTECTED]> wrote: > > > > Hi Danny, > > > > eventually we could additionally support also the old name for the > > configuration parameter, and only phase this out in a major release? > > > > regards, > > > > Martin > > > > On 8/28/07, Danny Robinson < [EMAIL PROTECTED]> wrote: > > > In the hope of pre-empting some emails, be aware that the trunk has > > switched > > > to now use the following config setting for disabling skin > > compression. > > > > > > org.apache.myfaces.trinidadinternal.DISABLE_CONTENT_COMPRESSION > > > became > > > org.apache.myfaces.trinidad.DISABLE_CONTENT_COMPRESSION > > > > > > The docs and release notes in trunk also mention this. > > > > > > Thanks, > > > > > > Danny > > > -- > > > Chordiant Software Inc. > > > www.chordiant.com > > > > > > -- > > > > http://www.irian.at > > > > Your JSF powerhouse - > > JSF Consulting, Development and > > Courses in English and German > > > > Professional Support for Apache MyFaces > > > >
Re: [TRINIDAD] - IMPORTANT - Custom Skin Developers
The old parameter was "trinidadinternal", and therefore was never officially supported. If we were changing from one supported name to another, I'd agree that we should keep the old name around, but supporting backwards compatibility for anything "trinidadinternal" is a precedent I don't think we should set... -- Adam On 8/31/07, Martin Marinschek <[EMAIL PROTECTED]> wrote: > > Hi Danny, > > eventually we could additionally support also the old name for the > configuration parameter, and only phase this out in a major release? > > regards, > > Martin > > On 8/28/07, Danny Robinson <[EMAIL PROTECTED]> wrote: > > In the hope of pre-empting some emails, be aware that the trunk has > switched > > to now use the following config setting for disabling skin compression. > > > > org.apache.myfaces.trinidadinternal.DISABLE_CONTENT_COMPRESSION > > became > > org.apache.myfaces.trinidad.DISABLE_CONTENT_COMPRESSION > > > > The docs and release notes in trunk also mention this. > > > > Thanks, > > > > Danny > > -- > > Chordiant Software Inc. > > www.chordiant.com > > > -- > > http://www.irian.at > > Your JSF powerhouse - > JSF Consulting, Development and > Courses in English and German > > Professional Support for Apache MyFaces >
Re: [TRINIDAD] - IMPORTANT - Custom Skin Developers
Hi Danny, eventually we could additionally support also the old name for the configuration parameter, and only phase this out in a major release? regards, Martin On 8/28/07, Danny Robinson <[EMAIL PROTECTED]> wrote: > In the hope of pre-empting some emails, be aware that the trunk has switched > to now use the following config setting for disabling skin compression. > > org.apache.myfaces.trinidadinternal.DISABLE_CONTENT_COMPRESSION > became > org.apache.myfaces.trinidad.DISABLE_CONTENT_COMPRESSION > > The docs and release notes in trunk also mention this. > > Thanks, > > Danny > -- > Chordiant Software Inc. > www.chordiant.com -- http://www.irian.at Your JSF powerhouse - JSF Consulting, Development and Courses in English and German Professional Support for Apache MyFaces
[TRINIDAD] - IMPORTANT - Custom Skin Developers
In the hope of pre-empting some emails, be aware that the trunk has switched to now use the following config setting for disabling skin compression. org.apache.myfaces.trinidadinternal.DISABLE_CONTENT_COMPRESSION became org.apache.myfaces.trinidad.DISABLE_CONTENT_COMPRESSION The docs and release notes in trunk also mention this. Thanks, Danny -- Chordiant Software Inc. www.chordiant.com