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

Gnome 3.4 #21

Open
leezu opened this issue Mar 14, 2012 · 10 comments · Fixed by #23
Open

Gnome 3.4 #21

leezu opened this issue Mar 14, 2012 · 10 comments · Fixed by #23

Comments

@leezu
Copy link

leezu commented Mar 14, 2012

Hey there,

it would be great, if sb could enable (or build, dont know if there were API changes or so) gnome 3.4 support :-)

thanks for the great work so far @ all

@leezu
Copy link
Author

leezu commented Mar 14, 2012

ok, actually its version gnome-unstable/gnome-shell 3.3.91-0 ;-)

@slix
Copy link

slix commented Apr 4, 2012

I tried simply adding "3.4" as a valid version to the metadata file, but that didn't work. (The extensions website reports an error.)

@lxp
Copy link

lxp commented Apr 6, 2012

galpec mentioned on extensions.gnome.org that he has modified the extension for Gnome 3.4.
https://extensions.gnome.org/extension/214/pidgin-conversation-integration/

@skoruppa
Copy link

One more waiting for this update ;)

@atomfrede
Copy link

Not sure but isn't it enough to add 3.4.0 to the metadata.json file or requires 3.4 something more?

@bugflux
Copy link

bugflux commented Apr 13, 2012

Just "3.4" doesn't work for me as well...
There's some problem about metadata.locale not being evaluated as an UTF-8 string. Hardcoding the "/usr/share/locale" seems to solve the issue...

@boypt
Copy link
Contributor

boypt commented Apr 13, 2012

Try this, worked for me.

diff --git a/extension.js b/extension.js
index 4e3267f..12bebc0 100644
--- a/extension.js
+++ b/extension.js
@@ -695,6 +695,6 @@ PidginClient.prototype = {
 }

 function init(metadata) {
-    imports.gettext.bindtextdomain('gnome-shell-extensions', metadata.localedir);
+    imports.gettext.bindtextdomain('gnome-shell-extensions', imports.misc.config.LOCALEDIR);
     return new PidginClient();
 }
diff --git a/metadata.json b/metadata.json
index 0fa128b..ddcb81e 100644
--- a/metadata.json
+++ b/metadata.json
@@ -2,7 +2,7 @@
  "uuid": "pidgin-conversation-integration@kagesenshi.org",
  "name": "Pidgin Conversation Integration",
  "description": "Display Pidgin chats as notifications in the Shell message tray.",
- "shell-version": [ "3.1.91", "3.1.92", "3.2" ],
+ "shell-version": [ "3.1.91", "3.1.92", "3.2", "3.4" ],
  "localedir": "/usr/share/locale",
  "url": "https://github.com/kagesenshi/gnome-shell-extensions-pidgin"
 }

@ambasta
Copy link

ambasta commented May 30, 2012

The suggested patch breaks the extension on gnome-shell 3.2 with:
Failed to evaluate init function. Error invoking bindtext domain at argument 2 Object is not a string, cannot convert to UTF-8

@ildar
Copy link

ildar commented Oct 10, 2012

plz upgrade to 3.6

1 similar comment
@rajeshksv
Copy link

plz upgrade to 3.6

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

Successfully merging a pull request may close this issue.

10 participants