Bug 430892

Summary: ./copy_from_breeze.sh does not work on openSUSE Leap 15.2
Product: [Applications] kmymoney Reporter: Thomas Baumgart <tbaumgart>
Component: buildsystemAssignee: KMyMoney Devel Mailing List <kmymoney-devel>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: git (master)   
Target Milestone: ---   
Platform: openSUSE   
OS: Linux   
Latest Commit: Version Fixed In:

Description Thomas Baumgart 2020-12-28 10:34:13 UTC
SUMMARY


STEPS TO REPRODUCE
1. Run ./copy_from_breeze.sh <breeze-source-directory>

OBSERVED RESULT
prints error message

  sed: -e expression #1, char 59: Invalid content of \{\}

on console and does not copy the icons referenced in icons.cpp

EXPECTED RESULT
does not print error message and copies icons referenced in icons.cpp

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: openSUSE Leap 15.2
(available in About System)
KDE Plasma Version: 5.18.6
KDE Frameworks Version: 5.71.0
Qt Version: 5.12.7

ADDITIONAL INFORMATION
Comment 1 Thomas Baumgart 2020-12-28 11:55:56 UTC
Git commit 477b0da5921ddd8795422c3dfe5a03e4b2d78dff by Thomas Baumgart.
Committed on 28/12/2020 at 11:55.
Pushed by tbaumgart into branch 'master'.

Improve copy_from_breeze.sh

- Check arguments and print error messages
- Escape braces in sed substitution expression
- Remove trailing / from directory argument
- Check for presence of icons.cpp
- Allow to run out of any directory
- Convert files to *NIX line ending if tool available

M  +61   -11   kmymoney/icons/copy_from_breeze.sh

https://invent.kde.org/office/kmymoney/commit/477b0da5921ddd8795422c3dfe5a03e4b2d78dff