This issue tracker has been migrated to GitHub ,
and is currently read-only.
For more information,
see the GitHub FAQs in the Python's Developer Guide.
Created on 2016年03月25日 04:49 by martin.panter, last changed 2022年04月11日 14:58 by admin. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| doc-warnings.patch | martin.panter, 2016年03月25日 04:49 | review | ||
| doc-warnings.v2.patch | martin.panter, 2016年04月16日 10:46 | review | ||
| disable-warning.patch | martin.panter, 2016年04月16日 10:47 | review | ||
| doc-warnings.v3.patch | martin.panter, 2016年07月26日 07:03 | review | ||
| Messages (17) | |||
|---|---|---|---|
| msg262401 - (view) | Author: Martin Panter (martin.panter) * (Python committer) | Date: 2016年03月25日 04:49 | |
Defeat many warnings and related fixes building documentation * Mask various CLI options with exclamation marks (!) * Correct link to CLI section in zipapp * Make CLI section label in timeit less ambiguous The following warnings from unittest remain. Ideally they should be linked to the Python CLI documentation, but this does not seem possible because the "unittest" documentation defines its own CLI options. Doc/library/unittest.rst:1965: WARNING: unknown option: -Wd Doc/library/unittest.rst:1965: WARNING: unknown option: -Wa Doc/library/unittest.rst:2051: WARNING: unknown option: -W |
|||
| msg262592 - (view) | Author: Martin Panter (martin.panter) * (Python committer) | Date: 2016年03月29日 07:14 | |
FTR the warnings that I am fixing were apparently enabled in Sphinx 1.3.4, reverted in 1.3.6, and added in 1.4. |
|||
| msg263405 - (view) | Author: STINNER Victor (vstinner) * (Python committer) | Date: 2016年04月14日 12:52 | |
> FTR the warnings that I am fixing were apparently enabled in Sphinx 1.3.4, reverted in 1.3.6, and added in 1.4. Would it be possible to turn off the warning? Doc/using/cmdline.py uses ".. cmdoption::". If we cannot turn off the warning on option, maybe we should use a new :cmdoption:`xxx` which wouldn't emit a warning? I'm not strongly opposed to doc-warnings.patch. I understand that Sphinx expects the exact option, for example -W, whereas the doc uses option with parameter like -Wd. Sphinx is unable to link to the option. Maybe we need something like :cmdoption:`-Wd <-W>`: display -Wd but link to -W? |
|||
| msg263434 - (view) | Author: Martin Panter (martin.panter) * (Python committer) | Date: 2016年04月14日 23:24 | |
I’m not an expert on Sphinx. Maybe <http://www.sphinx-doc.org/en/stable/config.html#confval-suppress_warnings> could help with disabling these warnings. Considering there are so many false positives, it might be best to disable the warning. Though my patch does have a few genuine improvements as well. I already used the :option:`-Wd <-W>` syntax in some cases; see Doc/library/warnings.rst for example. The problem in unittest.rst is I think Sphinx is trying to find a -W option defined within unittest.rst (with the unittest --buffer etc options), rather than the main Python options from using/cmdline.rst. |
|||
| msg263539 - (view) | Author: Roundup Robot (python-dev) (Python triager) | Date: 2016年04月16日 07:47 | |
New changeset 23a884c32a39 by Martin Panter in branch '3.5': Issue #26638: Fix links to some CLI options and section headings https://hg.python.org/cpython/rev/23a884c32a39 New changeset 68b84643dffd by Martin Panter in branch 'default': Issue #26638: Merge link fixes from 3.5 https://hg.python.org/cpython/rev/68b84643dffd New changeset 3e0321584f23 by Martin Panter in branch '2.7': Issue #26638: Fix links to some CLI options https://hg.python.org/cpython/rev/3e0321584f23 |
|||
| msg263547 - (view) | Author: Martin Panter (martin.panter) * (Python committer) | Date: 2016年04月16日 10:42 | |
I committed the parts of my patch that improve real links in the end result. doc-warnings.v2.patch has the remaining changes which add exclamation marks to mask undefined options. |
|||
| msg263548 - (view) | Author: Martin Panter (martin.panter) * (Python committer) | Date: 2016年04月16日 10:47 | |
This patch disables the warning, but I tend to think it is not a good long-term solution. |
|||
| msg271335 - (view) | Author: Roundup Robot (python-dev) (Python triager) | Date: 2016年07月26日 06:53 | |
New changeset 3996dd17ca21 by Martin Panter in branch '3.5': Issue #26638: Don’t link 2to3 -l option to the tarfile -l option https://hg.python.org/cpython/rev/3996dd17ca21 New changeset d7cbaaba59e8 by Martin Panter in branch 'default': Issue #26638: Merge CLI link fix from 3.5 https://hg.python.org/cpython/rev/d7cbaaba59e8 |
|||
| msg273588 - (view) | Author: Terry J. Reedy (terry.reedy) * (Python committer) | Date: 2016年08月24日 20:18 | |
I built the 3.5 docs on Win10 with Sphinx 1.3.3 and corresponding dependencies and got a new warning as a result of 8ba0000c90be for #26462: F:\Python\dev36円\Doc\whatsnew3円.2.rst:2063: WARNING: Pygments lexer name 'doscon' is not known Thinking the problem was an out-of-date pygments module, I updated sphinx and dependencies (to 1.4.6, etc). I was right that updating pygments fixed the warning above, but now I get 140 warnings!!! 4 are from the reading phase for configparser.rst. The first: F:\Python\dev35円\Doc\library\configparser.rst:240: WARNING: Could not lex literal_block as "ini". Highlighting skipped. I presume this could be fixed, but I don't know how. 136 are about "<path: WARNING: unknown option: -x." Sphinx, at least with how we use it, and the Python docs are out of sync here. I think we need more eyes on this. "This patch disables the warning," ??? Not here. " but I tend to think it is not a good long-term solution." Printing 136 warnings is also not a good solution. I rebuilt the docs locally because I was planning to patch a misspelling I found while reading the doc. Looking in a list of several pages for a warning I might have caused is failure-prone and no fun. In the Windows console, the page-up and page-down keys do not page up and down. To not miss anything, I would have to carefully use a mouse wheel or scrollbar clicks. I am building with "sphinx-build -bhtml . build/html" is a .bat file. Is there anything I can add to suppress only 'unknown option' warnings? |
|||
| msg273591 - (view) | Author: Zachary Ware (zach.ware) * (Python committer) | Date: 2016年08月24日 20:33 | |
Terry: for now, you can downgrade Sphinx to 1.3.6 as it is known to build with no warnings (that's the version the Docs buildbot currently uses). |
|||
| msg273597 - (view) | Author: Terry J. Reedy (terry.reedy) * (Python committer) | Date: 2016年08月24日 21:39 | |
Only 138 warnings with 2.7. Zack, how do I downgrade. I examined the result of pip install -h but cannot see how. I presume that 'sphinx' and '1.3.6' should be combined into a '<requirement specifier>' but this latter is not defined and the things I tried, including the output format of 'pip list' did not work. Does anything else need to be downgraded? |
|||
| msg273656 - (view) | Author: Zachary Ware (zach.ware) * (Python committer) | Date: 2016年08月25日 15:20 | |
The syntax is `<pip> install "<package><comparison><version>"`, so in this case 'pip install "Sphinx==1.3.6"` (or `pip install "Sphinx<1.4"`) should do it for you. That ought to also take care of downgrading any other packages that need it (though I don't think there are any), or at least tell you which ones should be downgraded and to what. I'm not sure why there are both `pip list` and `pip freeze`, but `pip freeze` is the one that outputs usable syntax for a requirement specifier. |
|||
| msg273674 - (view) | Author: Terry J. Reedy (terry.reedy) * (Python committer) | Date: 2016年08月25日 19:34 | |
Thank you, that worked -- and should have been documented in the help output. All the sphinx dependencies were inequalities, and rebuilding worked. The devguide only says "You are expected to have installed a recent version of Sphinx on your system or in a virtualenv,". Could we add "We currently recommend 'python -m pip install sphinx==1.3.6' as the easiest way to install and the best version for most people to install. See https://bugs.python.org/issue26638 for why." (I am assuming that pip works on all systems, not just Windows.) The sentence finishes with "so that the Makefile can find the sphinx-build command." On Windows. this is most easily done by installing Sphinx with a Python version that was added to PATH when installed. |
|||
| msg273747 - (view) | Author: Martin Panter (martin.panter) * (Python committer) | Date: 2016年08月27日 02:21 | |
Terry: the four "ini" syntax highlighting warnings were also mentioned in <https://bugs.python.org/issue26462#msg271241>. In that bug, we fixed many other similar warnings, but I think we left those as being less easy to fix. One thing (other than suppress_warnings = ['ref.option']) that might help you see warnings relevant to your change is if you first build the docs without your change, ignore all those warnings, and then rebuild with your change. The second build may produce less irrelevant warnings. |
|||
| msg279676 - (view) | Author: SilentGhost (SilentGhost) * (Python triager) | Date: 2016年10月29日 14:44 | |
Patch (v3) is fine, except it no longer applies cleanly. I have a patch against the tip which I could upload if needed. I would also be in favour of disabling the unittest options directly, currently no link is generated and it doesn't seem like there is a straightforward solution in sight. |
|||
| msg279708 - (view) | Author: Roundup Robot (python-dev) (Python triager) | Date: 2016年10月30日 05:50 | |
New changeset 307d7568b6ae by Martin Panter in branch '3.5': Issue #26638: Mask undefined CLI options to defeat new Sphinx warnings https://hg.python.org/cpython/rev/307d7568b6ae New changeset 57f4ba6b29bf by Martin Panter in branch '3.5': Issue #26638: Work around more CLI options that can’t be linked https://hg.python.org/cpython/rev/57f4ba6b29bf New changeset a0d272fbc7de by Martin Panter in branch '3.6': Issue #26638: Merge option warning fixes from 3.5 into 3.6 https://hg.python.org/cpython/rev/a0d272fbc7de New changeset 85e2cfe5b12d by Martin Panter in branch 'default': Issue #26638: Merge option warning fixes from 3.6 https://hg.python.org/cpython/rev/85e2cfe5b12d New changeset c4b934a77a08 by Martin Panter in branch '2.7': Issue #26638: Disable inappropriate links to Python interpreter options https://hg.python.org/cpython/rev/c4b934a77a08 New changeset 0ff00d53d6a9 by Martin Panter in branch '2.7': Issue #26638: Mask undefined CLI options to defeat new Sphinx warnings https://hg.python.org/cpython/rev/0ff00d53d6a9 New changeset cf91d48aa353 by Martin Panter in branch '2.7': Issue #26638: Cannot directly link to main option from the "timeit" module https://hg.python.org/cpython/rev/cf91d48aa353 |
|||
| msg279711 - (view) | Author: Martin Panter (martin.panter) * (Python committer) | Date: 2016年10月30日 06:34 | |
For the "unittest" module, I added separate text with a link: (see Warning control). Similarly for "timeit" in 2.7. Now the only Sphinx warnings I get are a handful about syntax highlighting, and two genuine ones relating to the missing -b documentation (Issue 11681). |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022年04月11日 14:58:28 | admin | set | github: 70825 |
| 2016年10月30日 06:34:21 | martin.panter | set | status: open -> closed resolution: fixed messages: + msg279711 stage: commit review -> resolved |
| 2016年10月30日 05:50:18 | python-dev | set | messages: + msg279708 |
| 2016年10月29日 14:44:06 | SilentGhost | set | versions:
+ Python 3.7 nosy: + SilentGhost messages: + msg279676 stage: patch review -> commit review |
| 2016年08月27日 02:21:57 | martin.panter | set | messages: + msg273747 |
| 2016年08月25日 19:34:25 | terry.reedy | set | messages: + msg273674 |
| 2016年08月25日 15:20:23 | zach.ware | set | messages: + msg273656 |
| 2016年08月24日 21:39:51 | terry.reedy | set | messages: + msg273597 |
| 2016年08月24日 20:33:24 | zach.ware | set | messages: + msg273591 |
| 2016年08月24日 20:18:53 | terry.reedy | set | nosy:
+ terry.reedy, ezio.melotti, georg.brandl, eric.araujo, zach.ware messages: + msg273588 |
| 2016年07月26日 07:03:45 | martin.panter | set | files: + doc-warnings.v3.patch |
| 2016年07月26日 06:53:10 | python-dev | set | messages: + msg271335 |
| 2016年04月16日 10:47:30 | martin.panter | set | files:
+ disable-warning.patch messages: + msg263548 |
| 2016年04月16日 10:46:14 | martin.panter | set | files: + doc-warnings.v2.patch |
| 2016年04月16日 10:42:30 | martin.panter | set | messages: + msg263547 |
| 2016年04月16日 07:47:28 | python-dev | set | nosy:
+ python-dev messages: + msg263539 |
| 2016年04月14日 23:24:15 | martin.panter | set | messages: + msg263434 |
| 2016年04月14日 12:52:19 | vstinner | set | nosy:
+ vstinner messages: + msg263405 |
| 2016年03月29日 07:14:25 | martin.panter | set | messages: + msg262592 |
| 2016年03月25日 04:49:40 | martin.panter | create | |