D20166: Keep desktoptheme SVG files uncompressed in repo, install (stripped) svgz

Friedrich W. H. Kossebau noreply at phabricator.kde.org
Mon Apr 1 13:30:22 BST 2019


kossebau created this revision.
kossebau added reviewers: Plasma, VDG.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
kossebau requested review of this revision.

REVISION SUMMARY
  The SVG format being based on plain text, storing the SVG in the repository
  not as .svgz, but .svg, helps both VCS tools (patching, showing diffs) as
  well as allows some developers to edit the SVG directly in any text editor,
  not only those which support automatic conversion from/to gzip format.
  
  While most artists will continue (and which shall be okay) to use GUI
  editors like inkscape, which might rewrite the complete structure on saving,
  using uncompressed format in the repo still allows the occasional direct
  edit of the text, .e.g. to change a colour, which then is also easily seen
  in the commit diff.
  
  To still keep the svgz format when deployed, a build step is introduced,
  which uses gzip to create svgz files in the build dir. This conversion can
  be controlled using the option GZIP_DESKTOPTHEME_SVG (default: ON).
  
  With such a build processing in place for SVG files, next to that an
  optional stripping of the installed SVG files is added, which removes data
  which is not needed for rendering, e.g. indentation whitespaces or
  editor-only XML data. This experimental stripping can be controlled using
  the option STRIP_DESKTOPTHEME_SVG (default: OFF).

TEST PLAN
  Themes are still working (with & without cache removed).

REPOSITORY
  R242 Plasma Framework (Library)

BRANCH
  uncompressedsvginrepo

REVISION DETAIL
  https://phabricator.kde.org/D20166

AFFECTED FILES
  CMakeLists.txt
  cmake/Findgzip.cmake
  cmake/Findsvgcleaner.cmake
  src/desktoptheme/CMakeLists.txt
  src/desktoptheme/air/CMakeLists.txt
  src/desktoptheme/breeze/CMakeLists.txt
  src/desktoptheme/oxygen/CMakeLists.txt

To: kossebau, #plasma, #vdg
Cc: kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190401/4e6c2f9d/attachment.html>


More information about the Kde-frameworks-devel mailing list