Skip to content

Fix bash syntax errors and typos from code review #3

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.

Sign up for GitHub

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

Draft
Copilot wants to merge 2 commits into
base: claude/split-vendor-files-011CV6EBs7W5mSUdhp91B5GD
Choose a base branch
from
Draft

Fix bash syntax errors and typos from code review #3

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
e02a6e5
Initial plan
Copilot Nov 14, 2025
11cd714
Fix review comments: variable syntax, typos, and safety checks
Copilot Nov 14, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 5 additions & 3 deletions primemover.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,9 @@
# You'll need to already have manually built your sites at RunCloud and have WordPress successfully running there BEFORE trying to move sites in from other sources.
# ServerPilot site build code (via API) is already built but needs to be reintegrated to this work.

source ~/.bash_profile
if [ -f ~/.bash_profile ]; then
source ~/.bash_profile
fi

if [[ $EUID -ne 0 ]]; then
echo "This script must be run as root, exiting!!!"
Expand Down Expand Up @@ -201,7 +203,7 @@ VerifySiteMigration() {

MeImCounting() {

echo "This is all very VERY aplha right now. Use at your own risk."
echo "This is all very VERY alpha right now. Use at your own risk."
echo " "
echo "All kinds of things might be broken. It's a work in progress and we'll get it hammered out shortly."
echo " "
Expand Down Expand Up @@ -508,7 +510,7 @@ PackageSite() {

elif [ $envir == "SP" ]
then
echo "Packaging local ServerPilot powered site $appname for user $D..."
echo "Packaging local ServerPilot powered site $appname for user $username..."

# Get to the choppa...
cd /srv/users/$username/apps/$appname/public
Expand Down
8 changes: 4 additions & 4 deletions vendors/runcloud.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,15 +56,15 @@ RCtoSP() {

if [ -f "/etc/php56rc/fpm.d/$appname.conf" ]
then
echo "PHP56RC file found... setting PHP to verison 5.6"
echo "PHP56RC file found... setting PHP to version 5.6"
php="php5.6"
elif [ -f "/etc/php70rc/fpm.d/$appname.conf" ]
then
echo "PHP70RC file found... setting PHP to verison 7.0"
echo "PHP70RC file found... setting PHP to version 7.0"
php="php7.0"
elif [ -f "/etc/php71rc/fpm.d/$appname.conf" ]
then
echo "PHP71RC file found... setting PHP to verison 7.1"
echo "PHP71RC file found... setting PHP to version 7.1"
php="php7.1"
else
echo "No PHP file found... defaulting to PHP7.0"
Expand Down Expand Up @@ -320,7 +320,7 @@ RCtoGP() {
# Get all RunCloud domains
rcDomains

$site_to_clone="ALL"
site_to_clone="ALL"

DoWork

Expand Down
2 changes: 1 addition & 1 deletion vendors/serverpilot.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -485,7 +485,7 @@ SPtoGP() {
# Get all ServerPilot domains
spDomains

$site_to_clone="ALL"
site_to_clone="ALL"

DoWork

Expand Down