Skip to content
This repository has been archived by the owner on Aug 12, 2019. It is now read-only.

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

Open
chikamichi opened this issue Jul 4, 2015 · 4 comments
Open

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

chikamichi opened this issue Jul 4, 2015 · 4 comments

Comments

@chikamichi
Copy link

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 evert added the bug label Jul 5, 2015
@evert
Copy link
Member

evert commented Jul 5, 2015

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
Copy link
Member

Hywan commented Jul 5, 2015

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.

@Hywan Hywan self-assigned this Jul 5, 2015
@chikamichi
Copy link
Author

Sounds like a plan, @Hywan

@Hywan
Copy link
Member

Hywan commented Jul 5, 2015

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

@Hywan Hywan added the ready label Jul 17, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants