Skip to content

Conversation

grimreaper
Copy link
Contributor

autotools as a mechanism to detect that the functions it detects works,
not just exists. For these functions, use those macros.

@grimreaper grimreaper force-pushed the eax/build/workingauto branch 4 times, most recently from 273c5a4 to 1270803 Compare May 19, 2018 00:22
@grimreaper grimreaper force-pushed the eax/build/workingauto branch 2 times, most recently from e967572 to 998b876 Compare May 25, 2018 14:19
@grimreaper grimreaper force-pushed the eax/build/workingauto branch from 998b876 to 6db57ee Compare June 9, 2018 17:46
@pablogsal
Copy link
Member

Please, open an issue in http://bugs.python.org and reference that issue number in the pull request title. More info about the workflow:

https://devguide.python.org/pullrequest/#submitting

@pablogsal
Copy link
Member

The CI is failing with:

Traceback (most recent call last):
  File "/home/travis/build/python/cpython/Lib/test/test_logging.py", line 3389, in test_time
    r.created = time.mktime(dt.astimezone(None).timetuple())
AttributeError: module 'time' has no attribute 'mktime'
``

@grimreaper grimreaper force-pushed the eax/build/workingauto branch 2 times, most recently from 64d631c to 982f148 Compare June 29, 2018 14:32
autotools as a mechanism to detect that the functions it detects works,
not just exists. For these functions, use those macros.
@grimreaper grimreaper force-pushed the eax/build/workingauto branch from 982f148 to 5c78280 Compare July 28, 2018 07:42
@csabella
Copy link
Contributor

Thank you for your interest in improving CPython. This is not a trivial change and would require an issue to be created on bugs.python.org, as previously requested. If you are interested in pursuing this, please open an issue on the bug tracker and reopen this pull request with the bpo number in the title as per the devguide. Also, please resolve the merge conflicts once reopened. Thanks!

@csabella csabella closed this May 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants