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

README.md: expand credit for original #43

Merged
merged 2 commits into from
Feb 14, 2024
Merged

Conversation

IanDarwin
Copy link
Contributor

Sudo was invented on Berkeley Unix on a VAX, and the current version is maintained by Todd Miller of the OpenBSD Unix project. It is thus correct to refer to the original as Unix/Linux as it is not a project originated by the Linux developers.

Sudo was invented on Berkeley Unix on a VAX, and the current version is maintained by Todd Miller of the OpenBSD Unix project. It is thus correct to refer to the original as Unix/Linux as it is not a project originated by the Linux developers.
@joadoumie joadoumie self-requested a review February 12, 2024 17:10
@omnivagant
Copy link

Better, but I would call the sudo project just "the sudo project", with a reference to the sudo project and with less occurances of "Unix/Linux" in the README because it gets pretty repetitive.

Suggested by "omni" to have 'less occurrences of "Unix/Linux" in the README' - reads better this way.
@IanDarwin
Copy link
Contributor Author

@omnivagant Point taken, 2nd commit revises accordingly. Thanks.

Copy link
Collaborator

@joadoumie joadoumie left a comment

Choose a reason for hiding this comment

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

Looks good to me - thanks!

@joadoumie joadoumie merged commit 72d6ce5 into microsoft:main Feb 14, 2024
1 check 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.

None yet

4 participants