Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.debian.maint.python > #15478 > unrolled thread

Spinx help needed

Back to article view | Back to linux.debian.maint.python

Contents

  Spinx help needed Andreas Tille <andreas@an3as.eu> - 2024-02-15 08:40 +0100
    Re: Spinx help needed c.buhtz@posteo.jp - 2024-02-15 09:20 +0100
    Re: Spinx help needed Dmitry Shachnev <mitya57@debian.org> - 2024-02-15 09:50 +0100
      Re: Spinx help needed Andreas Tille <andreas@an3as.eu> - 2024-02-15 19:20 +0100
        Re: Spinx help needed Dmitry Shachnev <mitya57@debian.org> - 2024-02-15 20:00 +0100
          Re: Spinx help needed Andreas Tille <andreas@an3as.eu> - 2024-02-15 20:10 +0100

#15478 — Spinx help needed

Control: tags -1 pending

Hi,

I pushed fixes for #1056419 and #1058311 to Git and I think should be
fixed as well.  The only remaining build problem is new and caused by
sphinx[1]:

  dh_sphinxdoc -i -O--buildsystem=pybuild
dh_sphinxdoc: error: debian/python-lmfit-doc/usr/share/doc/python3-lmfit/html/search.html top-level node does not have data-content_root attribute


Unfortunately I have no idea how to fix this.  Any ideas?

Kind regards
    Andreas.


[1] https://buildd.debian.org/status/package.php?p=lmfit-py&suite=experimental

-- 
http://fam-tille.de

[toc] | [next] | [standalone]

#15479

It is always hard to follow such threads.

It is unclear if this message is about the Sphinx package itself or a 
package that do use Sphinx in its build process.

There are two bug numbers in the CC fields. But the mail miss links to 
the bug reports or other information giving more context.
This makes it very hard for new contributors participating. A bit more 
transparency would improve the process itself and attract more 
contributors.

Thanks in advance
Christian

[toc] | [prev] | [next] | [standalone]

#15480

[Multipart message — attachments visible in raw view] — view raw

Hi Andreas!

On Thu, Feb 15, 2024 at 08:37:38AM +0100, Andreas Tille wrote:
> Control: tags -1 pending
> 
> Hi,
> 
> I pushed fixes for #1056419 and #1058311 to Git and I think should be
> fixed as well.  The only remaining build problem is new and caused by
> sphinx[1]:
> 
>   dh_sphinxdoc -i -O--buildsystem=pybuild
> dh_sphinxdoc: error: debian/python-lmfit-doc/usr/share/doc/python3-lmfit/html/search.html top-level node does not have data-content_root attribute
> 
> Unfortunately I have no idea how to fix this.  Any ideas?

lmfit-py ships a vendored copy of sphinx13 theme [1], which was copied from
Sphinx source code with a minor modification in 2020 [2] and rebased in
January 2022 [3]. However, there were more Sphinx releases since that month,
and the theme needs to be updated for compatibility with them.

In particular, the basic_layout.html file misses the change which was made
in Sphinx commit [4], without which the search will not work. There is a
comment under that commit which illustrates how exactly it will not work:
contentRoot will be undefined, and the browser will attempt to make requests
to a URL that has "undefined" in it. dh_sphinxdoc catches such issues and
produces an error about them.

So, to fix this issue, you should copy sphinx/themes/basic/layout.html from
the latest stable version of Sphinx to lmfit-py's basic_layout.html, applying
the one-line change which is described in [2] and [3].

