Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

bpo-29527: Disable broken Travis docs job #7

Merged
merged 1 commit into from Feb 11, 2017
Merged

bpo-29527: Disable broken Travis docs job #7

merged 1 commit into from Feb 11, 2017

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Feb 10, 2017

Comment the job in .travis.yml until a fix is found.

See http://bugs.python.org/issue29527

@refi64
Copy link
Contributor

@refi64 refi64 commented Feb 10, 2017

You could always just remove the -W temporarily, too, that way flat-out broken errors will still get caught...

@vstinner
Copy link
Member Author

@vstinner vstinner commented Feb 10, 2017

You could always just remove the -W temporarily

Ok, let's try that. We can start without -W. And later, once all warnings are fixed, reenable -W. I'm now waiting for the busy Travis :-)

@brettcannon brettcannon self-requested a review Feb 11, 2017
@vstinner vstinner mentioned this pull request Feb 11, 2017
Merged
@vstinner vstinner merged commit 0d5f110 into python:master Feb 11, 2017
1 check was pending
@vstinner vstinner deleted the fix_travis branch Feb 11, 2017
@vstinner
Copy link
Member Author

@vstinner vstinner commented Feb 11, 2017

I had prefer to wait for a confirmation that Travis tests pass, but Travis is currently flooded of new jobs because CPython just moved to GitHub and many happy developpers just created new pull requests. So I pushed the change anyway.

native-api pushed a commit to native-api/cpython that referenced this issue Jun 5, 2018
@roseman
Merge pull request python#7 from native-api/roseman_7
7551744 
python#7287 (comment)
@shihai1991 shihai1991 mentioned this pull request Feb 18, 2020
Merged
@pablogsal pablogsal mentioned this pull request Jun 12, 2020
Closed
@shihai1991 shihai1991 mentioned this pull request Jun 23, 2020
Closed
tiran referenced this issue in tiran/cpython Nov 18, 2020
@tiran
Fix reference leak in _warnings.c:create_filter()
de6090b 
Direct leak of 57 byte(s) in 1 object(s) allocated from:

    #0 0x7fa6397cc667 in __interceptor_malloc (/lib64/libasan.so.6+0xb0667)
    #1 0x777579 in PyUnicode_New Objects/unicodeobject.c:1459
    #2 0x86aa4e in unicode_decode_utf8 Objects/unicodeobject.c:5129
    #3 0x8b6050 in PyUnicode_DecodeUTF8Stateful Objects/unicodeobject.c:5259
    #4 0x8b6050 in PyUnicode_FromString Objects/unicodeobject.c:2311
    #5 0x8b6050 in PyUnicode_InternFromString Objects/unicodeobject.c:15788
    #6 0x8f1e0b in create_filter Python/_warnings.c:67
    #7 0x8f1e0b in init_filters Python/_warnings.c:95
    #8 0x8f1e0b in _PyWarnings_InitState Python/_warnings.c:123
    python#9 0xa52178 in pycore_init_types Python/pylifecycle.c:704
    python#10 0xa52178 in pycore_interp_init Python/pylifecycle.c:760
    python#11 0xa5eab1 in pyinit_config Python/pylifecycle.c:807
    python#12 0xa5eab1 in pyinit_core Python/pylifecycle.c:970
    python#13 0xa60037 in Py_InitializeFromConfig Python/pylifecycle.c:1155
    python#14 0x47a842 in pymain_init Modules/main.c:66
    python#15 0x4802a2 in pymain_main Modules/main.c:698
    python#16 0x4802a2 in Py_BytesMain Modules/main.c:731
    python#17 0x7fa638a5b041 in __libc_start_main (/lib64/libc.so.6+0x27041)

Signed-off-by: Christian Heimes <christian@python.org>
nanjekyejoannah pushed a commit to nanjekyejoannah/cpython that referenced this issue Apr 19, 2022
@bors
Merge python#7
5b876e2 
7: Add warnings for sorting and comparison r=ltratt a=nanjekyejoannah

Most of the warnings are covered on the list sort method.

I added the missing warnings for the `cmp` and `__cmp__` method.

This replaces python#4 

Co-authored-by: Joannah Nanjekye <jnanjekye@python.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brettcannon brettcannon

Assignees
No one assigned
Labels
CLA signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@vstinner @refi64 @brettcannon @the-knights-who-say-ni