Skip to content

Conversation

@owlhuang
Copy link
Contributor

@owlhuang owlhuang commented Feb 3, 2024

Pass the check if any one of the library (of the arch) satisfies the requirement.

Fixes microsoft/vscode-remote-release#9475

@owlhuang
Copy link
Contributor Author

owlhuang commented Feb 3, 2024

@microsoft-github-policy-service agree

@gjsjohnmurray
Copy link
Contributor

Pinging @deepak1556

@lszomoru lszomoru assigned deepak1556 and unassigned lszomoru Feb 5, 2024
Copy link
Collaborator

@deepak1556 deepak1556 left a comment

Choose a reason for hiding this comment

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

Makes sense, thanks!

@deepak1556 deepak1556 added this to the February 2024 milestone Feb 5, 2024
@n8falke
Copy link

n8falke commented Feb 5, 2024

The loop over libstdc++.so.6 works on my system. Looks good for me. Thanks.

I've no system with more than one libc.so.6 (which is probably very problematic).

@deepak1556
Copy link
Collaborator

Thanks for confirming!

@owlhuang
Copy link
Contributor Author

owlhuang commented Feb 6, 2024

@deepak1556 The suggestions all look good to me. Do I need to commit the suggestions or resolve the comments?

@deepak1556
Copy link
Collaborator

Feel free to do either, commit suggestions from github or make the changes locally and commit them (which will also resolve the comments)

deepak1556
deepak1556 previously approved these changes Feb 6, 2024
Copy link
Collaborator

@deepak1556 deepak1556 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!

@deepak1556 deepak1556 enabled auto-merge (squash) February 6, 2024 05:09
@owlhuang
Copy link
Contributor Author

owlhuang commented Feb 6, 2024

I like your exit checks. Easier to read! Thanks for all the efforts on VSCode!

DonJayamanne
DonJayamanne previously approved these changes Feb 6, 2024
@deepak1556 deepak1556 dismissed stale reviews from DonJayamanne and themself via e09b14f February 7, 2024 23:36
@deepak1556 deepak1556 merged commit f6a5654 into microsoft:main Feb 8, 2024
@owlhuang owlhuang deleted the patch-2 branch February 8, 2024 00:15
@microsoft microsoft locked and limited conversation to collaborators Jun 10, 2024
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.

check-requirements.sh fails when attaching to container

8 participants