pi_blink_ip icon indicating copy to clipboard operation
pi_blink_ip copied to clipboard

Refactor to use crontab libs

Open uresu opened this issue 5 years ago • 3 comments

Use crontab library to avoid problems with stderror outputing an empty crontab Use try except to copy file and catch same file error Use geteuid to get effective user id (sudo and su insisted I was still pi after su)

uresu avatar Feb 15 '21 11:02 uresu

This change has some good stuff in it, didn't know about the crontab library. But it's based on a previous change that I don't want to merge. Also, I'd want to simplify it a bit so it isn't longer than the old version. Unfortunately, that requires some time to focus on it, which I don't have a this time.

Matthias-Wandel avatar Feb 15 '21 19:02 Matthias-Wandel

Why the hell not! You've finished the bloody bandsaw!

Only joking, love your work. Happy to help on future projects :)

uresu avatar Feb 15 '21 19:02 uresu

Bandsaw and plans still have two weeks work minimum

On Mon, Feb 15, 2021 at 3:31 PM Wes Thompson [email protected] wrote:

Why the hell not! You've finished the bloody bandsaw!

Only joking, love your work. Happy to help on future projects :)

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/Matthias-Wandel/pi_blink_ip/pull/13#issuecomment-779412718, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOSCO3HXOC6LISWTKMFZSG3S7FY75ANCNFSM4XURD5LA .

Matthias-Wandel avatar Feb 15 '21 21:02 Matthias-Wandel