CLI installer could check for access rights on /data/

sabre-io / katana

:hocho: A contact, calendar, task list and file server, synced, everywhere, all the time

http://sabre.io/katana/

Other

149 stars 22 forks source link

CLI installer could check for access rights on /data/ #280

Open chikamichi opened 9 years ago

chikamichi commented 9 years ago

Hi,

When using the CLI installer, nothing warns the user about access rights on /data/. The web installer has a few well-written guidelines on that topic. Maybe one could check whether access rights are fine prior to any other question, and if not, display some instructions about that and cancel the installation?

evert commented 9 years ago

Also wondering if we should just disable the cli installer altogether, but not sure about that one. Thoughts @DominikTo ? I think a reduced surface area will help increase quality.

Hywan commented 9 years ago

Since we have the Sabre\Katana\Installer class, we can move some other checks inside this class and run them from both Web and CLI installers. This is the current approach.

chikamichi commented 9 years ago

Sounds like a plan, @Hywan

Hywan commented 9 years ago

This is the actual landing plan. We just have to extend it a little bit more.