-
Notifications
You must be signed in to change notification settings - Fork 23.8k
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
Fix skeleton issue in user module #75152
Conversation
@@ -1218,6 +1218,9 @@ def modify_user(self): | |||
|
|||
def create_homedir(self, path): | |||
if not os.path.exists(path): | |||
if path == "/dev/null": |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please create a changelog fragment. See this fragment as an example. Also, write a test for this change here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is changing detection on home dir not on skeleton (see a few lines below) also you will need to do this for more than just this version of create_homedir.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
needs fixes
closing due to inactivity and #75948 seems to be closer to a mergable solution |
SUMMARY
Fixes #75063
ISSUE TYPE