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.

Sign up for GitHub

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

Jump to bottom

config: support default user on IBM i #65

Merged
merged 1 commit into from
Dec 10, 2018
Merged

config: support default user on IBM i #65

merged 1 commit into from
Dec 10, 2018

Conversation

dmabupt
Copy link
Contributor

@dmabupt dmabupt commented Sep 3, 2018

setuid() can not accept a 'nobody' parameter on IBM i.
The default user (QSECOFR) on IBM i has user id 0.

@dmabupt
config: support default user on IBM i
88105d7 
setuid() can not accept a 'nobody' parameter on IBM i.
The default user (QSECOFR) on IBM i has user id 0.
@dmabupt dmabupt requested a review from a team as a code owner September 3, 2018 07:22
@dmabupt dmabupt mentioned this pull request Oct 8, 2018
Closed
4 tasks
@zkat zkat added the semver:patch semver patch level for changes label Nov 13, 2018
@zkat zkat changed the base branch from latest to release-next November 13, 2018 14:53
zkat
zkat approved these changes Nov 13, 2018
View reviewed changes
Copy link
Contributor

@zkat zkat left a comment

Choose a reason for hiding this comment

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

This seems reasonable enough...

@zkat zkat merged commit 1065a78 into npm:release-next Dec 10, 2018
@aeschright aeschright mentioned this pull request Jan 17, 2019
Closed
4 tasks
aeschright pushed a commit that referenced this pull request Jan 18, 2019
@dmabupt @aeschright
config: support default user on IBM i (#65)
3053a75 
setuid() can not accept a 'nobody' parameter on IBM i.
The default user (QSECOFR) on IBM i has user id 0.

PR-URL: #65
Credit: @dmabupt
Reviewed-By: @zkat
@zkat zkat mentioned this pull request Jan 29, 2019
Closed
4 tasks
@kevinjm39 kevinjm39 mentioned this pull request May 1, 2022
Open
This was referenced Sep 29, 2022
Open
Open
Open
@bumplzz69 bumplzz69 mentioned this pull request Oct 9, 2022
Open
@gutwrinch gutwrinch mentioned this pull request Jan 6, 2024
Open
@Kingcrypto38 Kingcrypto38 mentioned this pull request Jan 11, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zkat zkat zkat approved these changes

Assignees
No one assigned
Labels
semver:patch semver patch level for changes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@dmabupt @zkat