Full rewrite #185
Merged
Full rewrite #185
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This was
linked to
issues
May 16, 2022
use a improved argument parser add a -- flag, to treat all input as file or folder or drive id input stop using set -e anywhere add a improved config parser fix all edgecases in authentication commonize more code as possible move update code to update.sh do not edit zshrc or bashrc with the install script, instead show a help text on what to add to the zshrc or bashrc move all files to src folder and release scripts in release folder fix all shellcheck warnings resolve labbots#174 labbots#176
|
@labbots This is ready to merge. Github actions is failing because a new release has not been created. |
labbots
previously approved these changes
May 23, 2022
|
I'm getting this error when trying to install. |
Closed
Will be fixed after a new release has been created. |
|
@labbots Approve again. |
…ix labbots#183 It will use either netcat or python3 to start the http server Now no need to copy paste the authorizaton ccode manually https://developers.google.com/identity/protocols/oauth2/native-app#create-code-challenge
labbots
approved these changes
Jul 14, 2022
|
After updating to the new version, I always get an error I also tried to uninstall / reinstall but same result :-( |
|
That is a very weird error. You are on mac os, right ? Assuming your bash version is less than 4 and posix script was installed but it is trying to run the script with bash. can you upload you gupload binary file and send me ? also, try upgrading your bash and reinstall again. |
|
checking my bash version when I run the uninstaller, I got this output needed to removed the .google-drive-upload manually and removed the addition from /Users/michael/.zshrc manually as described. Reinstalled using the script, same error. Will search now on how to upgrade the bash... |
This pull request was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
flags: Add more documentation
[FEATURE] Support for sub-folders with --create-dir flag. | Resolve #155
install.sh: Do not fail if username is in small case
[SKIP-CI] Fix some shellcheck warinings
Use grep for -ex and -in flag | Resolve #176
Full rewrite
use a improved argument parser
add a -- flag, to treat all input as file or folder or drive id input
stop using set -e anywhere
add a improved config parser
fix all edgecases in authentication
commonize more code as possible
move update code to update.sh
do not edit zshrc or bashrc with the install script, instead show a help text on what to add to the zshrc or bashrc
move all files to src folder and release scripts in release folder
fix all shellcheck warnings
resolve #174 #176