I tried installing on a D7 site, and got this message. Thought I'd let you know.

Comments

jacine’s picture

jacine
she/her
Location New York
commented

I'm pretty sure this is a core bug, because of course Sky defines a name attribute. I was getting simliar notices on admin/appearance as well, even without Sky in the themes directory. There's an issue for what I was seeing here: #1015376: Notice: Undefined index: name in _update_process_info_list() (line 181 of modules\update\update.compare.inc).

Just out of curiosity, how did you install it? Through the UI, or manually?

e_log’s picture

e_log commented

same problem here: installed via url.

resolv_25’s picture

resolv_25 commented

Yes, the same problem, but can be fixed simply:
Delete the first line in sky.info:
// $Id$

Then will work.
Maintainers could remove this line from the package.

Best regards,
Frank

jacine’s picture

jacine
she/her
Location New York
commented
Status: Active » Fixed

Oh crap. This is a bug in Sky. It should be ; $Id$ instead of // $Id$.

It figures as I added the CVS tag code last minute. I also happened to update to 7.0 form RC-4 at the exact same time, so that's why I thought it was a core bug! Oops.

Thanks guys. I've committed the fix, and a new release will be up shortly.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

seanr’s picture

seanr
he/him
commented

We just ran into this with weird unprintable characters at the beginning of the line - took five people better part of an hour to figure it out! So if everything looks correct and it still doesn't work, you might want to try opening it in a hex editor. ;-)