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

Proposal : Podcast - Alarm - Timer #67

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

Proposal : Podcast - Alarm - Timer #67

wants to merge 2 commits into from

Conversation

JB314
Copy link

@JB314 JB314 commented Aug 7, 2015

Hi,
Here you can find some features developed on top of the dev branch.
You can use it or reject it (or ask me to adapt) as your convenience.

New features :

  • Add podcast player.
    RSS and Atom supported.
    We can use a more robust library than lastRSS but need php-dom which is not included in the php's compilation path.
    I've adapted the library in order to accept Atom feeds and self closing tag.
  • Add wakeup function (alarm).
  • Add count down listening (stop playing after x minutes)

Mandatory adaptations :

  • create the folder /mnt/MPD/Podcast
  • install "at" command. Used by count down listening
  • start "crond" service. Used by wakeup feature

root added 2 commits August 7, 2015 15:41
add wakeup function
add count down listening
@JB314 JB314 changed the title Proposal Proposal : Podcast - Alarm - Timer Aug 7, 2015
@ACXgit
Copy link
Member

ACXgit commented Aug 11, 2015

Hi JB314, sorry for the late reply. I'll take a look at your code in the details in the next days. The features are interesting and worth to be integrated, although I'd instead do that directly in the new version of RuneAudio, which we will publish soon. The 0.4 release will be the last one based on the old code and can be considered a transitional one towards the new one.

@superjasiek
Copy link

Hi tried to apply this code - unfortunately wake up feature and sleep does not work - podcast works perfect! Can't wait for new version of rune!

@JB314
Copy link
Author

JB314 commented Oct 5, 2016

Hi,
I guess it's related to the execution right on the both script command/stop and command/start.
Could you try to allow execution :
chmod +x /var/www/command/stop
chmod +x /var/www/command/start
(I don't remember if it needs to change the owner too (with command "chown"))

I've to publish an updated version of thoses scripts which supports the Spotify renderer. (force the MPD when it starts, and stop spotify playback correctly)
Let me know
JB
[edit : e-mail reply = bad format]

@superjasiek
Copy link

Unfortunately it did not make a trick. I'v changed chown (to http) and chmod (executable) mpd does not stop and when i try to activate wakeup feature i only get "Software Failure. Press left mouse button to continue.Guru Meditation #22.65045032" Any idea? All best Jan

@JB314
Copy link
Author

JB314 commented Oct 5, 2016

quite strange ! which version are you using ? 0.3 or dev branch (~0.4) ?
Probably the source files have changed a lot.
But, can you check if cronie is correctly installed and started :
systemctl enable cronie
systemctl start cronie
If yes, you can check the content of the files /var/www/command/wakeup and /var/www/command/wakeup.cron if there is something inside.
The 1st one stores the data for the WebUI and the 2nd one stores the data for cron job.
If they are empty, that means the data are not correctly submitted or saved. You can check the owner and group and verify the write access on both files. (not sure it's needed in fact)
You can also check contab items to see if there is something pending : crontab -l
And finally, I remember it needs "at" command, you can verify if there is some pending command with "atq".

@superjasiek
Copy link

Cool thank's i will check solutions, i'm using 0.3 version. Take care! And I will let you know.

@superjasiek
Copy link

while executing atq - it says that there are some tasks pending added by http. In case of cronie - it is enabled but says (cronetab -l) no crontab for root. Also there are no such files in as wakeup and wakeup.cron in /var/www/command.

@superjasiek
Copy link

ok it is 0,4 version -sorry for putting wrong information

@JB314
Copy link
Author

JB314 commented Oct 5, 2016

It looks like the latest source doesn't match with my code (or the opposit ;) )
Perhaps you can try to create empty files for wakeup and wakup.cron with right rights (rw).
It seems the command sent by the UI is not correctly handled by the backend :/
If you execute manually the stop and start "playlistname" command, do you get something positiv ?

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.

3 participants