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 scanning samba folders #36

Closed
MrPretzel opened this issue Sep 13, 2014 · 7 comments
Closed

Not scanning samba folders #36

MrPretzel opened this issue Sep 13, 2014 · 7 comments

Comments

@MrPretzel
Copy link

I can't seem to get Taiga to scan a samba folder for episodes. While scanning a local folder works fine. This worked in an earlier version (0.9). Is there something different I need to do to get it to work?

@rr-
Copy link

rr- commented Sep 13, 2014

Confirmed, it doesn't work for me either.

@erengy
Copy link
Owner

erengy commented Sep 13, 2014

Excuse my ignorance, but what's a samba folder? How would I go about reproducing the issue?

The file search method works pretty much the same since v0.9. The only important changes that I can think of are:

  • Taiga now prepends \\?\ to the path in order to extend the length limit from 260 to 32,767 characters.
  • Taiga now ignores hidden/system files and directories.

@rr-
Copy link

rr- commented Sep 13, 2014

TL;DR: Samba folders are Windows shares, that can be accessed with \\computer\path\to\share like regular paths. In my case, I run Samba server on my Debian box so I can watch anime both on desktop and tablet while having it stored in one location.
Therefore, prepending \\?\ to path is probably the reason behind this issue and should be done only if path doesn't already start with \\.

@erengy
Copy link
Owner

erengy commented Sep 14, 2014

I ended up making a lot of changes rather than simply not using the \\?\ prefix. Could you test this executable and see if it works now?

If it does, I will also appreciate it if you can create a path that is longer than 260 characters (e.g. by nesting folders) and try to reach it over network via Taiga.

@rr-
Copy link

rr- commented Sep 14, 2014

Samba folders work perfectly now. Thanks 👍

Regarding testing paths longer than 260 characters - Windows Explorer doesn't seem to let me do that:

20140914_092123_nsd

...even if I nest folders. I cannot do it with cmd either:

Z:\(...)> mkdir ttttttttttttttttttttttttttttttttttttttttttttt
The full path of ttttttttttttttttttttttttttttttttttttttttttttt is too long.

It seems to be expected with Samba, however, so if it works for local file system, chances are everything is fine.

@rr-
Copy link

rr- commented Sep 14, 2014

Heads up: I was able to do it with cygwin on my computer. Anime inside long paths on local file system works fine (even if Explorer can't handle it, lol - when I try to open it, it just ignores me and stays in parent folder). Doing it on Samba shares still throws error, though:

mkdir: cannot create directory ‘tttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttttt’: File name too long

It happens if I do it both remotely from cygwin on PC, or at the server itself (i.e. on Debian box that hosts the files). Which is interesting, because I have ext4 there, which probably should support long paths.

Anyway, it seems you cannot create such paths on Samba shares, so looks like everything that could work, works.

@erengy
Copy link
Owner

erengy commented Sep 14, 2014

All right then, this should be fixed with fc607ea and bfe2195.

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

No branches or pull requests

3 participants