Colon not supported in folder name about adb-sync HOT 8 OPEN

Not sure if I'll get to coding this, but it can be fixed by replacing the colon in the android folder name with a Modified Letter Colon ꞉ which is fairly similar to a regular colon. Essentially, as @divVerent said, this could be used to encode it both ways while maintaining the look of the colon on android.

from adb-sync.

It's actually a new restriction when using the FUSE backed emulated storage and the code is in MediaProvider:

    public int insertFileIfNecessaryForFuse(@NonNull String path, int uid) {
...
            if (!path.equals(getAbsoluteSanitizedPath(path))) {
                Log.e(TAG, "File name contains invalid characters");
                return OsConstants.EPERM;
            }

Since it's a limitation in the system (and to be fair, colon is indeed an invalid character for a FAT file system), I don't think adb-sync can do anything here.

EDIT:

You can edit PerformCopies and put in dst_name = self.dst[i] + re.sub(b'["*:<>?\\\\|]', b'_', name)

from adb-sync.

This used to work on Android 10 (for me) and I see absolutely no reason why ":" cannot be a valid character on modern filesystems.

from adb-sync.

I would prefer not introducing any kind of replacing as otherwise syncing back and forth would create duplicates.

If anything we would need either a bijective encoding, or - if activated by a special flag - an injective one (e.g. replacing invalid characters by escapes like =3A for a colon).

from adb-sync.

Just to check, has anyone done a root cause analysis on this? Most Unix filesystems have no problem with colon characters (on Mac, maybe, but that's worked around) so I am intrigued by what is causing this?

Edit: good heavens, there are several possible issues here:

    if subprocess.call(
        self.adb +
        [b'shell', b'mkdir -p %s' % (self.QuoteArgument(path),)]) != 0:
      raise OSError('mkdir failed')

from adb-sync.

For others on Windows who want to still be able to copy the files with invalid characters, this is the workaround I currently use.

Replace the dst_name = self.dst[i] + name line in the PerformCopies function with: dst_name = self.dst[i] + re.sub(b'[^\w\-_\.\(\) \\/]', b'_', name)

If you merely want the files with invalid names to be ignored, see here to have it just skip over files that have errors (you just add a try-catch in the loop): #29 (comment)

from adb-sync.

I'm hit by this as well - and I'm using adb-sync on Linux. The problem seems to be with the underlying adb implementation because I can't adb push files with colon in their names either. This EPERM seems to be a new behavior on Andorid 10 or 11.

from adb-sync.

This is not only caused by colons, but is also by asterisks and double quotes.

from adb-sync.