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

Support native docker run instead of using the launcher script #47

Merged
merged 6 commits into from Apr 12, 2018

Conversation

ZmingZinnia
Copy link
Contributor

No description provided.

@ZmingZinnia ZmingZinnia changed the title Change launcher Support native docker run instead of using the launcher script Apr 3, 2018
@ZmingZinnia ZmingZinnia force-pushed the change_launcher branch 2 times, most recently from 5973dfe to d1286ba Compare April 3, 2018 10:13
@ZmingZinnia ZmingZinnia force-pushed the change_launcher branch 4 times, most recently from 1952923 to baf294d Compare April 4, 2018 02:31
@ZmingZinnia ZmingZinnia force-pushed the change_launcher branch 8 times, most recently from a10d1fa to aef7474 Compare April 9, 2018 06:27
README.md Outdated

#### Password and plan B
#### Password and emergency plan

Copy link
Contributor

Choose a reason for hiding this comment

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

"emergency plan" 有点不适合, 应该说 "custom admin username and password"

README.md Outdated

View the container logs: `sudo ./launcher logs`
The Docker repository will always contain the latest built version at: https://hub.docker.com/r/seafileltd/seafile/, you should not need to build the base image.
Copy link
Contributor

Choose a reason for hiding this comment

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

关于 /image 的都从 README.md 中去掉吧, README.dev.md 中有就行了

README.md Outdated

#### `/image`
```
docker rm seafile-server
Copy link
Contributor

Choose a reason for hiding this comment

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

正在运行中的 container 需要 docker rm -f seafile-server

launcher Outdated
@@ -1,432 +0,0 @@
#!/bin/bash
Copy link
Contributor

Choose a reason for hiding this comment

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

#!/bin/bash

echo "=============================================================================="
echo "The launcher script is now deprecated. Please see https://github.com/haiwen/seafile-docker/blob/master/upgrade_from_old_format.md"
echo "=============================================================================="
echo
echo "Or run this command directly:"
echo
echo "   docker rm -f seafile"
echo "   docker run -d -it --name seafile -v $(dirname $PWD)/shared:/shared -p 80:80 -p 443:443 seafileltd/server"
echo
echo "=============================================================================="

@ZmingZinnia ZmingZinnia force-pushed the change_launcher branch 2 times, most recently from 11ea776 to 0bd8bd3 Compare April 12, 2018 05:32
@lins05 lins05 merged commit 9c3fdca into master Apr 12, 2018
@lins05 lins05 deleted the change_launcher branch April 12, 2018 09:12
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.

None yet

3 participants