After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 73585 - theme changes are not being honoured
theme changes are not being honoured
Status: VERIFIED FIXED
Product: gnome-control-center
Classification: Core
Component: [obsolete] settings-daemon
1.5.x
Other Solaris
: Urgent critical
: GNOME2.0
Assigned To: jacob berkman
jacob berkman
: 72576 74624 80216 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2002-03-05 17:25 UTC by Shane O'Connor
Modified: 2009-08-15 18:40 UTC
See Also:
GNOME target: ---
GNOME version: 2.0


Attachments
truss output when changing themes (13.20 KB, text/plain)
2002-05-07 13:29 UTC, Shane O'Connor
  Details
patch to print a message at startup. (488 bytes, patch)
2002-05-07 16:06 UTC, jacob berkman
none Details | Review

Description Shane O'Connor 2002-03-05 17:25:26 UTC
There is not way of applying changes made through the Theme Selector
CApplet.
Comment 1 Luis Villa 2002-03-05 21:55:13 UTC
And no other method is available so marking up to Urgent. As usual, if
this is someone not running gnome2-settings-daemon, please close.
Comment 2 Shane O'Connor 2002-03-06 14:19:40 UTC
gnome2-settings-daemon is running
Comment 3 Luis Villa 2002-03-07 17:16:34 UTC
Lauris, can you look at this?
Comment 4 Lauris Kaplinski 2002-03-07 20:19:21 UTC
Works for me.
Which themes you are trying with? Do they otherwise work well?
Are theme setting values persisten between gnomecc (theme capplet) 
sessions?
Any warning/debug messages from capplet and or settings daeomn?
Comment 5 Shane O'Connor 2002-03-11 10:54:26 UTC
sorry for lack of detail in above reports:

started the gnome2-settings-daemon before attempting to change theme
(with Gtk+ them selector).

For each of the installed themes (Default, Metal & Redmond65) an error
message was displayed in terminal:
"setting Net/ThemeName Default
Invalid XSETTINGS property (read off end)
setting Net/ThemeName Metal
Invalid XSETTINGS property (read off end)
setting Net/ThemeName Redmond65
Invalid XSETTINGS property (read off end)"



Same error is displayed when running through dbx.
Comment 6 Lauris Kaplinski 2002-03-13 20:09:24 UTC
Is it possible, that this is related and/or duplicate of 71150?
Dave, does it work for you on sparc/solaris?
Comment 7 Shane O'Connor 2002-03-14 10:35:40 UTC
This bug has regressed.

gtk+ theme selector capplet that shipped with Beta2: in addition to
not (instant) applying changes to the current gnome theme it will not
allow a user to preview a theme.

The 'Preview' button and the checkbox for automatic preview are not
present - I have logged a new bug (bugid: 74624) against this. 
Comment 8 Luis Villa 2002-03-21 13:47:34 UTC
*** Bug 74624 has been marked as a duplicate of this bug. ***
Comment 9 Lauris Kaplinski 2002-03-21 17:59:31 UTC
The preview button is gone, because it should now be true instant apply
(and is for me).
Dave, can you check, whether it works on your sun machine?
Comment 10 Dave Camp 2002-03-21 18:42:53 UTC
My sun machine doesn't have gnome on it yet.  Shane, the summary says
"... not instant apply", does that imply that it is applying, but not
instantly?  Or is the setting not applying at all?

I'm inclined to change the summary of this bug to "gtk xsettings do
not apply on solaris."  Would this be accurate?
Comment 11 Shane O'Connor 2002-03-25 10:53:07 UTC
the settings do not apply at all - in addtion to this the preview does
not work - 

Comment 12 Shane O'Connor 2002-04-08 09:23:39 UTC
In beta3 the gnome2-settings-daemon is crashes after desktop login and
everytime you try to start it - see bugid:77775. Cannot apply any
changes for the Gtk+ theme editor without the gnome2-settings-daemon.
Comment 13 Luis Villa 2002-04-10 02:52:34 UTC
Updating all cc bugs that have the GNOME2 keyword set to the GNOME2.0 milestone,
to help jrb triage/prioritize cc bugs. Filter on 'luis doing GNOME2 work' to
ignore this spam.
Comment 14 jacob berkman 2002-04-12 19:37:39 UTC
the gnome-settings-daemon crash has been fixed.

there's a new font/theme capplet in CVS, and when running it on my
solaris workstation i do get the 'Invalid XSETTINGS property (read off
end)' messages, which i believe are inhibitting the theme change to
take effect.

