<table><tr><td style="">sitter added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D27557">View Revision</a></tr></table><br /><div><div><p>Oh, oh boy, there's many more things wrong with this than just the bashism :|</p>

<p><a href="https://phabricator.kde.org/p/ndavis/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ndavis</a> <a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ngraham</a> why don't you generate the 24x version inside the source?</p>

<p><tt style="background: #ebebeb; font-size: 13px;">        COMMAND ${BASH_EXE} ${CMAKE_SOURCE_DIR}/generate-24px-versions.sh ${BREEZE_INSTALL_DIR}</tt> this is bypassing make DESTDIR control, which will make this entire thing unpackable. It'll also outright not work if the target dir is not writable by the user running make (e.g. as is the case during packaging or when a user builds for /usr/local).</p>

<p>There's also a var interpolation bug here: <tt style="background: #ebebeb; font-size: 13px;">    OUTPUT_FILE="$OUTPUT_DIR/${INPUT_FILE/22/24}"</tt> (mind the INPUT_FILE expansion closing way after the var name ended)</p>

<p>I'm installing freebsd now to find my way to a fix ^^</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R266 Breeze Icons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27557">https://phabricator.kde.org/D27557</a></div></div><br /><div><strong>To: </strong>ngraham, VDG, ndavis, Frameworks, sitter<br /><strong>Cc: </strong>bcooksley, kossebau, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns<br /></div>