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

Change default DB from MySQL placeholder to valid SQLite #19

Closed
wants to merge 1 commit into from

Conversation

liilac
Copy link
Contributor

@liilac liilac commented Jun 20, 2016

See: #11 for discussion; #17 adds schema and instructions without making SQLite the default

  • has zero impact on already set up sites
  • at the moment all MySQL settings in settings.inc.php are placeholders
  • MySQL requires substantially more setup than SQLite
  • SQLite simplifies pomf set up
  • instructions on how to set up SQLite (very simple) are covered in Added instructions and schema to use or migrate to SQLite #17
  • no instructions on setting up MySQL provided

@jithatsonei
Copy link
Member

this is implicating #17 is merged, which it has yet to be, so I will close it for now until that changes in the near future

@liilac
Copy link
Contributor Author

liilac commented Jun 22, 2016

@ewhal @alucard0134

Justification for closing was perceived reliance on open PR. That PR has since been merged, so that objection is no longer relevant.

@jithatsonei
Copy link
Member

jithatsonei commented Jun 23, 2016

yep, reopened.

@jithatsonei jithatsonei reopened this Jun 23, 2016
@jithatsonei
Copy link
Member

..Then closed, we feel that MySQL will be the way to go for now in terms of the example configuration provided.

Thanks tho <3

Closing #11 also

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

2 participants