[Marble-devel] Review Request: using other local map directory

Bastian Holst bastianholst at gmx.de
Wed Apr 7 14:29:35 CEST 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3473/
-----------------------------------------------------------

(Updated 2010-04-07 12:29:35.788820)


Review request for marble.


Changes
-------

* no data goes to ~/.kde anymore
* message is more user comprehensible


Summary
-------

trying to fix
https://bugs.kde.org/show_bug.cgi?id=232907 (Marble loads map from another directory than KHNS3 saves them)
(which is fixed)

and 

partly addressing
https://bugs.kde.org/show_bug.cgi?id=230714 (follow XDG basedir specification)

I think this can't be done at the same time as KHNS3 saves its data to
KStandardDirs::locateLocal("data", name)

which does not follow XDG basedir specification, see
http://techbase.kde.org/Development/Tutorials/Collaboration/HotNewStuff/Introduction


Diffs (updated)
-----

  trunk/KDE/kdeedu/marble/src/QtMainWindow.cpp 1112108 
  trunk/KDE/kdeedu/marble/src/lib/CMakeLists.txt 1112108 
  trunk/KDE/kdeedu/marble/src/lib/DataMigration.h PRE-CREATION 
  trunk/KDE/kdeedu/marble/src/lib/DataMigration.cpp PRE-CREATION 
  trunk/KDE/kdeedu/marble/src/lib/DataMigrationWidget.ui PRE-CREATION 
  trunk/KDE/kdeedu/marble/src/lib/MarbleDirs.h 1112108 
  trunk/KDE/kdeedu/marble/src/lib/MarbleDirs.cpp 1112108 
  trunk/KDE/kdeedu/marble/src/lib/MarbleWidget.cpp 1112108 
  trunk/KDE/kdeedu/marble/src/marble.knsrc 1112108 
  trunk/KDE/kdeedu/marble/src/marble_part.cpp 1112108 

Diff: http://reviewboard.kde.org/r/3473/diff


Testing
-------


Thanks,

Bastian



More information about the Marble-devel mailing list