openmtp icon indicating copy to clipboard operation
openmtp copied to clipboard

Sync Mode

Open tcgoetz opened this issue 5 years ago • 39 comments

A mode where you can sync a directory on the phone with a directory on the computer, copying only files not present and updating files that have changed.

tcgoetz avatar Apr 02 '19 20:04 tcgoetz

I 2nd this suggestion 👍

LeeBinder avatar Apr 09 '19 20:04 LeeBinder

Thanks for the suggestion. Right now I am trying to rebuild the native MTP module of this app as it is not compatible with certain devices! Once that is completed I shall definitely look into creating a new sync feature.

If any of you is interested in contributing towards the development of the new MTP kernel of this app, which is basically written is C++, it'd be great help as I am unable to squeeze in some time for the same.

ganeshrvel avatar Apr 09 '19 20:04 ganeshrvel

Hello ganeshrvel. Unfortunately my coding skills end with Java Script and a bit of php.

Rebuilding the native MTP module is of much higher importance because openMTP 2.0/2.1 are not working with my Samsung Galaxy A8 ROM, either (but Android File Transfer and Commander One MTP are). Even uninstalling ATF and reboot did not help.

LeeBinder avatar Apr 09 '19 23:04 LeeBinder

Rather (or I guess in addition to) than adding syncing to the app itself, could you expose a mount point for the android device, so that we just open openMTP and then use rsync or whatever existing synching app we want to take care of the rest?

gjw avatar Apr 10 '19 19:04 gjw

@gjw : https://www.reddit.com/r/apple/comments/48yb2b/guide_mounting_android_device_under_os_x_using/ (or if you like YouTube: https://www.youtube.com/watch?v=_jzMiyz4Zt4 )

(please don't expect an Orange when you pick a Banana .. ;) )

LeeBinder avatar Apr 10 '19 19:04 LeeBinder

@ganeshrvel : while we're in here, two suggestions:

  1. please make the key combo for Quit the regular Mac cmd + q. Now it's cmd + shift - q, which is not usual (and is reserved for account log-out on some if not most Macs)
  2. please replace the 'Root' on the bottom left of both panes with the Mac default / for hard disk base level. 'root' is reserved for super user / system admin

:)

LeeBinder avatar Apr 10 '19 19:04 LeeBinder

@ganeshrvel @leebinder Here a HTC 10 user 'Android File Transfer' (that i don't want to use) connects, but openMTP doesn't connect. Same computer, system and cable. And yes, cmd + shift + q on Mac brings a message to logout the system.

Anyway thanks for the app, it looks beautiful and i wanna try! ;-)) I am in to test if needed.

bartatgithub avatar Jun 06 '19 12:06 bartatgithub

I have been trying to build a new MTP kernel which can detect a wider range of devices. I have created a separate repo for the same here: https://github.com/ganeshrvel/mtp-nodejs-port-native-addon/tree/to-async

Due to my limited understanding in C++ I am unable to solve the final parts of solving this issue.
If anyone can help me solve this issue which is mentioned here: https://stackoverflow.com/questions/56347264/nodejs-libuv-non-blocking-event-callback I might be able to release a new beta version pretty soon.

ganeshrvel avatar Jun 07 '19 05:06 ganeshrvel

@bartatgithub @LeeBinder I was going through the keymaps and I don't see "cmd + shift + q" being referenced anywhere? Could you tell me where have been asked to use "cmd + shift + q " keyboard shortcut?

ganeshrvel avatar Jun 09 '19 07:06 ganeshrvel

I have created a new GitHub thread for discussing "unrecognized devices issue". Track the thread for further updates

ganeshrvel avatar Jun 09 '19 10:06 ganeshrvel

@bartatgithub @LeeBinder I was going through the keymaps and I don't see "cmd + shift + q" being referenced anywhere? Could you tell me where have been asked to use "cmd + shift + q " keyboard shortcut?

