<div dir="ltr"><div dir="ltr">Hii</div><div dir="auto"><div class="gmail_quote"><div dir="ltr" class="gmail_attr"><br></div><div dir="ltr" class="gmail_attr">On Sun, 23 Feb, 2020, 2:07 AM Chris Johns, <<a href="mailto:chrisj@rtems.org" target="_blank">chrisj@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi<br>
<br>
How did you test this patch?<br></blockquote><div><br></div><div>As I already told the search field works without this patch in sphinx versions newer</div><div>than v1.6.4. This is a bug with all version of sphinx between v1.6.4 and v.1.5.0</div><div>So to reproduce this bug in my local machine I had to install an older<br></div><div>version. And to make this accurate I wanted to install the same version of the sphinx</div><div>as in the doc server.</div><div>So, I started looking at the sphinx release notes and commits. After trying out different</div><div>versions, I came to the conclusion that the server uses sphinx v1.5.4 or something</div><div>close to that. After installing v1.5.4 in my local machine I was able to reproduce this bug.</div><div>And after applying this patch I was able to fix it.</div><div><br></div><div>sphinx_rtd_theme was also facing the same issue but it was fixed by a patch same as</div><div>this</div><div><a href="https://github.com/readthedocs/sphinx_rtd_theme/pull/346/commits/a5e0e304d30ed7aa1eacc6cca04514913b8d2385">https://github.com/readthedocs/sphinx_rtd_theme/pull/346/commits/a5e0e304d30ed7aa1eacc6cca04514913b8d2385</a></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
I have applied the patch, cleaned, built and installed the built docs, opened<br>
the User manual and searched for "Windows" and it found nothing. I am not sure<br>
if this is my set up, I am in another country to the machine with the docs. It<br>
could be something else.<br>
<br>
Thanks<br>
Chris<br>
<br>
On 22/2/20 11:05 pm, G S Niteesh Babu wrote:<br>
> This patch fixes the search field which previously was not working<br>
> due to this commit 71dd8bfbf94417ad55b2444e1dbd219db266f335 in<br>
> sphinx.<br>
> ---<br>
>  common/sphinx_rtd_theme_rtems/layout.html     | 3 ++-<br>
>  common/sphinx_rtd_theme_rtems/layout_old.html | 3 ++-<br>
>  2 files changed, 4 insertions(+), 2 deletions(-)<br>
> <br>
> diff --git a/common/sphinx_rtd_theme_rtems/layout.html b/common/sphinx_rtd_theme_rtems/layout.html<br>
> index 44978c9..ca63b80 100644<br>
> --- a/common/sphinx_rtd_theme_rtems/layout.html<br>
> +++ b/common/sphinx_rtd_theme_rtems/layout.html<br>
> @@ -167,7 +167,8 @@<br>
>              VERSION:'{{ release|e }}',<br>
>              COLLAPSE_INDEX:false,<br>
>              FILE_SUFFIX:'{{ '' if no_search_suffix else file_suffix }}',<br>
> -            HAS_SOURCE:  {{ has_source|lower }}<br>
> +            HAS_SOURCE:  {{ has_source|lower }},<br>
> +            SOURCELINK_SUFFIX: '{{ sourcelink_suffix }}'<br>
>          };<br>
>      </script><br>
>      {%- for scriptfile in script_files %}<br>
> diff --git a/common/sphinx_rtd_theme_rtems/layout_old.html b/common/sphinx_rtd_theme_rtems/layout_old.html<br>
> index deb8df2..f0dc2af 100644<br>
> --- a/common/sphinx_rtd_theme_rtems/layout_old.html<br>
> +++ b/common/sphinx_rtd_theme_rtems/layout_old.html<br>
> @@ -91,7 +91,8 @@<br>
>          VERSION:     '{{ release|e }}',<br>
>          COLLAPSE_INDEX: false,<br>
>          FILE_SUFFIX: '{{ '' if no_search_suffix else file_suffix }}',<br>
> -        HAS_SOURCE:  {{ has_source|lower }}<br>
> +        HAS_SOURCE:  {{ has_source|lower }},<br>
> +        SOURCELINK_SUFFIX: '{{ sourcelink_suffix }}'<br>
>        };<br>
>      </script><br>
>      {%- for scriptfile in script_files %}<br>
> <br><br>
</blockquote></div></div>
</div>