toltec icon indicating copy to clipboard operation
toltec copied to clipboard

rMscreens

Open pr0fsmith opened this issue 2 years ago • 13 comments

rMscreens is a commandline utility that allows you to easily change your rM screen (sleep screen, power off screen etc). It can also be used to backup and restore your original screens easily.

I wrote this code myself and tested it on the rM2 software version 2.12 https://github.com/pr0fsmith/rMscreens/blob/main/changescrn

pr0fsmith avatar Mar 15 '22 01:03 pr0fsmith

I would really recommend adding help output to the script.

I don't think we can merge this until https://github.com/pr0fsmith/rMscreens/issues/2 is resolved though.

Eeems avatar Mar 18 '22 16:03 Eeems

@pr0fsmith did you have a chance to address the requested changes?

Eeems avatar Jul 06 '22 03:07 Eeems

@pr0fsmith did you have a chance to address the requested changes?

I will do that today.

pr0fsmith avatar Jul 06 '22 15:07 pr0fsmith

@Eeems All changes have been made.

pr0fsmith avatar Jul 06 '22 21:07 pr0fsmith

@pr0fsmith I don't see any changes to this PR? Did you intend to update the source and sha256sum in the package file to pull in the latest changes?

Eeems avatar Jul 06 '22 21:07 Eeems

@pr0fsmith I don't see any changes to this PR? Did you intend to update the source and sha256sum in the package file to pull in the latest changes?

@Eeems I actually didn't make any changes. It seems as though I made the changes months ago so the existing source should be fine.

pr0fsmith avatar Jul 06 '22 22:07 pr0fsmith

@pr0fsmith it looks like this is still pointing at an old commit and doesn't have the changes that were requested.

Eeems avatar Jul 06 '22 22:07 Eeems

@pr0fsmith poke

Eeems avatar Sep 26 '22 14:09 Eeems

I want to but I honestly dont understand how. I find the whole process confusing.

Sept 26, 2022 10:57:48 a.m. Nathaniel van Diepen @.***>:

@pr0fsmith[https://github.com/pr0fsmith] poke

— Reply to this email directly, view it on GitHub[https://github.com/toltec-dev/toltec/pull/571#issuecomment-1258167387], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AM5FBGDMLBTMCII7ERDT6STWAG2WZANCNFSM5QXGBEYA]. You are receiving this because you were mentioned.[Tracking image][https://github.com/notifications/beacon/AM5FBGE4MJ4U3C4D37BGVELWAG2WZA5CNFSM5QXGBEYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOJL7BYWY.gif]

pr0fsmith avatar Sep 26 '22 15:09 pr0fsmith

@pr0fsmith I've added a comment showing what needs to be changed. There is a button to directly apply the suggested changes.

Updates to your repository don't automatically show up here as we point at specific commits so that we know that the source doesn't change between builds. Whenever you update your repository you need to point the recipie at the new commit.

Eeems avatar Sep 26 '22 15:09 Eeems

I don't understand how to change the recipe. You did it for me last time.

pr0fsmith avatar Sep 26 '22 15:09 pr0fsmith

I don't understand how to change the recipe. You did it for me last time.

If you go to the files changed tab, you'll see the recipie there. It's the package/rmscreens/package file.

Eeems avatar Sep 26 '22 15:09 Eeems

@pr0fsmith I can't get this to work, there is no help output. When looking at the comment in the script saying how to use it, that doesn't work. It looks like your argument parsing has a loop where it will never work as both -c and -n expect the other setting to already be set. image

Eeems avatar Oct 01 '22 17:10 Eeems

@pr0fsmith any chance you could get this cleaned up and ready? I'd love to get it merged.

Eeems avatar May 08 '23 23:05 Eeems

It would be nice to have this automatically do a backup/restore on install, but we'd need a better mechanism to make sure that normal usage can't interfere with what files are backed up or restored.

Eeems avatar Aug 15 '23 23:08 Eeems