Skip to content

yes: remove GNU extension #455

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

Merged
merged 2 commits into from
Feb 13, 2024
Merged

yes: remove GNU extension #455

merged 2 commits into from
Feb 13, 2024

Conversation

mknos
Copy link
Contributor

@mknos mknos commented Feb 12, 2024

  • Align usage with Free/Net/Open/DragonFly BSDs and Mac-osx
  • Command options are not supported: "yes --hey" prints "--hey"
  • String printed is the 1st command argument; extra arguments are ignored: "yes a b" prints "a"

* Align usage with Free/Net/Open/DragonFly BSDs and Mac-osx
* Command options are not supported: "yes --hey" prints "--hey"
* String printed is the 1st command argument; extra arguments are ignored: "yes a b" prints "a"
@github-actions github-actions bot added Type: enhancement improve a feature that already exists Priority: low get to this whenever Program: yes The yes program labels Feb 12, 2024
Copy link
Owner

@briandfoy briandfoy left a comment

Choose a reason for hiding this comment

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

Since you are completely replacing the source, you should give yourself credit in the metadata too.

@briandfoy briandfoy added the Status: changes requested adjust the pull request as noted in comments label Feb 12, 2024
@briandfoy briandfoy merged commit 950971d into briandfoy:master Feb 13, 2024
@briandfoy briandfoy added Status: accepted The fix is accepted and removed Priority: low get to this whenever Status: changes requested adjust the pull request as noted in comments labels Feb 13, 2024
@briandfoy briandfoy self-assigned this Feb 14, 2024
@briandfoy
Copy link
Owner

changes: remove GNU extensions (we are a BSD clone)

@briandfoy briandfoy added Status: released there is a new release with this fix and removed Status: accepted The fix is accepted labels Mar 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Program: yes The yes program Status: released there is a new release with this fix Type: enhancement improve a feature that already exists
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants