Skip to content

fix: built-in awk strcmp check#1470

Merged
jart merged 1 commit into
jart:masterfrom
OEvgeny:fix/awk-exec
Jan 25, 2026
Merged

fix: built-in awk strcmp check#1470
jart merged 1 commit into
jart:masterfrom
OEvgeny:fix/awk-exec

Conversation

@OEvgeny

@OEvgeny OEvgeny commented Jan 20, 2026

Copy link
Copy Markdown

A small fix for the built-in awk strcmp check I found while wandering through the sources.

@github-actions github-actions Bot added the libc label Jan 20, 2026

@jart jart left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thank you!

@jart jart merged commit cc5f7de into jart:master Jan 25, 2026
1 check passed
@OEvgeny OEvgeny deleted the fix/awk-exec branch January 25, 2026 02:24
@jart

jart commented Jan 25, 2026

Copy link
Copy Markdown
Owner

I'm sorry this project does not accept anonymous contributions, as mentioned in CONTRIBUTING.md. I've removed this merge. It's important to me that when people see this project and look at the git log that they know real people wrote it. I don't believe there's any legitimate reason to do STEM anonymously unless you're Satoshi Nakamoto and even he ended up regretting it. Because I made a mistake in hastily merging this I'm going to need to remove you from the project unless you're open to fixing this, since contributor status was incorrectly granted which would allow you to bypass this review process in the future. Please reach out to jtunney@gmail.com and we'll discuss further.

@OEvgeny OEvgeny restored the fix/awk-exec branch January 25, 2026 03:02
@OEvgeny

OEvgeny commented Jan 25, 2026

Copy link
Copy Markdown
Author

@jart contacted by email, sorry for confusion. I thought sharing my name is enough, committed the charge using GitHub code spaces, so it should contain my personal info.

Anyways I'm not posting this change to gather any credit. Just thought PR is better than issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants