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

lua: don't strip the binary. #19820

Merged
merged 1 commit into from
Feb 7, 2022
Merged

Conversation

PiotrSikora
Copy link
Contributor

Signed-off-by: Piotr Sikora piotrsikora@google.com

@repokitteh-read-only
Copy link

As a reminder, PRs marked as draft will not be automatically assigned reviewers,
or be handled by maintainer-oncall triage.

Please mark your PR as ready when you want it to be reviewed!

🐱

Caused by: #19820 was opened by PiotrSikora.

see: more, trace.

@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Feb 4, 2022
@repokitteh-read-only
Copy link

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).
envoyproxy/dependency-shepherds assignee is @wrowe

🐱

Caused by: #19820 was opened by PiotrSikora.

see: more, trace.

@PiotrSikora
Copy link
Contributor Author

Stacked on #19818.

Signed-off-by: Piotr Sikora <piotrsikora@google.com>
@PiotrSikora PiotrSikora changed the title lua: don't strip when cross-compiling. lua: don't strip the binary. Feb 6, 2022
@PiotrSikora PiotrSikora marked this pull request as ready for review February 6, 2022 09:59
@PiotrSikora
Copy link
Contributor Author

I decided to always skip the strip command to make sure the code-path is always executed, instead of only doing it when corss-compiling. @lizan @moderation PTAL.

@moderation
Copy link
Contributor

/lgtm deps

I think this is OK but like @htuch to review

@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Feb 7, 2022
@lizan lizan merged commit cd1c4bc into envoyproxy:main Feb 7, 2022
joshperry pushed a commit to joshperry/envoy that referenced this pull request Feb 13, 2022
Signed-off-by: Piotr Sikora <piotrsikora@google.com>
Signed-off-by: Josh Perry <josh.perry@mx.com>
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.

4 participants