cmd + shift + q

OpenMTP 2.1 still displays 'Root' on the bottom left of both panes, but Mac default for hard disk base level is / ( 'root' is reserved for super user / system admin)

LeeBinder avatar Jun 09 '19 19:06 LeeBinder

@LeeBinder thank you for the screenshot! I have made a new release could you check and tell me whether your devices are getting recognised or not?

ganeshrvel avatar Jun 10 '19 03:06 ganeshrvel

@ganeshrvel as I wrote my screenshot already is from your newest 2.1 release and no, my Samsung is still not recognized.

LeeBinder avatar Jun 10 '19 06:06 LeeBinder

Thanks!

openMTP 2.1.0 works with a HTC 10 (Android 7.1), hopefully on other phones too. Be sure you use a charge + data USB cable.

Keymaps are still "cmd + shift + q", same as Logout, and work only that way, see screenshot: Screen Shot 2019-06-10 at 11 41 44

Screen Shot 2019-06-10 at 11 41 44

One thing with the interface: When you double click a folder to open, the checkbox go's also in on/off state. So often I have to click four times to get in a folder. Copy works. Will test more during the day's that come.

bartatgithub avatar Jun 10 '19 09:06 bartatgithub

@bartatgithub that's wonderful! Happy to know that the app works on HTC smartphones now.

Regarding the "double click" on a folder. You can just double click on any item to open not 4 times.

I will fix the quit menu issue in the next release.

ganeshrvel avatar Jun 10 '19 10:06 ganeshrvel

Ok, tested it and it was my impression because when you double click a folder name the checkbox go's also in on/off state. Gives a weird interaction impression, that's all, so not a big problem. Could you separate the interaction, click, on the checkbox and the interaction, click, on the folder, when you have extra time? Thanks!

bartatgithub avatar Jun 10 '19 10:06 bartatgithub

@bartatgithub I have fixed the Quit Keyboard shortcuts in the beta release along with some amazing performance improvement.

ganeshrvel avatar Jul 11 '19 13:07 ganeshrvel

Quit Keyboard shortcuts works now on MacOS, thanks!

bartatgithub avatar Jul 18 '19 09:07 bartatgithub

Quit Keyboard shortcuts works now on MacOS, thanks!

https://github.com/ganeshrvel/openmtp/releases/download/v2.2.2/OpenMTP-2.2.2.dmg

Better - but why the deviation from standard ⌘Q for quit - why ⌥⌘Q? What's your reasoning for not simply use ⌘Q as any other Mac app? ⌘Q is not even used for something else in your app, so there really is not a single reason not to use it for simple quit .. ;)

Also, it still says 'Root > Users > ..' vs. (correct) '/ > Users > ..'. root' is referred to as something else - the root user.

LeeBinder avatar Jul 18 '19 11:07 LeeBinder

@LeeBinder

Thanks for confirming, ⌥⌘Q is used instead of the standard ⌘Q because ⌘1 keymap is used to toggle the tabs. There are chances that users might accidentally press ⌘Q while trying to press ⌘1.

The breadcrumb path is clickable and there isn't much width around the character "/" to make it clickable. This might turn into a bad UX

ganeshrvel avatar Jul 18 '19 11:07 ganeshrvel

⌥⌘Q: really? How many users don't take off their gloves before hitting the keyboard .. ;)?

Request: can you please make this shortcut configurable as in shortcut prefs?

Root vs. /: simply add space characters around the / and have the clickable path be " / ". Three characters is enough for people even with paws, parkinsons, or panic syndrome .. ;)

LeeBinder avatar Jul 18 '19 11:07 LeeBinder

Strange in the beta release it's really ⌘Q and works also in that way? @LeeBinder: Download and install that one, you will see.

bartatgithub avatar Jul 18 '19 11:07 bartatgithub

@bartatgithub : neither in beta 1 nor in beta 2 it's ⌘Q ...

