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

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

Merged
merged 8 commits into from Feb 8, 2024

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
Contributor

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

resources/server/bin/helpers/check-requirements-linux.sh Outdated Show resolved Hide resolved
resources/server/bin/helpers/check-requirements-linux.sh Outdated Show resolved Hide resolved
resources/server/bin/helpers/check-requirements-linux.sh Outdated Show resolved Hide resolved
resources/server/bin/helpers/check-requirements-linux.sh Outdated Show resolved Hide resolved
resources/server/bin/helpers/check-requirements-linux.sh Outdated Show resolved Hide resolved
resources/server/bin/helpers/check-requirements-linux.sh Outdated Show resolved Hide resolved
@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
Contributor

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
Contributor

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
Contributor

@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
owlhuang and others added 8 commits February 8, 2024 08:31
Pass the check if one of the library (of the arch) satisfies the requirement.
@deepak1556 deepak1556 merged commit f6a5654 into microsoft:main Feb 8, 2024
6 checks passed
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.

check-requirements.sh fails when attaching to container
8 participants