Skip to content
This repository has been archived by the owner on Sep 8, 2023. It is now read-only.

Master updating runtimes nov2017 #80

Conversation

ivannaranjo
Copy link
Contributor

Adding the new images for the November release of the .NET Core runtime.

Fixes #79

@ivannaranjo
Copy link
Contributor Author

Ping.

Copy link

@ILMTitan ILMTitan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure why you are renaming the temp_builders_root folder, but otherwise LGTM.

@@ -37,7 +37,7 @@ else
fi

# Where to store the modified app definitions.
readonly temp_builders_root=$(mktemp -d -t test_run)
readonly temp_builders_root=$(mktemp -d -t test_run.XXXX)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure why you are adding to this filename.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The mktemp tool requires multiple X characters so it can generate a unique name in Linux. It does not require this on Mac. I was running this script on my Linux box when I got the warning.

@ivannaranjo ivannaranjo merged commit 0cb2f9a into GoogleCloudPlatform:master Nov 28, 2017
@ivannaranjo ivannaranjo deleted the master_updating_runtimes_nov2017 branch November 28, 2017 11:02
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