-
Notifications
You must be signed in to change notification settings - Fork 15
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
makeservices: add easydoku script #111
base: master
Are you sure you want to change the base?
Conversation
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.
Thanks for doing this.
I wonder if we could perhaps make the script more resilient to intermediate failure, e.g. deleting half-extracted directories if the extraction fails...
Thanks for the feedback, I've made the changes. I don't think these failure cases are a huge deal, but if this gets used a lot and we see users confused by it, we should revisit. |
Structured similarly to the easywp script, this installs DokuWiki. It includes a prompt for subfolder, since some groups might want this at a separate URL. Co-authored-by: Kevin Peng <[email protected]>
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.
wfm with the noted change. I wish they would provide GPG signatures on their .zip
s so we don't have to just hope that the site isn't compromised lol. oh well
Co-Authored-By: Christopher Cooper <[email protected]>
DokuWiki will be packaged in Buster, which means eventually we might not need to download from their website, with the added bonus that it'll be updated and managed by us, assuming we can get it to work with our Apache userdir setup. |
It looks to me like this probably could just be merged? I guess we could wait for bullseye, but it may be a while before tsunami is on bullseye... |
I'm probably OK with this being merged today. The OCF doesn't really have a precedent of using Debian packages for vhost software (though in theory we could do this for wordpress). Anyone want to test it quickly one last time? |
Yeah I can test it in a little while. |
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 looks great!
Edit: oh but it looks like there is no symlink in one of the bin directories.
Also for some reason downloading docuwiki was at 100Kb/s, so we may want to consider mirroring it if possible. |
Structured similarly to the easywp script, this installs DokuWiki. It includes a prompt for subfolder, since some groups might want this at a separate URL.
Can someone please test this? It wfm but I may have missed something important.