Skamasle / sk-import-cpanel-backup-to-vestacp

Import cPanel backup in vestacp
36 stars 31 forks source link

Just a small fix adding the full path to some cli commands in a coupl… #5

Closed nebadom closed 6 years ago

nebadom commented 6 years ago

This is just a small fix on a couple of lines, to add the full path to some cli command.

Without that, it would trigger a "command not found" in my system (Ubunt/VestaCP 16.04)

Skamasle commented 6 years ago

Thanks this is permanent vesta + ubuntu issue, but can be solved with a reboot

nebadom commented 6 years ago

Good to know.

Also, it would be nice if the import could take the backup from another drive and not from root. My root disk is very small, and I already locked down my vesta testing the import process (thankfully I have snapchots so I could roll back to a good state).

In any case, excellent work pal. Trully a very helpfull tool. I whish I could contribute with something else.

On Tue, Jan 23, 2018 at 6:30 PM, Maksim Usmanov | Maks < notifications@github.com> wrote:

Thanks this is permanent vesta + ubuntu issue, but can be solved with a reboot

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Skamasle/sk-import-cpanel-backup-to-vestacp/pull/5#issuecomment-359937193, or mute the thread https://github.com/notifications/unsubscribe-auth/AJvdcCzt2aoE0lfbKSeBeiHfZcHsXgIAks5tNk-CgaJpZM4RqVkU .

-- Alberto Alexander Rodriguez MCSE | MCSA | MCTS | MCP https://www.mcpvirtualbusinesscard.com/VBCServer/nebadom/profile

UpWork https://www.upwork.com/freelancers/~015998fd67363847db?viewMode=1

Skamasle commented 6 years ago

Yeah, I do it just fot fun and some needs, so I put it all in root, i not think in nobody in that time just for, and not put some variable to configure paths.

I think you can SED file and modify /root/ by some orther path and maybe it work or edit it manully, maybe in next changes I dot some var to configure path/ easy