You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lenya.apache.org by Andreas Hartmann <an...@apache.org> on 2010/07/11 23:50:16 UTC

Re: svn commit: r962487 - /lenya/branches/BRANCH_2_1_X/externals/

Hi Richard,

Am 09.07.10 13:46, schrieb rfrovarp@apache.org:
> Author: rfrovarp
> Date: Fri Jul  9 11:46:48 2010
> New Revision: 962487
>
> URL: http://svn.apache.org/viewvc?rev=962487&view=rev
> Log:
> Getting untranslated errors again.
> Will investigate once back in the states.

the reason for the untranslated error is that the IncludeTransformer in 
Cocoon 2.1.11 doesn't yet support the strip-root attribute which 
resulted in an invalid catalogue XML. I just committed a workaround 
since I didn't do an SVN update after your change and noticed too late 
that this was due to the different Cocoon version. I guess I should 
revert my change now.

BTW, the missing support for the strip-root attribute also causes errors 
with the news module. So I think we should stick with the 2.1.X branch 
of Cocoon instead of the 2.1.11 release.

WDYT?

-- Andreas




>
> Modified:
>      lenya/branches/BRANCH_2_1_X/externals/   (props changed)
>
> Propchange: lenya/branches/BRANCH_2_1_X/externals/
> ------------------------------------------------------------------------------
> --- svn:externals (original)
> +++ svn:externals Fri Jul  9 11:46:48 2010
> @@ -1 +1 @@
> -cocoon_2_1_x http://svn.apache.org/repos/asf/cocoon/tags/cocoon-2.1/RELEASE_2_1_11
> +cocoon_2_1_x http://svn.apache.org/repos/asf/cocoon/branches/BRANCH_2_1_X


-- 
Andreas Hartmann, CTO
BeCompany GmbH
http://www.becompany.ch
Tel.: +41 (0) 43 818 57 01


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lenya.apache.org
For additional commands, e-mail: dev-help@lenya.apache.org


Re: Cocoon branch or release?

Posted by Richard Frovarp <rf...@apache.org>.
On 07/13/2010 08:06 AM, Andreas Hartmann wrote:
> Am 13.07.10 13:04, schrieb Richard Frovarp:
>> On 07/11/2010 04:50 PM, Andreas Hartmann wrote:
>>> Hi Richard,
>>>
>>> Am 09.07.10 13:46, schrieb rfrovarp@apache.org:
>>>> Author: rfrovarp
>>>> Date: Fri Jul 9 11:46:48 2010
>>>> New Revision: 962487
>>>>
>>>> URL: http://svn.apache.org/viewvc?rev=962487&view=rev
>>>> Log:
>>>> Getting untranslated errors again.
>>>> Will investigate once back in the states.
>>>
>>> the reason for the untranslated error is that the IncludeTransformer
>>> in Cocoon 2.1.11 doesn't yet support the strip-root attribute which
>>> resulted in an invalid catalogue XML. I just committed a workaround
>>> since I didn't do an SVN update after your change and noticed too late
>>> that this was due to the different Cocoon version. I guess I should
>>> revert my change now.
>>>
>>> BTW, the missing support for the strip-root attribute also causes
>>> errors with the news module. So I think we should stick with the 2.1.X
>>> branch of Cocoon instead of the 2.1.11 release.
>>>
>> I would like to stick with Cocoon 2.1.11. However, I'm taking it that
>> your workaround won't fix the news module as well?
>
> This is correct. I started adding the necessary workarounds for the 
> news module, but this was not as trivial as with the i18n module.
>
> WDOT? Should we use Cocoon 2.1.11 or the current 2.1.X branch or a 
> particular revision of the 2.1.X branch?
>
> -- Andreas
>
>

If we can't do 2.1.11, I would have to go with a particular revision of 
2.1.x. Fix it like we have for releases to the current one. Any track 
the Cocoon 2.1.x updates enough to know when we would need to update? 
Are they thinking about a 2.1.12 release?

Richard


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lenya.apache.org
For additional commands, e-mail: dev-help@lenya.apache.org


Cocoon branch or release? (was: Re: svn commit: r962487 - /lenya/branches/BRANCH_2_1_X/externals/)

Posted by Andreas Hartmann <an...@apache.org>.
Am 13.07.10 13:04, schrieb Richard Frovarp:
> On 07/11/2010 04:50 PM, Andreas Hartmann wrote:
>> Hi Richard,
>>
>> Am 09.07.10 13:46, schrieb rfrovarp@apache.org:
>>> Author: rfrovarp
>>> Date: Fri Jul 9 11:46:48 2010
>>> New Revision: 962487
>>>
>>> URL: http://svn.apache.org/viewvc?rev=962487&view=rev
>>> Log:
>>> Getting untranslated errors again.
>>> Will investigate once back in the states.
>>
>> the reason for the untranslated error is that the IncludeTransformer
>> in Cocoon 2.1.11 doesn't yet support the strip-root attribute which
>> resulted in an invalid catalogue XML. I just committed a workaround
>> since I didn't do an SVN update after your change and noticed too late
>> that this was due to the different Cocoon version. I guess I should
>> revert my change now.
>>
>> BTW, the missing support for the strip-root attribute also causes
>> errors with the news module. So I think we should stick with the 2.1.X
>> branch of Cocoon instead of the 2.1.11 release.
>>
> I would like to stick with Cocoon 2.1.11. However, I'm taking it that
> your workaround won't fix the news module as well?

This is correct. I started adding the necessary workarounds for the news 
module, but this was not as trivial as with the i18n module.

WDOT? Should we use Cocoon 2.1.11 or the current 2.1.X branch or a 
particular revision of the 2.1.X branch?

-- Andreas


-- 
Andreas Hartmann, CTO
BeCompany GmbH
http://www.becompany.ch
Tel.: +41 (0) 43 818 57 01


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lenya.apache.org
For additional commands, e-mail: dev-help@lenya.apache.org


Re: svn commit: r962487 - /lenya/branches/BRANCH_2_1_X/externals/

Posted by Richard Frovarp <rf...@apache.org>.
On 07/11/2010 04:50 PM, Andreas Hartmann wrote:
> Hi Richard,
>
> Am 09.07.10 13:46, schrieb rfrovarp@apache.org:
>> Author: rfrovarp
>> Date: Fri Jul  9 11:46:48 2010
>> New Revision: 962487
>>
>> URL: http://svn.apache.org/viewvc?rev=962487&view=rev
>> Log:
>> Getting untranslated errors again.
>> Will investigate once back in the states.
>
> the reason for the untranslated error is that the IncludeTransformer 
> in Cocoon 2.1.11 doesn't yet support the strip-root attribute which 
> resulted in an invalid catalogue XML. I just committed a workaround 
> since I didn't do an SVN update after your change and noticed too late 
> that this was due to the different Cocoon version. I guess I should 
> revert my change now.
>
> BTW, the missing support for the strip-root attribute also causes 
> errors with the news module. So I think we should stick with the 2.1.X 
> branch of Cocoon instead of the 2.1.11 release.
>
> WDYT?
>
> -- Andreas

I would like to stick with Cocoon 2.1.11. However, I'm taking it that 
your workaround won't fix the news module as well?


Richard

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lenya.apache.org
For additional commands, e-mail: dev-help@lenya.apache.org