Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Allow for execution on Windows, although this breaks time-out handling. #21

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

Closed
wants to merge 1 commit into from
Closed

Conversation

southernsun
Copy link

by checking if SIGALRM exists we can determine if it is either running on UNIX or Windows. If it is running on Windows, ignore the timeout feature and just continue.

by checking if SIGALRM exists we can determine if it is either running on UNIX or Windows. If it is running on Windows, ignore the timeout feature and just continue.
@freedomofkeima
Copy link
Collaborator

Hi, thank you for your contribution.

Since AWS Lambda itself runs under Linux environment (See http://docs.aws.amazon.com/lambda/latest/dg/current-supported-versions.html), I guess it's better to output a warning if it is executed in Windows environment, so that users understand Timeout feature will not work. Even better, could you also update README file to include what kind of limitation is expected if python-lambda-local is used in Windows environment?

@yxd-hde
Copy link
Contributor

yxd-hde commented Sep 3, 2018

See: #32

@freedomofkeima
Copy link
Collaborator

Implemented in #32, released via version 0.1.6.

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.

3 participants