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

Not able to push files if path is not present #130

Closed
philipp923 opened this issue Mar 27, 2015 · 7 comments
Closed

Not able to push files if path is not present #130

philipp923 opened this issue Mar 27, 2015 · 7 comments
Labels

Comments

@philipp923
Copy link

If the push command is executed for specific files/folders and the path is not jet present in Google Drive, these folders are not transferred.

There is no error message.

@odeke-em
Copy link
Owner

Hello @philipp923 welcome to drive.
Would you mind giving me a bit more information as well as diagnostics? What commands did you execute? What version are you?
drive pushes upstream content that is not or may have been modified so this issue pretty much defeats the purpose of it. Hopefully we can get this solved asap.

@philipp923
Copy link
Author

Hi odeke-em,
thanks for the fast reply. Very nice to notice that the project is active.
I tested a little bit and came up with a minimal example.

Folder structure:
gdrive <- root folder GoogleDrive

  • folderNotOnline
    • fileNotOnline.txt

philipp@x220:/xxx/gdrive$ drive push folderNotOnline/fileNotOnline.txt
Resolving...

  • /folderNotOnline/fileNotOnline.txt
    Addition count 1
    Proceed with the changes? [Y/n]: Y
    1 / 1 [=====================================================================================================] 100.00 % 0

Everything looks fine, but the folderNotOnline as well as the fileNotOnline.txt are not transmitted to GoogleDrive.

philipp@x220:/xxx/gdrive$ drive stat folderNotOnline/fileNotOnline.txt
/folderNotOnline/fileNotOnline.txt: remote path doesn't exist
philipp@x220:/xxx/gdrive$ drive stat folderNotOnline/
/folderNotOnline: remote path doesn't exist

drive is working if I push the entire folderNotOnline:

philipp@x220:/xxx/gdrive$ drive push folderNotOnline/
Resolving...

  • /folderNotOnline
  • /folderNotOnline/fileNotOnline.txt
    Addition count 2
    Proceed with the changes? [Y/n]: Y
    2 / 2 [====================================================================================================] 100.00 % 2s

For my use case it would be very helpful to have the ability to push individual sub-folders as well as individual files located in these sub-folder.

Here are the versions I am using:
drive version 0.1.5
go version go1.2.1 linux/amd64
Linux x220 3.16.0-33-generic #44-Ubuntu SMP Thu Mar 12 12:19:35 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux

Thanks in advance and have a nice day.
Philipp

@kcwu
Copy link

kcwu commented Mar 28, 2015

I can reproduce the minimal case, too

@odeke-em
Copy link
Owner

Hello @philipp923 sorry for the late reply, Friday nights offset Saturdays by a bit ;)
Oh yes in deed, this issue should be present because now it just dawned onto me that during remoteAdditions I was doing lookups of parents instead of remoteMkdirAll of parents which should be the case. This behaviour was changed a couple of weeks ago when the Google API client was buggy and was uploading files title 'Untitled Title' for every folder for which I used remoteMkdirAll, hence I backed off. However the API client got fixed and I also patched that in drive however not in that spot. Thank you very much for bringing this.
I am pushing up the fix in a few minutes.

odeke-em added a commit that referenced this issue Mar 29, 2015
odeke-em added a commit that referenced this issue Mar 29, 2015
odeke-em added a commit that referenced this issue Mar 29, 2015
@odeke-em
Copy link
Owner

For more details the Google Go api client offending change ages ago was googleapis/google-api-go-client@901b69e#commitcomment-9471898

The latest fix in drive is at 53ce76e

@odeke-em
Copy link
Owner

Please re-open if it persists, am making a release right now.

@philipp923
Copy link
Author

Hi odeke-em,
the new version of drive is working very well.
Thank you very much for the fast support!

Best regards,
Philipp

@odeke-em odeke-em added the bug label Apr 26, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants