<table><tr><td style="">ematirov updated this revision to Diff 12988.<br />ematirov edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-4mvwq66bz6l5oc7/" rel="noreferrer">(Show Details)</a><br />ematirov 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/D5188" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Fix review suggestions. Make folders of project buildable by right-clicking.<br />
Suggestions about naming are welcome - I renamed folder to buildsystem but classes should have Go prefix I think, for easier debug \ etc. (It looks strange to make break BuildSystem::methodName without prefix IMHO). However, I'm not sure if files should contains that "Go" prefix.</p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>For testing, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">create empty project</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">open a folder</span> with golang <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">code</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">project</span> and ch<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">ange in</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">oose "Go</span> project<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">name.kdev4:<br />
M</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> m</span>anager<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">=KDevGoBuildSystem</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">".</span><br />
 <div style="padding: 8px 0;">...</div>1) <span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">~~</span>No "build" \ "install" \ "clean" buttons on right-click menu in projects view. However,<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> "Build selection" works.~~ Fixed,</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">"B</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">now project folders are "b</span>uild <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">selection" works</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">folders" so they are buildable</span>.<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R59 KDevelop Go</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D5188?vs=12841&id=12988" rel="noreferrer">https://phabricator.kde.org/D5188?vs=12841&id=12988</a></div></div><br /><div><strong>BRANCH</strong><div><div>building_feature</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5188" rel="noreferrer">https://phabricator.kde.org/D5188</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>CMakeLists.txt<br />
buildsystem/CMakeLists.txt<br />
buildsystem/builder.cpp<br />
buildsystem/builder.h<br />
buildsystem/buildsystem.cpp<br />
buildsystem/buildsystem.h<br />
buildsystem/buildsystem.json<br />
buildsystem/buildsystem.qrc<br />
buildsystem/buildsystem.rc</div></div></div><br /><div><strong>To: </strong>ematirov, mwolff, KDevelop<br /><strong>Cc: </strong>mwolff, apol, kdevelop-devel, KDevelop<br /></div>