Skip to content

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