Review Request: BuilderJob: (eye-candy) flatten composite job hierarchies and generate a meaningful job name

[Ivan Shapovalov]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106461/
-----------------------------------------------------------

(Updated Sept. 22, 2012, 4:21 p.m.)


Review request for KDevelop.


Changes
-------

- Use QVector instead of a QList to store job metadata
- Use QList instead of a QSet to store build actions while generating job's name: this helps to preserve actions ordering and not to have something like "build, configure, install" in job name.

Notice: hunks which call updateJobName() where it is needed are included in this review, so it is dependent on https://git.reviewboard.kde.org/r/106459/.


Description
-------

1. If a BuilderJob is added into another BuilderJob, the latter's subjobs are inserted into the former rather than having two nested composite jobs.
2. A BuilderJob has its name generated from its contents (this is real eye-candy).


Diffs (updated)
-----

  plugins/execute/executeplugin.cpp 817c082 
  plugins/projectmanagerview/projectmanagerviewplugin.cpp 968376d 
  project/builderjob.h 297f187 
  project/builderjob.cpp 44519d3 

Diff: http://git.reviewboard.kde.org/r/106461/diff/


Testing
-------

Existing unit-tests, various hand-testing.


Thanks,

Ivan Shapovalov

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20120922/7490a3fe/attachment.html>