LeeBinder avatar Jul 18 '19 11:07 LeeBinder

I see it right now on my screen and it works. MacOS High Sierra OpenMTP 2.2.0-beta.1 see screenshot in attachment Screen Shot 2019-07-18 at 13 54 13

bartatgithub avatar Jul 18 '19 11:07 bartatgithub

@bartatgithub : not here: Screen Shot 2019-07-18 at 14 16 22 Screen Shot 2019-07-18 at 14 17 06

At OpenMTP-2.2.0-beta.1 he even writes ""Quit App" shortcut key is changed to CMD+ALT+Q"

Unless you're making this up, Ganesh must have released TWO different 2.2.0-beta.1 then..

.. and also same in 2.2.0-beta.1 and 2.2.0 final..

LeeBinder avatar Jul 18 '19 12:07 LeeBinder

That's weird? Do you have downloaded this one: https://github.com/ganeshrvel/openmtp/releases/download/v2.2.0-beta.1/OpenMTP-2.2.0-beta.1.dmg

bartatgithub avatar Jul 18 '19 13:07 bartatgithub

@bartatgithub please try: https://github.com/ganeshrvel/openmtp/releases/download/v2.2.2/OpenMTP-2.2.2.dmg

This issue should have been fixed in the latest version

ganeshrvel avatar Jul 18 '19 13:07 ganeshrvel

I will download OpenMTP-2.2.0-beta.2.dmg and see what happens. Do you have deleted the previous version or replaced it?

bartatgithub avatar Jul 18 '19 13:07 bartatgithub

@ganeshrvel: Ok, i will first try that one.

bartatgithub avatar Jul 18 '19 13:07 bartatgithub

@bartatgithub

Do you have deleted the previous version or replaced it?

not sure why this is happening. Please try 2.2.2

ganeshrvel avatar Jul 18 '19 13:07 ganeshrvel

Yes, ⌘Q and works also on the keyboard! OpenMTP 2.2.2

bartatgithub avatar Jul 18 '19 13:07 bartatgithub

@LeeBinder: Try to first delete the previous version and install OpenMTP 2.2.2, maybe that will work? That works for me.

bartatgithub avatar Jul 18 '19 13:07 bartatgithub

@LeeBinder: Try to first delete the previous version and install OpenMTP 2.2.2, maybe that will work? That works for me.

removed all OpenMTP related files with App Uninstaller then double-checked with iTrash that indeed all settings/ prefs/ caches etc. are gone. Keyboard shortcut remains the same with 2.2.2 and below.

Some Voodoo with OpenMTP which only surfaces on your system as it seems.

I'm running High Sierra 10.13.6.

LeeBinder avatar Jul 18 '19 18:07 LeeBinder

I have done some more tests and this are the weird results: iMac — MacOS High Sierra 10.13.6 OpenMTP 2.2.0-beta.1 and OpenMTP 2.2.2 ---> ⌘Q and works also on the keyboard!

MacBook Pro — MacOS El Capitan 10.11.4 OpenMTP 2.2.0-beta.1 and OpenMTP 2.2.2 ---> ⌥⌘Q and works only with ⌥⌘Q on the keyboard?!

Will test later on a MacBook Pro — MacOS High Sierra 10.13.6.

bartatgithub avatar Jul 19 '19 17:07 bartatgithub

In the meantime, for syncing, there's an ADB solution, by Google. https://github.com/google/adb-sync

hacker1024 avatar Apr 11 '20 22:04 hacker1024

@hacker1024 interesting, thanks for the link. Command line only, from what I see. Do you know if there is a GUI frontend?

LeeBinder avatar Apr 11 '20 23:04 LeeBinder

@ganeshrvel I'd also love to see a sync feature where a source (left pane) could be used to sync with a destination (right pane). Ideally, it would offer:

  1. Merge
  2. Mirror (left-to-right or right-to-left)

maxengel avatar May 26 '21 21:05 maxengel