Closed Bug 1259514 Opened 4 years ago Closed 4 years ago

Sphinx docs generation broken on RTD due to missing manifestparser

Categories

(Firefox Build System :: General, defect)

defect
Not set

Tracking

(firefox48 fixed)

RESOLVED FIXED
mozilla48
Tracking Status
firefox48 --- fixed

People

(Reporter: gps, Assigned: gps)

Details

Attachments

(1 file)

Just like bug 1252294 :/

Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/gecko/envs/latest/local/lib/python2.7/site-packages/sphinx/cmdline.py", line 243, in main
    opts.warningiserror, opts.tags, opts.verbosity, opts.jobs)
  File "/home/docs/checkouts/readthedocs.org/user_builds/gecko/envs/latest/local/lib/python2.7/site-packages/sphinx/application.py", line 144, in __init__
    self.setup_extension(extension)
  File "/home/docs/checkouts/readthedocs.org/user_builds/gecko/envs/latest/local/lib/python2.7/site-packages/sphinx/application.py", line 442, in setup_extension
    ext_meta = mod.setup(self)
  File "/home/docs/checkouts/readthedocs.org/user_builds/gecko/checkouts/latest/python/mozbuild/mozbuild/sphinx.py", line 180, in setup
    from moztreedocs import SphinxManager
  File "/home/docs/checkouts/readthedocs.org/user_builds/gecko/checkouts/latest/tools/docs/moztreedocs/__init__.py", line 9, in <module>
    from mozbuild.frontend.reader import BuildReader
  File "/home/docs/checkouts/readthedocs.org/user_builds/gecko/checkouts/latest/python/mozbuild/mozbuild/frontend/reader.py", line 44, in <module>
    from mozbuild.testing import (
  File "/home/docs/checkouts/readthedocs.org/user_builds/gecko/checkouts/latest/python/mozbuild/mozbuild/testing.py", line 17, in <module>
    import manifestparser
ImportError: No module named manifestparser
mozbuild Python changes to test manifest processing introduced these
dependencies.

DONTBUILD (NPOTB)

Review commit: https://reviewboard.mozilla.org/r/42247/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/42247/
Attachment #8734454 - Flags: review?(cmanchester)
Attachment #8734454 - Flags: review?(cmanchester) → review+
Comment on attachment 8734454 [details]
MozReview Request: Bug 1259514 - Add manifestparser and reftest to Python search path; r?chmanchester

https://reviewboard.mozilla.org/r/42247/#review38895
https://hg.mozilla.org/mozilla-central/rev/0ab0c9391d64
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.