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

chore(deps): add gcc and gcc-c++ to Dockerfile required for node-gyp #849

Merged
merged 2 commits into from
Jul 28, 2021

Conversation

ansgarm
Copy link
Member

@ansgarm ansgarm commented Jul 27, 2021

node-gyp is used by node-pty which is going to be used to test the interactive cdktf watch command

node-gyp is used by node-pty which is going to be used to test the interactive cdktf watch command
@ansgarm ansgarm requested a review from DanielMSchmidt July 27, 2021 21:06
…n Docker

The Asset construct failed to copy the local-asset.txt file while syntesizing after we upgraded to Node 14. It created the target file but then it blocked and the target file stayed empty. There were no visible permission errors on the source file local-asset.txt (it even worked with other files created by hand, which had the same permissions according to `ls -la` and `stat`) and it also worked on Windows and OS X. So this is going to stay a mystery for a now but we are happy that this works now
@DanielMSchmidt DanielMSchmidt merged commit d262eb3 into main Jul 28, 2021
@DanielMSchmidt DanielMSchmidt deleted the docker-node-gyp-deps branch July 28, 2021 11:55
@github-actions
Copy link
Contributor

I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you've found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants