Mafarricos / Mafarricos-xbmc-dev-addons

Addons still in development
GNU General Public License v2.0
1 stars 1 forks source link

Feature request to allow centralised tracking folder. #23

Closed gb160 closed 9 years ago

gb160 commented 9 years ago

Hi, is it possible to give the user the choice of where the tracking folder is located? This would mean that my resume points would be available to all my different clients, meaning I could start watching a movie downstairs , then resume watching it from my bedroom. Obviously each client would need to be set to look in the same location.

Obviously the default location would be where it currently is, Personally I would have it on my samba share where all my local media is stored.

thoughts?

Mafarricos commented 9 years ago

beta011 should work like that.

image

gb160 commented 9 years ago

Great Progress already !!!

Problem though... Ive set the Resume folder on my samba share, the individual files are being created when i start playback, but if i stop playback, they're not being picked up when i try to resume....the playback starts from the beginning again, and the time in the individual file just starts over again from 0. I never get the yes no dialog to resume, although i notice in the log it says "Resume point available..."

Full log here... http://pastebin.com/aWEf1WJs This feature can come in very useful, thanx !!

Mafarricos commented 9 years ago

Do you have admin permissions on smb login?

I see the errors: 15:20:57 T:4616290304 WARNING: Close: failed to delete temporary file "" 15:21:01 T:4601229312 WARNING: Previous line repeats 1 times. 15:21:01 T:4601229312 WARNING: SMBFile::OpenForWrite() called with overwriting enabled! - smb://RASPBIAN/Raspbian%20SD%20Card/Albehas%20Resumes/The%20Goldbergs%202013%20S02E18.txt 15:21:01 T:4599017472 NOTICE: -->Python Interpreter Initialized<-- 15:21:06 T:4601229312 WARNING: SMBFile::OpenForWrite() called with overwriting enabled! - smb://RASPBIAN/Raspbian%20SD%20Card/Albehas%20Resumes/The%20Goldbergs%202013%20S02E18.txt

It seems that the file can't be overwriten and also can't be deleted. I don't know if this is a code error or permission error.

gb160 commented 9 years ago

I can create and delete files as a guest in samba with no username or password....also if it was a permissions problem would it create the files in the first place?

This line ..." WARNING: SMBFile::OpenForWrite() called with overwriting enabled! - " appears very regularly in my log whenever samba read/write is occurring.

Shame as this feature could be very useful.

Mafarricos commented 9 years ago

If you search that error on google it states sometimes problems related to permissions. Will check for more details.

gb160 commented 9 years ago

There was a similar issue a while back with writing .strm files to my samba share...in the end I think you copied the code from genesis and it solved the issue.

Mafarricos commented 9 years ago

The code is the same. Probably is related with rewrite permissions. Will have to check how to reproduce the same problem to fix it ;)

Mafarricos commented 9 years ago

Check beta013.

It saved my on smb running on android and also ftp on internet.

If you still have problems is something to do with permissions on your smb share.

gb160 commented 9 years ago

Well done...I'll test this again tonight, at least now if I have problems it must be my system, thanks for testing.

gb160 commented 9 years ago

I can confirm that beta013 fixes my resume problem....i didn't need to change anything on my samba setup.

Is it safe to fork this now? You don't plan on updating anything in the next few hours?

Mafarricos commented 9 years ago

You don't need to fork, only to create a PR of my changes on your fork. But if is easier you can fork it again. I will not work on the addon. Right now no bug reported ;)

gb160 commented 9 years ago

new PR issued....im waiting for u to tell me whats wrong with it !

Mafarricos commented 9 years ago

Will check it at night ;)