-
Notifications
You must be signed in to change notification settings - Fork 377
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
grive upload itself to Google Drive #26
Comments
FWIW, i disagree that the name "grive" (without a dot) should be considered magical. It is a legitimate (though unusual) file/folder name for arbitrary users. Having the grive binary IN the directory to be synching is kind of like installing the dropbox binaries under ~/Dropbox oder the Ubuntu One binaries under ~/Ubuntu One (i still can't BELIEVE they put a space in the dir name!). IMHO that's just "bad practice." |
@sgbeal at the moment gdrive synchronize the files in the launching directory, so this is what we get. |
Filtering out argv[0] sounds like a very graceful solution :). |
Agreed that argv[0] is a very graceful solution. Once we implement the ignore list feature, we can put argv[0] there. |
I respectfully disagree. If you willingly put the grive executable inside the sync path, it's because you want it to be synchronized. If you don't want grive to be sync'd, why not put it on your exec path? (Which is, as I understand, standard procedure.) Mine is at /home/username/bin, and you can always put it on /usr/local/bin if you have root privileges and want Grive to be available to all users. Anyway, if this proves to be a popular request (even if I don't like it, get outvoted is one aspect of democracy) we maybe can think of a switch to filter (or not to filter) the grive executable. |
I created a folder "My Google Drive" in my home directory then copied "grive" executable inside.
Then I launched ./grive and it updates my files as it should.
However at the end of the sync process it uploaded itself to the google drive. I believe this is not intended ( as grive already ignores the .grive and .grive_state files it should also ignore the executable )
The text was updated successfully, but these errors were encountered: