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

Netflix Button Addition #159

Closed
wants to merge 744 commits into from
Closed

Netflix Button Addition #159

wants to merge 744 commits into from

Conversation

zang74
Copy link

@zang74 zang74 commented Apr 7, 2015

Added missing key command for Netflix-Button Boxee Boxes.

hudokkow and others added 30 commits December 19, 2014 23:31
[Helix][rebrand] Nuke some more XBMC links and references
(cherry picked from commit d90cb8a)
dvdplayer: make sure overlay container is cleared when player stops
 DXVA: add HD5xxx to ati work-around, only capable of opening a single t...
[release] bump to Helix 14.0 final
CFile: Don't cast potential negative values to unsigned int
@quasar1
Copy link
Collaborator

quasar1 commented Apr 7, 2015

@zang74 Great work,I have uploaded a test build so you can confirm it works your end, I have tested here (I don't have the Netflix button but everything appears to work fine, no regressions).
https://www.sendspace.com/file/s6jwfa
I added the system folder too, not needed unless you want the new improved advancedsettings.xml.

@zang74
Copy link
Author

zang74 commented Apr 7, 2015

Hi,

I’ve tested, but am getting nothing. That said, the file creation date/time of the Kodi.bin you sent is March 27th, the same as the last release. Is it possible I got the old build?

On Apr 7, 2015, at 1:46 PM, quasar1 notifications@github.com wrote:

@zang74 https://github.com/zang74 Great work,I have uploaded a test build so you can confirm it works your end, I have tested here (I don't have the Netflix button but everything appears to work fine, no regressions).
https://www.sendspace.com/file/s6jwfa https://www.sendspace.com/file/s6jwfa
I added the system folder too, not needed unless you want the new improved advancedsettings.xml.


Reply to this email directly or view it on GitHub #159 (comment).

@quasar1
Copy link
Collaborator

quasar1 commented Apr 7, 2015

@zang74 Yes looks like I packaged the wrong kodi.bin.
Try this one..
https://www.sendspace.com/file/3eda10

@zang74
Copy link
Author

zang74 commented Apr 7, 2015

That did it! It works as expected.

On Apr 7, 2015, at 5:10 PM, quasar1 notifications@github.com wrote:

@zang74 https://github.com/zang74 Yes looks like I packaged the wrong rar.
Try this one..
https://www.sendspace.com/file/3eda10 https://www.sendspace.com/file/3eda10

Reply to this email directly or view it on GitHub #159 (comment).

@quasar1
Copy link
Collaborator

quasar1 commented Apr 7, 2015

Great, do you mind doing the PR to quasar1 github Helix branch?
Reason being there's a lot of merge conflicts to do on this github merge as it's so far behind in commits.

@zang74
Copy link
Author

zang74 commented Apr 7, 2015

Done.

On Apr 7, 2015, at 5:22 PM, quasar1 notifications@github.com wrote:

Great, do you mind doing the PR to quasar1 github Helix branch?
Reason being there's a lot of merge conflicts to do on this github merge as it's so far behind in commits.


Reply to this email directly or view it on GitHub #159 (comment).

@quasar1
Copy link
Collaborator

quasar1 commented Apr 8, 2015

Thanks, merged on the quasar1 github.
I will update the Helix branch here when I get the subtitles fixed, IF I can find a way to do that...

@zang74
Copy link
Author

zang74 commented Apr 8, 2015

Thanks! Wish I could be of more help. I'm no C++ coder; just a PHP guy who got tired of that damn button taunting me with its impotence and did some sleuthing.

Also; any idea how I'd get past the read-only tag to shoehorn the new binary into my internal install, or should I ask Orizzle?

On Apr 8, 2015, at 4:35 AM, quasar1 notifications@github.com wrote:

Thanks, merged on the quasar1 github. I will update the Helix branch here when I get the subtitles fixed, IF I can find a way to do that...

Reply to this email directly or view it on GitHub.

@quasar1
Copy link
Collaborator

quasar1 commented Apr 8, 2015

I'm no C++ coder, just sheer determination, dedication, trial an error and research.
So, it goes to prove it's only ones self stopping any progression.

I am not sure how the internal install works not tried it. You may be able to browse the Boxee Box using WinSCP FTP/Putty and change the file permissions that way.

I'm hoping to do another release with your changes at some point, in the middle of attempting a subtitle glitch fix.

@quasar1
Copy link
Collaborator

quasar1 commented Apr 13, 2015

@zang74
https://github.com/quasar1/boxeebox-xbmc/releases/tag/2.3-alpha_Helix
Contains your fix plus some other enhancements.

@zang74
Copy link
Author

zang74 commented May 2, 2015

Just discovered something interesting, Quasar. Apparently, Kodi is logging debug notifications even when all logging is turned off in advancedsettings.xml using . The build is a debug build, [23:07:18 T:3038897888 NOTICE: Using Debug Kodi x32 build] and thus that logging cannot be turned off without a recompile. Apparently there was a similar issue with the Android version, and there's a thread about it here: http://forum.kodi.tv/showthread.php?tid=207278&page=1

This leads into:

I run my BBs 24/7, and I've also discovered that log files are being buffered somewhere before being written to kodi.log, as "cat /dev/null > kodi.log" will empty the log file, but within a minute the file will be re-written with all the info that was just erased. I saw this happen with an 11.6MB log file. If this is being stored in RAM, this could be an issue, and could be slowly eating RAM the longer the machine remains on, and could be causing some of the RAM issues recent builds have had. I can't see buffering ~12MB of log files an efficient use of memory.

@zang74 zang74 closed this May 2, 2015
@quasar1
Copy link
Collaborator

quasar1 commented May 2, 2015

@zang74
I'm not sure you can totally disable 'logging' altogether, and I think that's what you are implying?
Have you tried the results of doing the same settings in a Kodi 14.2 Windows final build?
I will do a test build with --enable-debug=no configure flag to find out more.

More info here:
http://forum.kodi.tv/showthread.php?tid=213190

@devil-strike
Copy link

Be aware quarnster enabled allot of logging option for video part, intel smd etc debug, think if you can disable that it will run allot better. Dont think you need it anymore(there are no devs that can fix that part of errors)

@zang74
Copy link
Author

zang74 commented May 2, 2015

Apparently, in release builds one can disable logging altogether by using the loglevel tag in advancedsettings.xml. I tried this (loglevel -1) after trying loglevel 0 (just notices errors, no debug comments), only to discover that neither worked. This is apparently because the build is set as a debug build and not a release build, meaning debugging is turned on by default and turning it off is disabled. This may also be why it often creates insanely large log files without offloading to kodi.old.log. As it is now, the log file will continue to grow with every debug notice (and that includes every keypress, loading of GUI elements, etc.) with no end until the machine is rebooted.

@quasar1
Copy link
Collaborator

quasar1 commented May 2, 2015

@devil-strike Yes you are absolutely right. Looks like we aren't going to get any further dev help regarding Intel SMD so must make best of what we have.
Also some of the Kodi commits that were introduced after Gotham final to do with refclock and drop sync could be removed also. They seemed to have caused regressions, except the seek handling was a bit better. I think a new branch will be the best option, the changes will be rather invasive but we are/were in alpha stage and will always be.

@zang74 I have just compiled a test build, I will upload it shortly for you to try.

BTW. Can someone direct Orizzle in this post http://boxeed.in/forums/viewtopic.php?f=13&t=1273 to quasar1#12

I have opened 'issues' on the quasar1 github, thanks.

@quasar1
Copy link
Collaborator

quasar1 commented May 2, 2015

@zang74
Copy link
Author

zang74 commented May 3, 2015

I finally got a chance to test this version.

The debugging messaging appears to have stopped, and confluence appears markedly faster—I'm guessing because it's not having to write every GUI element load or keypress to log. Library scanning and launching of videos is slower, but I'm chalking that up to testing Kodi and its db entirely off a slow USB key (my main install is internal and I use a networked MariaDB server for the database). None of the log messages seem terribly out of the ordinary though.

@quasar1
Copy link
Collaborator

quasar1 commented May 3, 2015

There's a noticeable difference even with default with just debugging off.

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

Successfully merging this pull request may close these issues.