[1]: doc/sphinx/theme/sphinx13/*
[2]: https://github.com/lmfit/lmfit-py/commit/29e4712036606913149e16b246340a7fbedd8829
[3]: https://github.com/lmfit/lmfit-py/commit/e2418377c9870e02c820d0fe40d2232187864a81
[4]: https://github.com/sphinx-doc/sphinx/commit/8e730ae303ae686705ea12f44ef11da926a87cf5

--
Dmitry Shachnev

[toc] | [prev] | [next] | [standalone]

#15482

Hi Dmitry,

thanks a lot for this hint.

Am Thu, Feb 15, 2024 at 11:46:45AM +0300 schrieb Dmitry Shachnev:
> lmfit-py ships a vendored copy of sphinx13 theme [1], which was copied from
> Sphinx source code with a minor modification in 2020 [2] and rebased in
> January 2022 [3]. However, there were more Sphinx releases since that month,
> and the theme needs to be updated for compatibility with them.

I've created a quilt patch which replaces the vendored theme by the
sphinx13 one which is currently shipped in Debian.  The patches by
upstream seem to be void now - at least none applied to the current
version.

Unfortunately this does not build as well - now with other errors
you can see in Salsa CI at

   https://salsa.debian.org/science-team/lmfit-py/-/jobs/5305535

Do you have any further hints?

Kind regards
   Andreas.
 
> In particular, the basic_layout.html file misses the change which was made
> in Sphinx commit [4], without which the search will not work. There is a
> comment under that commit which illustrates how exactly it will not work:
> contentRoot will be undefined, and the browser will attempt to make requests
> to a URL that has "undefined" in it. dh_sphinxdoc catches such issues and
> produces an error about them.
> 
> So, to fix this issue, you should copy sphinx/themes/basic/layout.html from
> the latest stable version of Sphinx to lmfit-py's basic_layout.html, applying
> the one-line change which is described in [2] and [3].
> 
> [1]: doc/sphinx/theme/sphinx13/*
> [2]: https://github.com/lmfit/lmfit-py/commit/29e4712036606913149e16b246340a7fbedd8829
> [3]: https://github.com/lmfit/lmfit-py/commit/e2418377c9870e02c820d0fe40d2232187864a81
> [4]: https://github.com/sphinx-doc/sphinx/commit/8e730ae303ae686705ea12f44ef11da926a87cf5
> 
> --
> Dmitry Shachnev



-- 
http://fam-tille.de

[toc] | [prev] | [next] | [standalone]

#15483

[Multipart message — attachments visible in raw view] — view raw

On Thu, Feb 15, 2024 at 07:15:18PM +0100, Andreas Tille wrote:
> Hi Dmitry,
>
> thanks a lot for this hint.
>
> I've created a quilt patch which replaces the vendored theme by the
> sphinx13 one which is currently shipped in Debian.  The patches by
> upstream seem to be void now - at least none applied to the current
> version.
>
> Unfortunately this does not build as well - now with other errors
> you can see in Salsa CI at
>
>    https://salsa.debian.org/science-team/lmfit-py/-/jobs/5305535
>
> Do you have any further hints?

It looks like what you did was copying layout.html to basic_layout.html
within lmfit-py's doc/sphinx/theme/sphinx13/ directory.

But what I suggested was copying a file from *Sphinx* source package,
sphinx/themes/basic/layout.html to doc/sphinx/theme/sphinx13/basic_layout.html
in lmfit-py.

With your change basic_layout.html tried to extend itself, which caused
a recursion error.

I pushed a fix and the build job succeeded now. build-i386 failed, but that
one is not related to Sphinx.

--
Dmitry Shachnev

[toc] | [prev] | [next] | [standalone]

#15484

Hi Dmitry,

thanks a lot and sorry for my naivity

   Andreas.

Am Thu, Feb 15, 2024 at 09:54:55PM +0300 schrieb Dmitry Shachnev:
> On Thu, Feb 15, 2024 at 07:15:18PM +0100, Andreas Tille wrote:
> > Hi Dmitry,
> >
> > thanks a lot for this hint.
> >
> > I've created a quilt patch which replaces the vendored theme by the
> > sphinx13 one which is currently shipped in Debian.  The patches by
> > upstream seem to be void now - at least none applied to the current
> > version.
> >
> > Unfortunately this does not build as well - now with other errors
> > you can see in Salsa CI at
> >
> >    https://salsa.debian.org/science-team/lmfit-py/-/jobs/5305535
> >
> > Do you have any further hints?
> 
> It looks like what you did was copying layout.html to basic_layout.html
> within lmfit-py's doc/sphinx/theme/sphinx13/ directory.
> 
> But what I suggested was copying a file from *Sphinx* source package,
> sphinx/themes/basic/layout.html to doc/sphinx/theme/sphinx13/basic_layout.html
> in lmfit-py.
> 
> With your change basic_layout.html tried to extend itself, which caused
> a recursion error.
> 
> I pushed a fix and the build job succeeded now. build-i386 failed, but that
> one is not related to Sphinx.
> 
> --
> Dmitry Shachnev



-- 
http://fam-tille.de

[toc] | [prev] | [standalone]

Back to top | Article view | linux.debian.maint.python

csiph-web