Fix compatibility of build_theme.sh with Dash
ClosedPublic

Authored by gepardo on Sun, Oct 6, 8:45 PM.

Details

Summary

The script build_sh didn't work with Dash necause of "[[". When running it, I got the following:

$ ./build_theme.sh
Installing into /home/gepardo/.local/share/themes/Breeze
./build_theme.sh: 111: [[: not found
cp: cannot create directory '': No such file or directory

Test Plan

After changing "[[" to "[", it worked fine and installed the theme

Diff Detail

Repository
R98 Breeze for Gtk
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
gepardo created this revision.Sun, Oct 6, 8:45 PM
Restricted Application added a project: Plasma. · View Herald TranscriptSun, Oct 6, 8:45 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
gepardo requested review of this revision.Sun, Oct 6, 8:45 PM
svuorela accepted this revision.Tue, Oct 15, 6:48 PM
This revision is now accepted and ready to land.Tue, Oct 15, 6:48 PM
cblack accepted this revision.Tue, Oct 15, 7:44 PM

LGTM 👍

Thanks. Can you please merge it? I don't have a developer access to KDE repos.

Thanks. Can you please merge it? I don't have a developer access to KDE repos.

Sure, I'll do that.

This revision was automatically updated to reflect the committed changes.