You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@forrest.apache.org by Lorenz Froihofer <Lo...@gmx.at> on 2004/09/27 13:11:58 UTC

[patch] Bug in fresh-site site.xml

It seems that I do not have enough permissions to create a new issue in 
the issue tracker. The description for the patch follows:

If you do a "forrest seed" and "forrest site" in a new directory, the 
following bug is present.

Select 1st level tab Samples, afterwards select 2nd level tab "Index". 
Notice that now the 1st level tab "Home" is selected instead of "Samples".

The attached patch fixes this issue.

Note that there are some other failures during the build of a fresh site 
with "forrest site".

Kind regards,
   Lorenz Froihofer.

Re: [patch] Bug in fresh-site site.xml

Posted by Lorenz Froihofer <Lo...@gmx.at>.
Rick Tessner wrote:
...
> <snip what="BROKEN: No pipeline matched requests"/>
> 
> This can happen if in the skinconf.xml the <disable-xml-link> is set to 
> "false" (the default is "true").  In this case, this is the same issue 
> as http://issues.cocoondev.org/browse/FOR-284
> 
> If the <disable-xml-link> is set to "true", I don't know why you'd be 
> seeing this ... :(
> 
> What is the value of the <disable-xml-link> in your 
> src/documentation/skinconf.xml?
> 

The value is "true".

-- Lorenz.

Re: [patch] Bug in fresh-site site.xml

Posted by Rick Tessner <ri...@onnadayr.ca>.
Lorenz Froihofer wrote:
> 
>> What "other" are you referring to?
>> I can only see the "samples/ext:dtd-docs" failure
>> which we think is due to:
>> http://issues.cocoondev.org/browse/FOR-284
>>
>> --David
> 
> 
> The build failures I get when builing a fresh site (resulting from 
> "forrest seed") are:
> 

<snip what="BROKEN: No pipeline matched requests"/>

This can happen if in the skinconf.xml the <disable-xml-link> is set to 
"false" (the default is "true").  In this case, this is the same issue 
as http://issues.cocoondev.org/browse/FOR-284

If the <disable-xml-link> is set to "true", I don't know why you'd be 
seeing this ... :(

What is the value of the <disable-xml-link> in your 
src/documentation/skinconf.xml?

-- 
rick@onnadayr.ca
IT Firefighting and Prevention

Re: [patch] Bug in fresh-site site.xml

Posted by David Crossley <cr...@apache.org>.
David Crossley wrote:
> Lorenz Froihofer wrote:
> > David Crossley wrote:
> > > What "other" are you referring to?
> > > I can only see the "samples/ext:dtd-docs" failure
> > > which we think is due to:
> > > http://issues.cocoondev.org/browse/FOR-284
> > 
> > The build failures I get when builing a fresh site (resulting from 
> > "forrest seed") are:
<snip/>
> 
> There is something seriously wrong for you.
> What operating system are you on?
> What version of Forrest do you use?
> What skin are you using?
> I do not see these issues with the head of SVN.

Oops, sorry Lorenz, you are absolutely correct.
I was not up-to-date with the Head. Yes, it is
seriously broken again.

-- 
David Crossley


Re: [patch] Bug in fresh-site site.xml

Posted by David Crossley <cr...@apache.org>.
Lorenz Froihofer wrote:
> David Crossley wrote:
> > What "other" are you referring to?
> > I can only see the "samples/ext:dtd-docs" failure
> > which we think is due to:
> > http://issues.cocoondev.org/browse/FOR-284
> 
> The build failures I get when builing a fresh site (resulting from 
> "forrest seed") are:

There is something seriously wrong for you.
What operating system are you on?
What version of Forrest do you use?
What skin are you using?
I do not see these issues with the head of SVN.

--David

> X [0]                                     samples/ext:forrest/linking 
> BROKEN: No pipeline matched request: samples/ext:forrest/linking
> X [0]                                     samples/site:index	BROKEN: No 
> pipeline matched request: samples/site:index
> X [0]                                     samples/ext:linking	BROKEN: No 
> pipeline matched request: samples/ext:linking
> X [0]                                     samples/site:test3	BROKEN: No 
> pipeline matched request: samples/site:test3
> X [0]                                     samples/site:samples/faq 
> BROKEN: No pipeline matched request: samples/site:samples/faq
> X [0] 
> samples/site:samples/sample/section	BROKEN: No pipeline matched request: 
> samples/site:samples/sample/section
> X [0]                                     samples/ext:dtd-docs	BROKEN: 
> No pipeline matched request: samples/ext:dtd-docs
> X [0]                                     samples/ext:forrest/webapp 
> BROKEN: No pipeline matched request: samples/ext:forrest/webapp
> X [0]                                     samples/ext:webapp	BROKEN: No 
> pipeline matched request: samples/ext:webapp
> X [0]                                     samples/site:hello_print 
> BROKEN: No pipeline matched request: samples/site:hello_print
> X [0] 
> samples/ext:forrest/validation	BROKEN: No pipeline matched request: 
> samples/ext:forrest/validation
> X [0]                                     samples/site:linking	BROKEN: 
> No pipeline matched request: samples/site:linking
> X [0]                                     samples/ext:forrest	BROKEN: No 
> pipeline matched request: samples/ext:forrest
> X [0]                                     samples/site:faq	BROKEN: No 
> pipeline matched request: samples/site:faq
> X [0]                                     samples/site:samples	BROKEN: 
> No pipeline matched request: samples/site:samples
> 
> -- Lorenz.


Re: [patch] Bug in fresh-site site.xml

Posted by Lorenz Froihofer <Lo...@gmx.at>.
> What "other" are you referring to?
> I can only see the "samples/ext:dtd-docs" failure
> which we think is due to:
> http://issues.cocoondev.org/browse/FOR-284
> 
> --David

The build failures I get when builing a fresh site (resulting from 
"forrest seed") are:

X [0]                                     samples/ext:forrest/linking 
BROKEN: No pipeline matched request: samples/ext:forrest/linking
X [0]                                     samples/site:index	BROKEN: No 
pipeline matched request: samples/site:index
X [0]                                     samples/ext:linking	BROKEN: No 
pipeline matched request: samples/ext:linking
X [0]                                     samples/site:test3	BROKEN: No 
pipeline matched request: samples/site:test3
X [0]                                     samples/site:samples/faq 
BROKEN: No pipeline matched request: samples/site:samples/faq
X [0] 
samples/site:samples/sample/section	BROKEN: No pipeline matched request: 
samples/site:samples/sample/section
X [0]                                     samples/ext:dtd-docs	BROKEN: 
No pipeline matched request: samples/ext:dtd-docs
X [0]                                     samples/ext:forrest/webapp 
BROKEN: No pipeline matched request: samples/ext:forrest/webapp
X [0]                                     samples/ext:webapp	BROKEN: No 
pipeline matched request: samples/ext:webapp
X [0]                                     samples/site:hello_print 
BROKEN: No pipeline matched request: samples/site:hello_print
X [0] 
samples/ext:forrest/validation	BROKEN: No pipeline matched request: 
samples/ext:forrest/validation
X [0]                                     samples/site:linking	BROKEN: 
No pipeline matched request: samples/site:linking
X [0]                                     samples/ext:forrest	BROKEN: No 
pipeline matched request: samples/ext:forrest
X [0]                                     samples/site:faq	BROKEN: No 
pipeline matched request: samples/site:faq
X [0]                                     samples/site:samples	BROKEN: 
No pipeline matched request: samples/site:samples

-- Lorenz.

Re: [patch] Bug in fresh-site site.xml

Posted by David Crossley <cr...@apache.org>.
Lorenz Froihofer wrote:
> It seems that I do not have enough permissions to create a new issue in 
> the issue tracker. The description for the patch follows:
> 
> If you do a "forrest seed" and "forrest site" in a new directory, the 
> following bug is present.
> 
> Select 1st level tab Samples, afterwards select 2nd level tab "Index". 
> Notice that now the 1st level tab "Home" is selected instead of "Samples".
> 
> The attached patch fixes this issue.

However it adds an extra "Index" item into the menu.

Omitting the "@label" attribute from the new <index> entry
also "fixes" the issue, but it collapses the Sample menu.

This need for an <index> entry in site.xml might be a
symptom of some other bug.

> Note that there are some other failures during the build of a fresh site 
> with "forrest site".

What "other" are you referring to?
I can only see the "samples/ext:dtd-docs" failure
which we think is due to:
http://issues.cocoondev.org/browse/FOR-284

--David

> Kind regards,
>    Lorenz Froihofer.
>
> Index: src/core/fresh-site/src/documentation/content/xdocs/site.xml
> ===================================================================
> --- src/core/fresh-site/src/documentation/content/xdocs/site.xml	(revision 47281)
> +++ src/core/fresh-site/src/documentation/content/xdocs/site.xml	(working copy)
> @@ -38,6 +38,7 @@
>    </about>
>  
>    <samples label="Samples" href="samples/" tab="samples">
> +    <index label="Index" href="index.html" />
>      <sample label="Apache doc v2.0" href="sample.html" 
>        description="A nonsense document using all possible elements in the current document v2.0">
>        <top href="#top"/>