Generating Indices fails when used with Spinx 1.4.1

this is an example output:

$ sphinx-build -b pdf source build/pdf
Running Sphinx v1.4.1
loading pickled environment... done
building [mo]: targets for 0 po files that are out of date
building [pdf]: targets for 26 source files that are out of date
updating environment: 0 added, 0 changed, 0 removed
looking for now-outdated files... none found
processing PyMuPDF... index intro changes installation tutorial classes colorspace device displaylist document identity irect link linkdest matrix outline page pixmap point rect textpage textsheet functions vars app1 app2 [ERROR] pdfbuilder.py:130 too many values to unpack
Traceback (most recent call last):
  File "c:\python27\lib\site-packages\rst2pdf\pdfbuilder.py", line 122, in write
    appendices=opts.get('pdf_appendices', self.config.pdf_appendices) or [])
  File "c:\python27\lib\site-packages\rst2pdf\pdfbuilder.py", line 209, in assemble_doctree
    index_nodes=genindex_nodes(genindex)
  File "c:\python27\lib\site-packages\rst2pdf\pdfbuilder.py", line 385, in genindex_nodes
    for entryname, (links, subitems) in entries:
ValueError: too many values to unpack
FAILED
build succeeded.
$

Setting pdf_use_index = False in conf.py will get the document created - without index. I know that this same rst2pdf version worked with Sphinx 1.3.1.

[fgerstenberg]

i came across the same problem a while ago. It seems the new Sphinx version reports an additional field in its index entries. currently line 385 in pdfbuilder.py reads as following:

        for entryname, (links, subitems) in entries:
            if links:
                output.append('`%s <#%s>`_'%(entryname,nodes.make_id(links[0][1])))
                for i,link in enumerate(links[1:]):
                    output[-1]+=(' `[%s] <#%s>`_ '%(i+1,nodes.make_id(link[1])))
                output.append('')

But since the second part of entries is now a list of length 3, you get the ValueError. Making the following changes will solve the error and should be compatible with the old format as well:

        for entryname, add_info in entries:
            links = add_info[0]
            subitems = add_info[1]
            if links:
                output.append('`%s <#%s>`_'%(entryname,nodes.make_id(links[0][1])))
                for i,link in enumerate(links[1:]):
                    output[-1]+=(' `[%s] <#%s>`_ '%(i+1,nodes.make_id(link[1])))
                output.append('')

This will simply drop the additional field. In every case i tested it had the value None anyway.

[pmaupin]

Yeah, “on hiatus.”

Not enough hours in the day for everything that needs to be done…