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

Allow building on macos #4

Closed
wants to merge 3 commits into from
Closed

Conversation

mraerino
Copy link

@mraerino mraerino commented Dec 14, 2020

Issue #, if available: awslabs/aws-lambda-cpp#52

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@carlzogh
Copy link
Contributor

carlzogh commented Apr 1, 2021

Hey @mraerino - thanks for your contribution. However, I don't think this project is the right place to be making core changes to the aws-lambda-cpp project, especially since it seems that disabling builds on MacOS was a deliberate decision from the maintainers to avoid building and packaging functions on a non-Linux host when they are always going to run on Linux in Lambda.

Lambda only runs on Linux. So why bother with maintaining the code on macOS?
You can think of it as a fail-fast mechanism to prevent people from building and packaging a Lambda on macOS only to then see it fail when deployed.

see #34 for a similar question/answer.

ref. awslabs/aws-lambda-cpp#52 (comment)

@carlzogh carlzogh closed this Apr 1, 2021
@mraerino
Copy link
Author

mraerino commented Apr 1, 2021

that's fine. #8 would be the better approach imho

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants