Manual human review required : olivia

I can confirm there is a bug in the review-tools. @popey’s issued snap declaration is fine, however, as per The mpris interface and mentioned earlier in this topic, the name attribute is optional and when not present defaults to the snap name. I suggest changing the snapcraft.yaml from this:

name: olivia
slots:
  mpris:
    name: olivia
apps:
  olivia:
    slots:
    - mpris

to simply this:

name: olivia
apps:
  olivia:
    slots:
    - mpris

(I suspect this was a carryover from the olivia-test snap being renamed). This change is not required but cleans up the snapcraft.yaml a bit.

I’m preparing a fix for the review-tools now. While not needed for this snap due to the snap declaration, it should be in production next week.

3 Likes