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

Fix mysql import in nextcloud readme #2210

Merged
merged 3 commits into from
Oct 5, 2022
Merged

Fix mysql import in nextcloud readme #2210

merged 3 commits into from
Oct 5, 2022

Conversation

TheDevPanda
Copy link
Contributor

@TheDevPanda TheDevPanda commented Oct 1, 2022

[README] In docker-compose exec db sh -c "mysql -u USER -p PASSWORD nextcloud < /dmp":
Replaced -p with --password because of: 'If you use the short option form (-p), you cannot have a space between the option and the password.' - mysql man page.
Also replaced -u with --user for consistency.

This fix is necessary since the command otherwise failes because of a wrong password.

Replacing -p with --password because of: 'If you use the short option form (-p), you cannot have a space between the option and the password.' - mysql man page
@TheDevPanda
Copy link
Contributor Author

I moved the changes from the readme.md to content.md as per workflow feedback.

@tianon
Copy link
Member

tianon commented Oct 5, 2022

cc @tilosp @J0WI

J0WI added a commit to nextcloud/docker that referenced this pull request Oct 5, 2022
docker-library/docs#2210

Signed-off-by: J0WI <J0WI@users.noreply.github.com>
@yosifkit yosifkit merged commit 0175897 into docker-library:master Oct 5, 2022
ananace pushed a commit to ananace/docker-nextcloud that referenced this pull request May 10, 2024
docker-library/docs#2210

Signed-off-by: J0WI <J0WI@users.noreply.github.com>
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