MiSTer-devel / MiSTer-devel/Scripts_MiSTer
Google Drive scripts disabled due to not being compatible, but what is the issue?
Nobody has claimed this yet.
- Dominant language
- Shell
- Stars
- 122
- Forks
- 49
- Avg merge
- 8h 56m
- Merged PRs (30d)
- 1
Description
I noticed that the "gdrive_*" scripts are aborted when run on the latest kernel version (currently "5.15.1-MiSTer"). There is a check at the start of these scripts.
However, I was unable to find any data regarding the why and what of the issue. I figured I'd try the scripts anyway by commenting out the check, and they appear to function just fine.
Could someone (perhaps the author) tell me what the original issue was? I would like to verify if it is still an issue.
My own tests seem to suggest the scripts work as expected, so perhaps we can get rid of the check altogether at this point (or change the check range to not include the latest kernel).
That all being said, we also have the rclone_* scripts which can upload to Google Drive as well, if I am not mistaken, so perhaps the current gdrive scripts are just redundant in this form.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the compatibility check at the beginning of the gdrive_* scripts and compare it with the behavior of the rclone_* scripts. Reproduce the reported behavior on kernel 5.15.1-MiSTer, then determine what incompatibility the check addresses and whether the scripts still fail without it. Done means documenting the cause and updating the check only if the tests support doing so.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- bash, shell
- Domain
- cli
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100