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

easy install from PEAR channel #3

Closed
cyberwolf opened this issue Apr 2, 2012 · 1 comment
Closed

easy install from PEAR channel #3

cyberwolf opened this issue Apr 2, 2012 · 1 comment

Comments

@cyberwolf
Copy link

It would be nice if this tool could be very easily installed on a new server, for example from a PEAR channel.

@nicjansma
Copy link
Owner

Sorry it's taken me so long to respond to this.

I'm not sure the script is suitable for PEAR. There are several places in the script that are TODOs -- mostly configuration options such as your DB login, but also an important section regarding dropping/readding INDEXes and ENUM support. The script source / comments document this pretty well, but as a PEAR module, I would need to add interfaces for the configuration, to hook into the script to add support for dropping/readding INDEXes, and the other TODOs

As a PEAR module, it would also highly discourage folks from looking at the source code, which I think is undesirable as you should really understand what you're going to run before making such a drastic change on your database.

Were there other reasons that you thought it would be well suited as a PEAR module?

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

No branches or pull requests

2 participants