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.

Sign up for GitHub

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

Jump to bottom

Drag and drop not working #422

Closed
adben27 opened this issue Feb 4, 2019 · 4 comments
Closed

Drag and drop not working #422

adben27 opened this issue Feb 4, 2019 · 4 comments
Labels
bug drag&drop unicode windows

Comments

@adben27
Copy link

adben27 commented Feb 4, 2019

scrcpy_2019-02-04_18-45-27
When i try to upload a file, it doesn't work, even if i remove spaces. The error is on the screenshot ^^

Any solution ? (i'm in v. 1.6)
@rom1v
Copy link
Collaborator

rom1v commented Feb 4, 2019

The problem is with the è in Bibliothèque. Could you try without?

Could you also try to manually call adb push with the full path of your file (keeping the è)?

@adben27
Copy link
Author

adben27 commented Feb 4, 2019 •
edited
Loading

It works with the file without any space in his name. How to do this with a file with a space to adb push ?

@adben27
Copy link
Author

adben27 commented Feb 4, 2019 •
edited
Loading

I finally removed the é, and it is working 😄
But is there a way for the manipulation to work with directories with accents ?

rom1v added a commit that referenced this issue Feb 10, 2019
@rom1v
Create process with wide chars on Windows
0136c9e 
Windows does not support UTF-8, so pushing a file with non-ASCII
characters failed.

Convert the UTF-8 command line to a wide characters string and call
CreateProcessW().

Fixes <#422>
@rom1v
Copy link
Collaborator

rom1v commented Feb 10, 2019

But is there a way for the manipulation to work with directories with accents ?

Yes, there is a way 😉

It's now fixed on dev branch.

@rom1v rom1v closed this as completed Feb 10, 2019
rom1v added a commit that referenced this issue Feb 10, 2019
@rom1v
Create process with wide chars on Windows
477c0a2 
Windows does not support UTF-8, so pushing a file with non-ASCII
characters failed.

Convert the UTF-8 command line to a wide characters string and call
CreateProcessW().

Fixes <#422>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug drag&drop unicode windows
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rom1v @adben27