<div dir="ltr"><div class="gmail_extra">Good stuff!</div><div class="gmail_extra"><br></div><div class="gmail_extra">Here are my comments:</div><br>> sudo chown -R user:group /opt/kde<div><br></div><div>I think this should either use $USER or `whoami` rather than "user" ?</div><br><br><div>> Uncomment the last two lines if you also want to build the Applications and PIM suite.<div><br></div><div>This should probably be inside the kdesrc-buildrc itself. Maybe</div><div>it could also have a comment inside about the "make-options -j4"</div><div>needing adjusting and possibly should be mentioned outside too.</div><div><br></div><div>There's also the "ignore-kde-structure" option, should this perhaps</div><div>be set to true by default? I honestly see a little point having that</div><div>structure mirrored on your disk, it's just confusing and takes time</div><div>to find anything.</div><div><br></div><div>Should that be the default, then I'd also suggest to skip the<br>"mkdir -p extragear/utils" in the first steps and just clone directly</div><div>to the sources dir. It's also a bit cumbersome having to go to</div><div>/opt/kde/sources/extragear/utils/kdesrc-build each time you want</div><div>to rebuild things (yeah tab helps but it's still annoying).</div><div><br></div><div>Maaaaaybe the kdesrc-build could/should be linked to</div><div>/opt/kde/install/bin (which is added to $PATH)?  Then</div><div>the kdesrc-buildrc would have to be in ~/.kdesrc-buildrc I think.</div><div><br></div><div>Overall I feel the kdesrc-buildrc could be a bit better documented.</div><br><br></div><div>> #install-session-driver true</div><div><br></div><div>This should be removed altogether or documented why is it there.</div><div><br></div><br>> export QTDIR=/usr/lib/qt<div><br></div><div>As the kdesrc-buildrc mentions that the path should be changed</div><div>if custom qt is being build, probably it could use a comment here</div><div>as well ("Running Plasma" section).</div><div><br></div><br>> You may try going back to a previous code version (git skills required),<div>> or wait sometime for the developers to fix the code.</div><div><br></div><div>"...or try helping with fixing it"? ;)</div><div><br></div><div>Otherwise, thanks for doing that! It rocks.</div><div><br></div><div>Cheers<br><div><div><div><div><div><div class="gmail_extra">-- <br><div class="gmail_signature"><div><span style="color:rgb(102,102,102)">Martin Klapetek | KDE Developer</span></div></div>
</div></div></div></div></div></div></div></div>