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

Add old repo reference to webui-user.bat #16765

Closed
wants to merge 1 commit into from

Conversation

silveroxides
Copy link
Contributor

Added so that user can install dev branch separately easily and reference old repo without having to look up all arguments

Checklist:

Added so that user can install dev branch separately easily and reference old repo.
@catboxanon
Copy link
Collaborator

catboxanon commented Jan 1, 2025

See https://github.com/AUTOMATIC1111/stable-diffusion-webui/wiki/Contributing#quirks

webui-user.bat is never to be edited

@catboxanon catboxanon closed this Jan 1, 2025
@silveroxides
Copy link
Contributor Author

See https://github.com/AUTOMATIC1111/stable-diffusion-webui/wiki/Contributing#quirks

webui-user.bat is never to be edited

@catboxanon in 7 days, that addition to the wiki which also was misspelled, will be 2 years old. This obsolete quirk in a wiki that is filled with inaccuracies can't seriously be used as a reference. Also nothing about the edit I made cause any changes to any functionalities. They are commented out lines and when compared to the equivalent webui-user.sh, it is severely lacking in any form of explanation for its use.

@w-e-w
Copy link
Collaborator

w-e-w commented Jan 2, 2025

unfortunately it is not an obsolete quick

it is a quick that we have to deal with due to the initial bad decision of having user use the same webui-user.bat file for the there local config

if this file is edited when (which lots of people have edited it) then when people try to update with gut pull it will fail fail

git pull
error: cannot pull with rebase: You have unstaged changes.
error: Please commit or stash them.

resulting in them unable to update
wose case if they have use sort of update script it may cause ther local webui-user.bat config to be overwritten

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