-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Migration Tool #5978
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
Closed
alvonellos
wants to merge
20
commits into
BookStackApp:development
from
alvonellos:feature/bookstack-migrate-clean
Closed
Migration Tool #5978
alvonellos
wants to merge
20
commits into
BookStackApp:development
from
alvonellos:feature/bookstack-migrate-clean
+30,128
−1
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…n with PATH setup, improve MySQL validation - Restore Python bookstack_migration.py multi-path .env discovery - Update AUTO_INSTALL_EVERYTHING.sh: - Install Java 8 (not default version) - Install Rust via rustup - Install Maven for Java builds - Set JAVA_HOME and add to PATH for persistence - Improve MySQL validation with connection test - Keep Python/Perl/C/Rust credential discovery consistent - All scripts check /var/www/bookstack/.env first (standard BookStack location) - Fallback to /var/www/html/.env, .env, ../.env, ../../.env
…emove duplicate migration-tool-rust/
… all canonical versions in bookstack-migration/
- Restored deleted DokuWikiExporter.java to bookstack-migration/tools/ - Added loadEnvFile() method to search for .env in standard locations: - /var/www/bookstack/.env (standard) - /var/www/html/.env - .env, ../.env, ../../.env (relative) - Updated run() method to use .env credentials when CLI args not provided - Command-line arguments take precedence over .env values - Now all migration tools (Perl, Python, C, Rust, Java, PHP) have consistent .env discovery
…ent) - New install_perl_modules() function checks for required modules - Tries cpanm first (faster), falls back to cpan, then sudo cpanm - Runs automatically on script startup - Logs results with Sméagol commentary - Graceful error handling with manual install instructions if needed - All modules checked: DBI, DBD::mysql, JSON, LWP::UserAgent
- Add .github/copilot-instructions.md with comprehensive development guide - Reorganize migration tools into .github/migration/ directory - Add migration documentation, tests, and stage-based workflow - Clean up old migration scripts and temporary files meg
Member
|
I'll assume this is an accidental PR, and isn't intended to be merged into core, therefore I'll close this off. |
Author
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.