Return-Path: 
 <forrest-dev-return-2546-apmail-xml-forrest-dev-archive=xml.apache.org@xml.apache.org>
Delivered-To: apmail-xml-forrest-dev-archive@xml.apache.org
Received: (qmail 83104 invoked by uid 500); 19 Sep 2002 12:34:21 -0000
Mailing-List: contact forrest-dev-help@xml.apache.org; run by ezmlm
Precedence: bulk
list-help: <mailto:forrest-dev-help@xml.apache.org>
list-unsubscribe: <mailto:forrest-dev-unsubscribe@xml.apache.org>
list-post: <mailto:forrest-dev@xml.apache.org>
Reply-To: forrest-dev@xml.apache.org
Delivered-To: mailing list forrest-dev@xml.apache.org
Received: (qmail 83089 invoked from network); 19 Sep 2002 12:34:20 -0000
Received: from host179-154.pool80204.interbusiness.it (HELO aisa?2nd.aisa)
 (80.204.154.179)
  by daedalus.apache.org with SMTP; 19 Sep 2002 12:34:20 -0000
Received: from apache.org ([192.4.0.103]) by aisa_2nd.aisa (Lotus SMTP MTA
 v4.6.6  (890.1 7-16-1999)) with SMTP id C1256C39.0046D36D;
 Thu, 19 Sep 2002 14:53:36 +0200
Message-ID: <3D89C40C.3040500@apache.org>
Date: Thu, 19 Sep 2002 14:33:16 +0200
From: Nicola Ken Barozzi <nicolaken@apache.org>
Reply-To: nicolaken@apache.org
Organization: Apache Software Foundation
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US;
 rv:1.1) Gecko/20020826
X-Accept-Language: en-us, en
MIME-Version: 1.0
To: forrest-dev@xml.apache.org
Subject: Re: No more @skin@ in sitemap (Re: cvs commit:
 xml-forrest/src/resources/conf
 cocoon.xconf sitemap.xmap)
References: <20020916233425.21184.qmail@icarus.apache.org>
 <20020917064431.GA7006@expresso.localdomain>
 <3D8745FF.3040708@outerthought.org>
 <20020918024418.GA730@expresso.localdomain>
 <3D882F7C.4020803@outerthought.org>
 <20020918082528.GD730@expresso.localdomain>
 <3D886EA4.8000901@outerthought.org> <3D89B802.7080808@apache.org>
 <3D89C203.1010109@outerthought.org>
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit
X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N
Status: O
X-Status: 
X-Keywords: 


Marc Portier wrote:
> <snip/>
> 
>>>>>> <component-instance
>>>>>>    
>>>>>> class="org.apache.cocoon.components.modules.input.DefaultsMetaModule"
>>>>>>    name="defaults">
>>>>>> <input-module name="request"/> <!-- First use request params -->
>>>>>> <input-module name="config"/>  <!-- .. Then fall back to config 
>>>>>> params -->
>>>>>> <values>                       <!-- .. Otherwise use defaults -->
>>>>>>   <skin>forrest-site</skin>
>>>>>>   <base-url>/forrest</base-url>
>>>>>>  </values>
>>>>>> </component-instance>
>>>>>>
>>>>>> Umm. Slightly fictional but I don't see how else to fix the bot.
>>>>>>
>>>>>
>>>>> mmm, would it not be easier for the bot to generate the active 
>>>>> skinconf.xml file based on the local skinconf and possibly 
>>>>> overriding any variables in there with values supplied at 
>>>>> (bot)-generation time
>>>>
>>>>
>>>>
>>>>
>>>>
>>>> But skinconf.xml does not select the skin, so auto-generating is 
>>>> won't do
>>>> any good.
>>>>
>>> didn't catch that, sorry...
>>>
>>> I was refering to your idea of having a skinconf oriented inputmodule 
>>> (that's how these are called right?)
>>>
>>> so maybe this remark just pushes for a different name for the beast 
>>> then :-)
>>>
>>> Could there be a this-run-modifying-config.xml (maybe a simple props 
>>> file could suffice?) that is read in through these inputmodules? so 
>>> we get to have some {forrest:skin}  inside the sitemap?
>>
>>
>>
>> That's an idea.
>> I would prefere a JXPath input module that simply accesses the info in 
>> a JXPAthable object (xml DOM, javabeans, etc), just like the jxpath 
>> inputmodule for Ant embed does.
>> http://krysalis.org/cgi-bin/krywiki.pl?AntJXPath
>>
> 
> Is it bragging when I claim that I was hoping you to say this :-)
> in doubth about being nice to you or Konstatin now :-)
> 
> there is some advantage to the simplicity of having a simple props file 
> read in by the inputmodule though...
> 
> the most important one I see is that writing the sitemap will not 
> involve {forrest:some/nifty/xpath[hard-to-read(xpression)]/} but rather 
> straightforward {forrest:property.key.name}

{forrest:property/key/name}

Which is xpath. It all depends on what you want to read.

{forrest:property/key/name/@attr}
{forrest:property/key/name[@attr='isitthere']/@attr}

This is usually as hard as it can get.

> also the property-file impl could more easily be learnt to also work for 
> System props (and thus support the cocoon.Main -Dskin=... example Jeff 
> ended with)

Ok, put it on the list that system properties can override defined ones.

-- 
Nicola Ken Barozzi                   nicolaken@apache.org
             - verba volant, scripta manent -
    (discussions get forgotten, just code remains)
---------------------------------------------------------------------