ohio813 / openid-selector

Automatically exported from code.google.com/p/openid-selector
0 stars 0 forks source link

/trunk/generate-sprite.sh doesn't work #30

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. ./generate-sprite.sh

What is the expected output? What do you see instead?
Initially it was saying "locale not found" because it was looking for the wrong 
file (js/openid-${LOCALE}.js instead of js/openid-jquery-${LOCALE}.js). After 
correcting that, it just hangs without any output, instead of generating the 
new images/openid-providers-en.png file.

What version of the product are you using? On what operating system?
The latest revision: r115. On Mac OS X 10.6.6

Please provide any additional information below.
The following changes will eliminate the problem:

1. line 37 : JSFILE=js/openid-jquery-${LOCALE}.js
2. line 53 : ./remcomments.sed < ${JSFILE} | sed -n '/providers_large/,/;/p' | 
sed -n '/: {/p' | sed 's/.*\t\(.*\) : {/\1/' | sed -E 's/:[[:space:]]    *{//'  
> tmp.txt

I've attached the modified file

Original issue reported on code.google.com by deb5...@gmail.com on 30 Jan 2011 at 12:07

Attachments:

GoogleCodeExporter commented 9 years ago
1. about ./generate-sprite.sh looking for the wrong file 
(js/openid-${LOCALE}.js instead of js/openid-jquery-${LOCALE}.js).

go to
http://code.google.com/p/openid-selector/source/browse/#svn%2Ftrunk%2Fjs

and notice there are no openid-jquery-<locale>.js files in the trunk

I guess you have downloaded ./generate-sprite.sh separately and are trying to 
apply it to v1.2

You should use trunk version of everything instead, or try v1.3-beta1

Original comment by andriy.gerasika on 2 Feb 2011 at 8:10

GoogleCodeExporter commented 9 years ago
@andriy.gerasika: Yes, you're right. It was a silly mistake on my part. I'd 
like to thank you for maintaining this wonderful project.

Original comment by deb5...@gmail.com on 2 Feb 2011 at 11:22

GoogleCodeExporter commented 9 years ago

Original comment by andriy.gerasika on 2 Feb 2011 at 11:59