Skip to content

Conversation

@neoh4x0r
Copy link

  1. Updated twit.tv.pl internetcontent script.
  • fixes scraping of videos and episodes (only grabs active shows)
  • uses the wiki's rss feed instead of the main site (faster and is less of a burden on their main site)
    2) Updated theme url (to http://themes.mythtv.org/themes/repository/trunk/themes.zip)
    • for 0.28-pre the theme url did not exist: http://themes.mythtv.org/themes/repository/0.28-pre/themes.zip
      3) Commented out some unnecessary osd messages about tuning/lock timeout in tv_rec::TuningSingalCheck
    • I use composite input which does not need to tune to a channel
    • the osd message must be exited out of before you can do anything with the interface and is generally not needed
    • note: it still logs these to the console/logfile/database if active

William L. DeRieux IV and others added 3 commits December 18, 2015 07:47
-- the twit.tv site has undergone some recent changes
Now the script scrapes the wiki's rss feeds instead of twit.tv directly
@dekarl
Copy link
Member

dekarl commented Dec 18, 2015

Are there already tickets for the three issues?

  • twit.tv.pl update
  • an OSD message appearing for a tuning timeout on recorders that do not tune (did I understand that correctly?)
  • The issue with the theme downloader not using trunk on master
    After a quick look the two latter patches appear to be hacks that break functionality down the road / in other use cases.

@neoh4x0r
Copy link
Author

I did do a search for the issues (as well as following the guidelines for checking if a ticket already exists) and I could not find any (open/closed).

Yes the tuning timeout message is displayed anytime you attempt to start live tv (or change channels) -- even if the input source is not tuned (eg composite/s-video and the like).

The theme url does not exist for v0.28-pre (http://themes.mythtv.org/themes/repository/0.28-pre/themes.zip) whereas the url v0.27.5 does exist.

Also, the twit.tv.pl script has been broken since at least 0.27.5 (maybe even earlier) since the twit.tv website was modified (but they do have standard rss 2.0 feeds on the wiki).

@dekarl
Copy link
Member

dekarl commented Dec 20, 2015

Ok. Can you please open a ticket for each issue so the relevant developers can take a look at it.
https://code.mythtv.org/trac/wiki/TicketHowTo#OpeningaTicket

@neoh4x0r
Copy link
Author

@dekarl -- I will open tickets for each of those issues.
It might be a little while before I can do that -- I am reverting my system back to Debian/Jessie after having tried out Stretch and Sid (too many broken packages to count).

@neoh4x0r
Copy link
Author

@dekarl : I have justed opened the tickets on the trac server

#12571 (Theme url does not exist for lastest git version 0.28-pre)
https://code.mythtv.org/trac/ticket/12571

#12572 (Non-tunable inputs always generate tuning timeout message spam when watching live tv)
https://code.mythtv.org/trac/ticket/12572

#12573 (Update to Mythnetvision twit.tv script by Auric)
https://code.mythtv.org/trac/ticket/12573

@neoh4x0r
Copy link
Author

neoh4x0r commented Jan 3, 2016

in regards to bug #12571 -- (https://code.mythtv.org/trac/ticket/12571)
the root cause of the url mismatch was because the .git folder was missing (it got deleted).
That meant that version.sh had defined MYTH_SOURCE_PATH as an empty string (instead of master).
in themechooser.cpp it expects the branch to be master or the url is built using MYTHTV_BINARY_VERSION (which is why the url did not exist 0.28 has not been released yet).

Also, bug #12572 (https://code.mythtv.org/trac/ticket/12572)
is still open (and was accepted by jpoet) -- I'm not sure of what the status is on it.

… and filename

instead of hashing the file's contents.
This allows the same file to be in multiple places (incase it it symlinked).
Each path to the same file should be unique -- and should a different hash.
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.

2 participants