<table><tr><td style="">geetamc updated this revision to Diff 12682.<br />geetamc edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-k7i24cvjznm66xb/" rel="noreferrer">(Show Details)</a><br />geetamc 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/D5122" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I believe that having combo box editable is a cleaner way to solve the same problem, I will revert the changes if reviewers feel that having a plain text edit with an entry for Custom Command is more appropriate.</p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">In makeconfig.ui <br />
 - Created a plain text edit and the corresponding config entry in corresponding .kcfg file<br />
 - Added a new entry in combo box that allows users to select custom command<br />
 - If that combobox entry is selected, the config entry is read, checked if it is empty,<br />
            - If it is</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">-In makeconfig.ui</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">the default command kdesu is selected</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">made the combo box for root installation command editable</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">            - Otherwise the custom command itself is run.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">-Updated the .kcfg so that suCommand is a string instead of an int. </span><br />
<br />
Reasons to accept this patch:<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R32 KDevelop</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D5122?vs=12668&id=12682" rel="noreferrer">https://phabricator.kde.org/D5122?vs=12668&id=12682</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5122" rel="noreferrer">https://phabricator.kde.org/D5122</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>projectbuilders/makebuilder/makebuilderconfig.kcfg<br />
projectbuilders/makebuilder/makeconfig.ui<br />
projectbuilders/makebuilder/makejob.cpp</div></div></div><br /><div><strong>To: </strong>geetamc, kfunk<br /><strong>Cc: </strong>kfunk, brauch, kdevelop-devel, KDevelop, geetamc, Pilzschaf, akshaydeo, surgenight, arrowdodger<br /></div>