Skip to content
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

Open
OpenNingia opened this issue May 21, 2012 · 5 comments
Open

grive upload itself to Google Drive #26

OpenNingia opened this issue May 21, 2012 · 5 comments

Comments

@OpenNingia
Copy link
Contributor

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 )

@sgbeal
Copy link

sgbeal commented May 21, 2012

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."

@OpenNingia
Copy link
Contributor Author

@sgbeal at the moment gdrive synchronize the files in the launching directory, so this is what we get.
However I never suggested to filter the "gdrive" file but the executable file name. Every program receive its own name as starting parameter ( argv[0] for tech users ) so it should be legitimate to filter itself, whatever the file name is.

@sgbeal
Copy link

sgbeal commented May 21, 2012

Filtering out argv[0] sounds like a very graceful solution :).

@match065
Copy link
Member

Agreed that argv[0] is a very graceful solution. Once we implement the ignore list feature, we can put argv[0] there.

@estomagado
Copy link
Contributor

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants