Skip to content

posix: fnmatch: fix known bugs in fnmatch() #90447

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nimisht
Copy link

@nimisht nimisht commented May 24, 2025

see issue #55186

all tests in tests/posix/c_lib_ext/src/fnmatch.c should pass except one (which is still commented)

Copy link

Hello @nimisht, and thank you very much for your first pull request to the Zephyr project!
Our Continuous Integration pipeline will execute a series of checks on your Pull Request commit messages and code, and you are expected to address any failures by updating the PR. Please take a look at our commit message guidelines to find out how to format your commit messages, and at our contribution workflow to understand how to update your Pull Request. If you haven't already, please make sure to review the project's Contributor Expectations and update (by amending and force-pushing the commits) your pull request if necessary.
If you are stuck or need help please join us on Discord and ask your question there. Additionally, you can escalate the review when applicable. 😊

@nimisht nimisht marked this pull request as ready for review May 24, 2025 22:24
@github-actions github-actions bot added the area: POSIX POSIX API Library label May 24, 2025
@github-actions github-actions bot requested review from cfriedt and ycsin May 24, 2025 22:24
cfriedt
cfriedt previously approved these changes May 25, 2025
Copy link
Member

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

…5186

all tests in tests/posix/c_lib_ext/src/fnmatch.c seem to pass
except one (which is still commented)

Signed-off-by: Nimish Tambe <nimisht@gmail.com>
@nimisht nimisht force-pushed the posix_fnmatch_fix_known_bugs_55186 branch from 635a61d to ccce1c3 Compare May 25, 2025 05:49
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: POSIX POSIX API Library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants