[issue29527] Travis: doc job is broken

[STINNER Victor]

Changes by STINNER Victor :


--
pull_requests: +716

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue29527] Travis: doc job is broken

[STINNER Victor]

STINNER Victor added the comment:


New changeset 0d5f11061a873e9fb67ae59e46b3313e5ba22fc3 by Victor Stinner in 
branch 'master':
Don't treat warnings as error in Travis docs job (#7)
https://github.com/python/cpython/commit/0d5f11061a873e9fb67ae59e46b3313e5ba22fc3


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue29527] Travis: doc job is broken

[STINNER Victor]

STINNER Victor added the comment:

I see that Travis CI now compiles the doc to HTML, there is a single warning 
and the doc works. So I close the issue. Thanks!

If you want to upgrade Sphinx, fix more warnings, etc. : please open new more 
specific issues for these tasks.

--
resolution:  -> fixed
stage:  -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue29527] Travis: doc job is broken

[Zachary Ware]

Zachary Ware added the comment:

PR16 made the Travis build situation better, but we still have some warnings to 
look into.  I thought we had a PR for that, but I can't seem to find it.

--
nosy: +zach.ware
pull_requests: +34

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue29527] Travis: doc job is broken

[Brett Cannon]

Brett Cannon added the comment:

It really depends on why the warnings are there. If it's a configuration issue 
with how Misc/NEWS is being built then that can be fixed to deal with a lot of 
warnings upfront. Otherwise we can also strip out the references from Misc/NEWS 
to deal with the warnings.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue29527] Travis: doc job is broken

[Ryan Gonzalez]

Ryan Gonzalez added the comment:

Trying to fix this in https://github.com/python/cpython/pull/9...but...

Currently there are approx *pause for effect* 6,245 warnings!

Out of those, around 6,243 are 'reference target not found' warnings, spanning 
over 290 files:


/media/ryan/stuff/cpython/Doc/whatsnew/3.7.rst:121: WARNING: c:member reference 
target not found: doc
/media/ryan/stuff/cpython/Doc/whatsnew/3.7.rst:121: WARNING: c:type reference 
target not found: PyGetSetDef
/media/ryan/stuff/cpython/Doc/whatsnew/3.7.rst:121: WARNING: c:type reference 
target not found: wrapperbase
/media/ryan/stuff/cpython/Doc/whatsnew/3.7.rst:157: WARNING: py:mod reference 
target not found: ntpath
../../Misc/NEWS:348: WARNING: py:data reference target not found: 
socket.TCP_CONGESTION
../../Misc/NEWS:348: WARNING: py:data reference target not found: 
socket.TCP_USER_TIMEOUT
../../Misc/NEWS:1005: WARNING: py:data reference target not found: 
socket.TCP_CONGESTION
../../Misc/NEWS:1005: WARNING: py:data reference target not found: 
socket.TCP_USER_TIMEOUT
../../Misc/NEWS:2718: WARNING: py:obj reference target not found: idlelib/*.py
../../Misc/NEWS:2718: WARNING: py:obj reference target not found: 
idle_test/test_*.py
../../Misc/NEWS:2837: WARNING: c:func reference target not found: malloc
../../Misc/NEWS:2883: WARNING: c:func reference target not found: 
PyObject_Malloc


For some reason, I don't feel like fixing 6,200 warnings is really feasible (or 
at least for me), so would it be worth it to, in the mean time, disable this 
specific kind of warning, at least for the CI builds?

--
nosy: +refi64

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue29527] Travis: doc job is broken

[STINNER Victor]

STINNER Victor added the comment:

> And we shouldn't disable the check entirely if fixing it is difficult, but 
> instead just let warnings be warnings until we fix all warnings and then make 
> them be errors again.

Ok, I changed my change (I got the same proposal in the PR). So I removed -W 
option:
https://github.com/python/cpython/commit/0d5f11061a873e9fb67ae59e46b3313e5ba22fc3

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue29527] Travis: doc job is broken

[Brett Cannon]

Brett Cannon added the comment:

What would it take to simply fix the docs?

And we shouldn't disable the check entirely if fixing it is difficult, but 
instead just let warnings be warnings until we fix all warnings and then make 
them be errors again.

--
nosy: +brett.cannon

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue29527] Travis: doc job is broken

[STINNER Victor]

New submission from STINNER Victor:

I suggest to disable the job until it's fixed, just to fix Travis right now. 
This issue will track the bug to find a solution.

https://travis-ci.org/python/cpython/jobs/200517775

$ make html SPHINXBUILD="./venv/bin/python3 -m sphinx" SPHINXOPTS="-nW -q -b 
linkcheck"
./venv/bin/python3 -m sphinx -b html -d build/doctrees -D latex_paper_size= -nW 
-q -b linkcheck . build/html 

Warning, treated as error:
WARNING: latex_preamble is deprecated. Use latex_elements['preamble'] instead.

make: *** [build] Error 1

--
components: Tests
messages: 287573
nosy: haypo
priority: normal
severity: normal
status: open
title: Travis: doc job is broken
versions: Python 3.7

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com