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

updated skill link not working #121

Open
Olzeke51 opened this issue Aug 25, 2021 · 1 comment
Open

updated skill link not working #121

Olzeke51 opened this issue Aug 25, 2021 · 1 comment
Labels

Comments

@Olzeke51
Copy link

Olzeke51 commented Aug 25, 2021

Describe the bug

[olzeke51@Linuxer ~]$ ssh -p 8222 [email protected]
(.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](https://github.com/MycroftAI/skill-npr-news/files/7050379/mycroft-news.tar.gz)

**Log files**


**Environment (please complete the following information):**
 - Device type: [Mark2-devkit / usb flash / network cable
 - OS: [default
 - Mycroft-core version: . 21.2
 - Other versions: latest 08-19
[edit vv ]
* this was written late aug 25 - if it is in the standard default - 
* noticed new screen display - but no audio for NPR
@Olzeke51 Olzeke51 added the bug label Aug 25, 2021
@krisgesling
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants