PlaneShift
Development => Development Deliberation => Topic started by: brandon_rioja on April 05, 2007, 01:48:17 am
-
in paws editor widgets that are loaded have camelCases:
1.1 (borrilli 26-Jan-05): succeeded &= LoadWidget("data/pawseditor/peWidgetTree.xml");
1.1 (borrilli 26-Jan-05): succeeded &= LoadWidget("data/pawseditor/peMenu.xml");
1.7 (borrilli 24-Jun-05): succeeded &= LoadWidget("data/pawseditor/peSkinSelector.xml");
in cvs, the files are lower cased
[brandon@localhost pawseditor]$ ls
CVS filenavigation.xml imagelist.xml keys_def.xml pebackground.xml pemenu.xml peskinselector.xml pewidgettree.xml
[brandon@localhost pawseditor]$
So the paws editor doesn't load can we change the cases in the application, or the names in cvs?
[brandon@localhost pawseditor]$ diff pawseditorapp.cpp pawseditorapp.cpp.~1.24.~
458,460c458,460
< succeeded &= LoadWidget("data/pawseditor/pewidgettree.xml");
< succeeded &= LoadWidget("data/pawseditor/pemenu.xml");
< succeeded &= LoadWidget("data/pawseditor/peskinselector.xml");
---
> succeeded &= LoadWidget("data/pawseditor/peWidgetTree.xml");
> succeeded &= LoadWidget("data/pawseditor/peMenu.xml");
> succeeded &= LoadWidget("data/pawseditor/peSkinSelector.xml");
[brandon@localhost pawseditor]$
-
Best way to report a bug is in the bugtracker (http://hydlaa.com/bugtracker). Of course, there are other means, like getting into IRC on irc.freenode.net in #planeshift or #planeshift-build channel.
Not to worry though, your bugfix has been noticed :)
Taken from the cvs history:
Thu Apr 5 02:49:09 2007 UTC (14 hours, 42 minutes ago) by kennygraunke
Branch: MAIN
Changes since 1.4179: +4 -0 lines
Diff to previous 1.4179
- Fix from Brandon Rioja - pawseditor was relying on case insensitivity when
loading its GUI xml files. This meant it wouldn't run under Linux.