I used to think these sorts of mistakes were purposeful to make you think about stuff and learn, I see them all the time like the op doesn’t test the code because they know what they are doing and simply wrote an article… Good thing you are onto it ;)

@Frank

We will review the guide but you can always add your code to the script to make it better. Thanks for sharing your thoughts with us.

This is flawed on very many levels. Besides what was already mentioned:

• from man 7 udev: “Running an event process for a long period of time may block all further events for this or a dependent device”.
• /bin/autobackup.sh has no error checking at all (and should not be in /bin/, but rather /usr/local/bin/ or …/sbin/).
• it is an add-only backup, no cleanup of old files. However, if you extend that approach, have a look at rsync’s –link-dest option.
• compressing locally transferred data (rsync’s -z option) will just waste CPU cycles.

I’d suggest taking that offline until it is re-worked. Good luck!

Kind regards,
Frank.

Still not fixed. Remove this shit before somebody loses their data.

@ABSOLUTE_BEGINNER

The corrections you were concerned about have now been made, thanks for the feedback.