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

Compatibility #13936

Merged
merged 2 commits into from
Nov 26, 2023
Merged

Compatibility #13936

merged 2 commits into from
Nov 26, 2023

Conversation

cabelo
Copy link
Contributor

@cabelo cabelo commented Nov 10, 2023

Description

  • a simple description of what you're trying to accomplish
  • a summary of changes in code
  • which issues it fixes, if any

Screenshots/videos:

Checklist:

My small and humble collaboration, with tests on Fedora and openSUSE.
I tested the following job functions:

@AUTOMATIC1111
Copy link
Owner

wouldn't changing PATH from /usr/sbin to just /sbin break things that previously needed it to be /usr/sbin?

@cabelo
Copy link
Contributor Author

cabelo commented Nov 20, 2023

wouldn't changing PATH from /usr/sbin to just /sbin break things that previously needed it to be /usr/sbin?

wouldn't changing PATH from /usr/sbin to just /sbin break things that previously needed it to be /usr/sbin?

Hello @AUTOMATIC1111 , thanks for your attention.

I did this because I tested on debian, opensuse and fedora. Then I verified that ldconfig is not found in /usr/sbin/ in fedora or opensuse. Already in /bin it works in debian, fedora and opensuse.

@cabelo
Copy link
Contributor Author

cabelo commented Nov 24, 2023

wouldn't changing PATH from /usr/sbin to just /sbin break things that previously needed it to be /usr/sbin?
"No", that wouldn't break things. I tested it on all versions (debian, opensuse and fedora).

@j05070415
Copy link

j05070415 commented Nov 24, 2023 via email

@cabelo
Copy link
Contributor Author

cabelo commented Nov 24, 2023

谢谢,您的邮件我已收到!

感謝您的答覆!

@AUTOMATIC1111 AUTOMATIC1111 merged commit e441032 into AUTOMATIC1111:dev Nov 26, 2023
3 checks passed
@w-e-w w-e-w mentioned this pull request Dec 4, 2023
@w-e-w w-e-w mentioned this pull request Dec 16, 2023
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.

3 participants