FYI, force pushed to kick off stalled checks.

FYI, force pushed to kick off stalled checks.

I tested the path on Catalina and its works well. Thanks!

Thank you, @remilapeyre !

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

I have made the requested changes; please review again.

FYI: test_peg_generator fails on my machine (master HEAD => 4044c84). I pushed this to GitHub to see if the CI also fails.

Thanks for making the requested changes!

@ned-deily: please review the changes made to this pull request.

Ref. my previous comment (and completely irrelevant to this PR): I see that the CI has no problems with test_peg_generator, so that problem is on my side.

BTW, detect_socket() in setup.py could be restructured a little bit. Something like this would be cleaner:

    def detect_socket(self):
        # socket(2)
        kwargs = {'depends': ['socketmodule.h']}
        if VXWORKS:
            kwargs['libraries'] = ['net']
        elif MACOS:
            # Issue #35569: Expose RFC 3542 socket options.
            kwargs['extra_compile_args'] = ['-D__APPLE_USE_RFC_3542']

        self.add(Extension('_socket', ['socketmodule.c'], **kwargs))

I guess that would need to go in a separate issue?

Your restructered detect_socket() looks nicer but, unless I'm missing something, it doesn't seem to behave the same as the original code. The current code doesn't attempt to build _socket on VxWorks if it can't find the net lib directory, no? Let's go with how the PR stands currently.

Thanks, @ned-deily !

Your restructered detect_socket() looks nicer but, unless I'm missing something, it doesn't seem to behave the same as the original code. The current code doesn't attempt to build _socket on VxWorks if it can't find the net lib directory, no?

You're right, I mistook the elif for an else!

This should do the trick:

    def detect_socket(self):
        # socket(2)
        if VXWORKS:
            if not self.compiler.find_library_file(self.lib_dirs, 'net'):
                return
            kwargs = {'libraries': ['net']}
        elif MACOS:
            # Issue #35569: Expose RFC 3542 socket options.
            kwargs = {'extra_compile_args': ['-D__APPLE_USE_RFC_3542']}
        else:
            kwargs = {}

        kwargs['depends'] = ['socketmodule.h']
        self.add(Extension('_socket', ['socketmodule.c'], **kwargs))

Or we could just initialize kwargs first, and drop the last else