mirror of
https://github.com/extremeshok/xshok-proxmox.git
synced 2026-04-25 13:45:48 +03:00
[GH-ISSUE #8] install-post.sh bug #5
Labels
No labels
enhancement
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/xshok-proxmox#5
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @stirch on GitHub (Feb 26, 2019).
Original GitHub issue: https://github.com/extremeshok/xshok-proxmox/issues/8
Hi!
This script failed with such error on fresh installed proxmox
Executing: /lib/systemd/systemd-sysv-install enable fail2ban
vm.swappiness = 10
vm.swappiness = 10
vm.min_free_kbytes = 524288
install-post.sh: line 334: syntax error: unexpected end of file
After removing from script the part about ## Pretty MOTD BANNER all working how intended
@JOduMonT commented on GitHub (Apr 15, 2019):
@Orfeous could you decompose the script and run command by command those
because it is the way I'm doing it and I don't any have error but as @stirch mentioned I also don't execute the MOTD BANNER
@Orfeous commented on GitHub (Apr 16, 2019):
@jodumont Sure, will do that.
To bypass the issue i just basically trashed out the whole MOTD part from the script.
For the first look it seems like an identication issue around EOF for the file contents. For some reason the script thinks that one is its own EOF and fails on it.
I'll show you my version and if you like it i create a pr as well.
@extremeshok commented on GitHub (Apr 27, 2019):
@Orfeous please create a PR or post the code change here.
Thanks
@Orfeous commented on GitHub (May 1, 2019):
It gave me some headache because i wanted to fix it by keeping the script indentation intact.
At the end i gave up and just "break it" by removing it and adding some space to fix the here-document part. See #10
Now it works. :)