Skip to content

improve the run_envoy_docker.sh compatibility on mac.#23832

Merged
phlax merged 1 commit intoenvoyproxy:mainfrom
doujiang24:ci-docker-mac
Nov 4, 2022
Merged

improve the run_envoy_docker.sh compatibility on mac.#23832
phlax merged 1 commit intoenvoyproxy:mainfrom
doujiang24:ci-docker-mac

Conversation

@doujiang24
Copy link
Copy Markdown
Member

Commit Message:
Additional Description:
Since the stat on mac missing the -c option, i.e.
stat: illegal option -- c
usage: stat [-FLnq] [-f format | -l | -r | -s | -x] [-t timefmt] [file ...]

Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Since the stat on mac missing the -c option, i.e.
stat: illegal option -- c
usage: stat [-FLnq] [-f format | -l | -r | -s | -x] [-t timefmt] [file ...]

Signed-off-by: doujiang24 <doujiang24@gmail.com>
Copy link
Copy Markdown
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

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

lgtm, thanks @doujiang24

@phlax
Copy link
Copy Markdown
Member

phlax commented Nov 4, 2022

/backport

@repokitteh-read-only repokitteh-read-only Bot added the backport/review Request to backport to stable releases label Nov 4, 2022
@phlax phlax merged commit ca9de5a into envoyproxy:main Nov 4, 2022
@doujiang24 doujiang24 deleted the ci-docker-mac branch November 4, 2022 15:16
phlax pushed a commit to phlax/envoy that referenced this pull request Nov 7, 2022
Since the stat on mac missing the -c option, i.e.
stat: illegal option -- c
usage: stat [-FLnq] [-f format | -l | -r | -s | -x] [-t timefmt] [file ...]

Signed-off-by: doujiang24 <doujiang24@gmail.com>
Signed-off-by: Ryan Northey <ryan@synca.io>
phlax pushed a commit to phlax/envoy that referenced this pull request Nov 7, 2022
Since the stat on mac missing the -c option, i.e.
stat: illegal option -- c
usage: stat [-FLnq] [-f format | -l | -r | -s | -x] [-t timefmt] [file ...]

Signed-off-by: doujiang24 <doujiang24@gmail.com>
Signed-off-by: Ryan Northey <ryan@synca.io>
phlax pushed a commit that referenced this pull request Nov 8, 2022
Since the stat on mac missing the -c option, i.e.
stat: illegal option -- c
usage: stat [-FLnq] [-f format | -l | -r | -s | -x] [-t timefmt] [file ...]

Signed-off-by: doujiang24 <doujiang24@gmail.com>
Signed-off-by: Ryan Northey <ryan@synca.io>
phlax pushed a commit that referenced this pull request Nov 8, 2022
Since the stat on mac missing the -c option, i.e.
stat: illegal option -- c
usage: stat [-FLnq] [-f format | -l | -r | -s | -x] [-t timefmt] [file ...]

Signed-off-by: doujiang24 <doujiang24@gmail.com>
Signed-off-by: Ryan Northey <ryan@synca.io>
@phlax phlax removed the backport/review Request to backport to stable releases label Jan 9, 2023
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