MycroftAI / skill-npr-news

Mycroft AI official News Skill, providing the latest news report from your favorite broadcast.
https://mycroft.ai/skills
Apache License 2.0
9 stars 42 forks source link

updated skill link not working #121

Open Olzeke51 opened 3 years ago

Olzeke51 commented 3 years ago

Describe the bug

[olzeke51@Linuxer ~]$ ssh -p 8222 mycroft@192.168.X.X
(.venv) mycroft@localhost:~$ msm install https://github.com/MycroftAI/skill-npr-news/tree/bugfix/rm-curl
INFO - building SkillEntry objects for all skills
WARNING - No such default skill: mycroft-pandora
WARNING - No such default skill: mycroft-pandora
INFO - Downloading skill: https://github.com/MycroftAI/skill-npr-news/tree/bugfix/rm-curl
INFO - Problem performing action. Restoring skill to previous state...
INFO - invalidating skills cache
CloneException: 
  stderr: 'Cloning into '/tmp/tmpyni1lf4d'...
fatal: repository 'https://github.com/MycroftAI/skill-npr-news/tree/bugfix/rm-curl/' not found
'
(.venv) mycroft@localhost:~$`

To Reproduce use msm to install from github

mycroft-news.tar.gz

Log files

Environment (please complete the following information):

krisgesling commented 3 years ago

Hey Gary, if you already have the News Skill on your device, your best bet is to use git to checkout the branch you want. In this case:

cd /opt/mycroft/skills/mycroft-npr-news.mycroftai
git checkout bugfix/rm-curl

I don't believe that MSM currently has the ability to parse the url and do this automatically. It's expecting the base url of the repo only - ie msm install https://github.com/MycroftAI/skill-npr-news