Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 545772 - gnome-extra/cinnamon notifications fail with JS ERROR: RangeError: invalid language tag
Summary: gnome-extra/cinnamon notifications fail with JS ERROR: RangeError: invalid la...
Status: RESOLVED UPSTREAM
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] GNOME (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Linux Gnome Desktop Team
URL: https://github.com/linuxmint/Cinnamon...
Whiteboard:
Keywords: NeedPatch
Depends on:
Blocks:
 
Reported: 2015-04-06 20:35 UTC by Marc Vinyals
Modified: 2015-11-04 21:57 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
emerge --info (emerge-info,6.47 KB, text/plain)
2015-04-06 20:35 UTC, Marc Vinyals
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Marc Vinyals 2015-04-06 20:35:18 UTC
Created attachment 400728 [details]
emerge --info

Cinnamon has trouble processing Freedesktop notifications. When running e.g.

notify-send "meh"

but also if an application such as rhythmbox or pidgin is sending a notification, instead of the notification displaying I get the following error in ~/.xsession-errors:

(cinnamon:28266): Cjs-WARNING **: JS ERROR: RangeError: invalid language tag: %l:%M %p
Notification.prototype.update@/usr/share/cinnamon/js/ui/messageTray.js:590
Notification.prototype._init@/usr/share/cinnamon/js/ui/messageTray.js:521
Notification@/usr/share/cinnamon/js/ui/messageTray.js:422
NotificationDaemon.prototype._notifyForSource@/usr/share/cinnamon/js/ui/notificationDaemon.js:401
NotificationDaemon.prototype.NotifyAsync/<@/usr/share/cinnamon/js/ui/notificationDaemon.js:385
_proxyInvoker/asyncCallback@resource:///org/gnome/gjs/modules/overrides/Gio.js:90

Commenting out line 590 in /usr/share/cinnamon/js/ui/messageTray.js fixes the problem, and so does removing the argument to the call to toLocaleTimeString(). 

In fact, according to the spidermonkey documentation, this function expects a locale, not a time format.

This may be related to the following unit test in gnome-extra/cjs being commented out because it was broken:
https://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/gnome-extra/cjs/files/cjs-2.4.0-disable-unittest-3.patch?view=markup
Comment 1 Pacho Ramos gentoo-dev 2015-04-06 21:06:03 UTC
Can you please report the bug directly to upstream and then post the link here to let us track it? Thanks
https://github.com/linuxmint/Cinnamon/issues
Comment 2 Marc Vinyals 2015-04-06 22:17:54 UTC
Reported at https://github.com/linuxmint/Cinnamon/issues/4024
Comment 3 Pacho Ramos gentoo-dev 2015-06-07 11:05:19 UTC
We cannot do much until upstream figures out how to fix it :/
https://github.com/linuxmint/Cinnamon/issues/4024