forrest-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Unico Hommes" <Un...@hippo.nl>
Subject RE: LocatioMapModule (was: RE: [VOTE RESULTS] Unico Hommes as a Cocoon/Forrest Committer)
Date Tue, 21 Oct 2003 09:37:17 GMT


Nicola Ken Barozzi wrote:
> 
> 
> Unico Hommes wrote:
> 
> > Nicola Ken Barozzi wrote:
> ...
> >>Yup, when you get access, don't hesitate to commit it as-is in the 
> >>src/java/** dir. 
> 
> Couldn't wait, did it myself ;-)
>

:-)

By the way I just did CVS update and tried to build Forrest but failed. It complained about
some resources not being there during validation.

...

validate-config:
validating **/skinconf.xml ...

BUILD FAILED
C:\eclipse\workspace\xml-forrest\build.xml:202: C:\eclipse\workspace\xml-forrest\src\resources
n
ot found.


> > Great, when we have finished stabalizing it I'll want to 
> move it over to cocoon CVS if noone objects to it.
> 
> I wouldn't even plan for that ATM. The reason is that code 
> must reside 
> where the community is. Look what happened when we started 
> moving Cocoon 
> things in Avalon, they basically started decaying. Moreover, Cocoon 
> committers can commit here too, so it's not an issue of karma.
> 
> I propose we discuss this again when we have real blocks.
> 

OK although less convenient for me, I see a point so I won't insist, for now.


> >
> > It looks like we may need to consider doing:
> > 
> > <select>
> >   <when test="..">
> >     <match pattern="">
> >       <location src="" />
> >     </match>
> >     <location src=".." />
> >   </when>
> > </select>
> > 
> > Which sacrafices brevity :-( Any ideas?
> 
> Hmmm, in this case:
> 
>   <select>
>     <location test=".." src=".." />
>   </select>
> 
> We are using the @test attribute on the element that should 
> be inside a 
> when like this.
> 
>   <select>
>     <when test="..">
>       <location src=".." />
>     </when>
>   </select>
> 
> In the match case, it's the same:
> 
>   <select>
>     <when test="..">
>       <match pattern="">
>         <location src="" />
>       </match>
>       <location src=".." />
>     </when>
>   </select>
> 
> Becomes:
> 
>   <select>
>     <match pattern="" test="..">
>       <location src="" />
>     </match>
>     <location src=".." />
>   </select>
> 
> Just remove the <when> tag and add the @test attribute to the element 
> inside it.
> 

Yes, that looks simple enough. My concern was that putting the test attribute on matchers
and selectors as well may be confusing also because it parts from the allready well-known
sitemap syntax. Let's go with your solution for now we can always change it when get into
trouble with that approach.

-- Unico


Mime
View raw message