is this a correct assessment?
Comment 15 jacob berkman 2002-04-12 21:54:12 UTC
fixed in CVS gtk+ and sun-patches:

2002-04-12  jacob berkman  <jacob@ximian.com>

	* gdk/x11/xsettings-client.c (fetch_card8): fix cut-n-paste bug
	which breaks big-endian machines (fixes #73585)
Comment 16 Shane O'Connor 2002-04-18 13:33:13 UTC
Jacob,

this is correct - I've installed a nightly from 17th April and when
trying to change the theme am getting the "Invalid XSETTINGS property
(read off end)" messages and no theme changes are getting applied.

I've also noted that the Font changes are not applied and I get errors
such as:

"(/opt/gnome-2.0/bin/gnome-font-and-theme-properties:10254):
Gtk-CRITICAL **: file gtktreemodel.c: line 333: assertion `path !=
NULL' failed

vi E(gnome-font-and-theme-properties:10223): Gtk-CRITICAL **: file
gtkliststore.c: line 571: assertion `GTK_LIST_STORE
(tree_model)->stamp == iter->stamp' failed"


This is same when run through dbx.
Comment 17 jacob berkman 2002-04-18 15:55:48 UTC
if you *are* getting "Invalid XSETTINGS property
(read off end)" then you aren't getting the fix for this.

which gtk are you using?  which branch, built by whom etc? 
specifically, if you are using the multihead branch it may not be in that.

i am 99% certain it is fixed as i could reproduce your problem exactly
until i fixed it. 
Comment 18 Shane O'Connor 2002-04-18 16:13:16 UTC
i'm using gtk+ 2.0.2 dated april 17th:-
"cvs -q co -D 'Wed Apr 17 01:26:00 BST 2002' gtk+"


Your fix is listed in the Change Log:-
"2002-04-12  jacob berkman  <jacob@ximian.com>
        * gdk/x11/xsettings-client.c (fetch_card8): fix cut-n-paste
bug which breaks big-endian machines (fixes #73585)"



Let me know if I can I check anything else?
Comment 19 Jody Goldberg 2002-04-18 16:26:20 UTC
I can not replicate any of those warnings with current CVS on linux.
Comment 20 jacob berkman 2002-04-18 16:33:55 UTC
jody, it was an endianess bug not an OS specific one.  people were
getting it on ppc linux too.


i am installing the latest gtk2 build and will retry with current CVS
theme switcher and settings daemon.
Comment 21 jacob berkman 2002-04-18 16:41:35 UTC
ok, theme switching still works for me.

what themes are you trying to switch between?
Comment 22 Shane O'Connor 2002-04-19 12:26:13 UTC
so it looks like its fixed in the latest gtk from cvs? i'll update gtk
on my system and check this.
Comment 23 Shane O'Connor 2002-04-19 12:37:22 UTC
I just updated gtk+ to a that taken from cvs at 1am April 19th - I am
still seeing the above mentioned error messages when changing theme
with the Gtk+ theme selector capplet.

I am switching between all the available themes - Default -> Redmond95
-> Metal. 

Font changes are also not being implemented - see above for more info.


Is there anything else I should have updated when updating Gtk+?
Comment 24 jacob berkman 2002-04-19 14:48:04 UTC
ok, i'll build those themes (i was just testing with a short gtkrc
which made text red).

just for fun, can you double check that what you're running links
against the gtk that you've built?  (ldd `which
gnome-font-and-theme-properties` | grep gtk) [ the binary name might
be different - i guessed from memory ]
Comment 25 Shane O'Connor 2002-04-19 15:07:53 UTC
i checked for links and the following is there:

"libgtk-x11-2.0.so.0 =>	 /opt/gnome-2.0/lib/libgtk-x11-2.0.so.0"

Comment 26 jacob berkman 2002-04-19 16:40:24 UTC
i am taking ownership as this seems to be solaris specific
Comment 27 jacob berkman 2002-04-19 16:42:55 UTC
ok, i just tested and there was a bug in the metal theme that would
crash with notebook tabs.  fixed in CVS.

however, i still do not get the XSETTINGS message, and theme switches
work fine.

can you double check - in gtk+/gdk, do a make clean, then build, then
reinstall (and verify that it goes into the right place).

also verify that you get this:

acob@wet-pants:gtk+ G2$ cvs status gdk/x11/xsettings-client.c
===================================================================
File: xsettings-client.c	Status: Up-to-date

   Working revision:	1.2
   Repository revision:	1.2	/cvs/gnome/gtk+/gdk/x11/xsettings-client.c,v
   Sticky Tag:		(none)
   Sticky Date:		(none)
   Sticky Options:	(none)
Comment 28 jacob berkman 2002-04-19 16:57:29 UTC
also font changes work for me.
Comment 29 Shane O'Connor 2002-04-29 15:03:02 UTC
*** Bug 80216 has been marked as a duplicate of this bug. ***
Comment 30 Shane O'Connor 2002-04-29 15:07:13 UTC
i am still seeing this - do i need to re-login for changes to take
effect? In beta4 I changing betweem Default, Metal, Redmond95 themes
does nothing - also changing fonts for Application or Desktop does
nothing.

I've tried turning off the 'Use Nautilus to draw desktop' option and
this makes no differenc.
Comment 31 jacob berkman 2002-04-29 15:55:36 UTC
did you check that you had the right version of xsettings-client.c?
Comment 32 Shane O'Connor 2002-04-29 15:58:50 UTC
hiya jacob,

how do i know what the right version of xsettings-client.c is?

Comment 33 jacob berkman 2002-04-29 16:00:53 UTC
as i said above:

> also verify that you get this:
>
> acob@wet-pants:gtk+ G2$ cvs status gdk/x11/xsettings-client.c
> ===================================================================
> File: xsettings-client.c	Status: Up-to-date
> 
>    Working revision:	1.2
>    Repository revision:	1.2 /cvs/gnome/gtk+/gdk/x11/xsettings-client.c,v
>    Sticky Tag:		(none)
>    Sticky Date:		(none)
>    Sticky Options:	(none)
Comment 34 bugzilla-gnome 2002-04-29 16:48:33 UTC
It works okay for me on Linux/PPC (Debian Woody), with a vicious build
scripts build from 29/4/2002 (morning GMT)
Comment 35 jacob berkman 2002-04-29 16:49:48 UTC
have you logged out/in since installing the gtk which i said had the fix?

you may need to restart gnome-settings daemon.
Comment 36 Bastien Nocera 2002-04-29 16:51:46 UTC
I have gtk+ 2.0.2 + jacob's endian patch and control-center 1.99.6-1
 running on Debian/PPC with X4.2, and the theme switcher work just fine
 for me. All the changes are instant-apply.
Comment 37 Luis Villa 2002-04-30 13:58:08 UTC
Shane, Keelin: what is the situation here?
Comment 38 Shane O'Connor 2002-04-30 15:10:06 UTC
as it stands it's still not working BUT I'm going to have another look
today 
Comment 39 jacob berkman 2002-05-01 15:35:36 UTC
*** Bug 72576 has been marked as a duplicate of this bug. ***
Comment 40 Nat Friedman 2002-05-03 17:19:33 UTC
Shane - did you manage to reproduce it?  We're still seeing it working
here.
Comment 41 Shane O'Connor 2002-05-07 13:28:46 UTC
I've tried again with source from the 5th of May and themes are still
not being applied on Solaris. The capplet runs fine and you can select
between the 3 available themes but there are no changes applied. I am
moving between Default, Metal & Redmond95. There is not much available
via debug - am I missing something, is there a flag that needs to be
set somewhere (some other sawfish capplet?) to allow theme changes?

Am attaching a file with truss output when trying to change themes.
Comment 42 Shane O'Connor 2002-05-07 13:29:28 UTC
Created attachment 8252 [details]
truss output when changing themes
Comment 43 jacob berkman 2002-05-07 16:05:58 UTC
the truss output isn't useful.

you also never verified that you had the required xsettings-client.c.

please apply the patch i'm about to attach, and verify that when you
run apps, you get a "Opening display..." message printed out.
Comment 44 jacob berkman 2002-05-07 16:06:39 UTC
Created attachment 8255 [details] [review]
patch to print a message at startup.
Comment 45 Shane O'Connor 2002-05-10 10:25:57 UTC
not sure how to apply patches but i'll checkout the gtk+ source and
talk to the sun build guys.
Comment 46 jacob berkman 2002-05-14 17:21:21 UTC
as per feedback from sun developers, this bug is definitely fixed.

shane, it must be some misconfiguration on your system with library
paths or something.  everyone else can confirm that this is working now.
Comment 47 Shane O'Connor 2002-05-15 13:55:26 UTC
yea - its working for me too...

cheers jacob

verifying fixed in source from 13th may
Comment 48 Shane O'Connor 2002-05-15 13:55:51 UTC
closing