delta/qbs.git/src/lib/buildgraph/processcommandexecutor.cpp, branch dynablaster

Fix meaning of "Command.silent".

2013-11-06T11:33:16+00:00

The new, saner meaning is: If this property is set to true, then qbs
will not print the status message (containing the command description).
Accordingly, the default is changed to false. Also, there is now a
check that prints a warning if a non-silent command has an empty
description.

Change-Id: I5a8fd4e2f8901f6e2858946901e8081a6929d43f
Reviewed-by: Joerg Bornemann

Fix incorrect logging output.

2013-09-04T12:44:18+00:00

commandArgsToString will already always start with a space so the extra
one was redundant and incorrect.

empty string arguments are also quoted for clarity (these empty strings
do get passed as actual arguments so this is good for debugging).

Change-Id: Iba1c825a4ce62544b4df5e554c9453d2d2e22b10
Reviewed-by: Christian Kandeler

introduce Transformer::product()

2013-08-12T08:17:57+00:00

Simplifies the calling code.
Remove pointless nullpointer check from jscommandexecutor.

Change-Id: I867181d2b750f32f04376ce860f5dee6555d3e33
Reviewed-by: Christian Kandeler

Some minor improvements to the Error class.

2013-06-18T08:57:25+00:00

- Rename "Error" to "ErrorInfo", to make clear that this class conveys
information about errors, including that there might not actually be
one.
- Rename "ErrorData" to "ErrorItem", to make clear that these are parts
of an aggregate structure.
- Introduce ErrorInfo::hasError() for quick checking of whether an error
occurred.

Change-Id: Icea6ed5240d6d14bd30e9cea189c6babd7004792
Reviewed-by: Joerg Bornemann

Log command-specific environment variables when tracing.

2013-06-12T10:36:50+00:00

In addition to the command line.

Change-Id: I0ef3576b4e3eb810fd923bef23799a7fff62aae6
Reviewed-by: Joerg Bornemann

Use private implementation in API classes.

2013-05-23T13:32:46+00:00

While we do not promise a stable ABI at the moment,
this is conceptually correct and has to be done sooner or
later, with increasing effort the longer we delay it.

Change-Id: Icc98443974e54d377167480ace601db4981ed43c
Reviewed-by: Joerg Bornemann

Help compiler with branch prediction.

2013-04-12T13:10:46+00:00

Mark error checks in potentially performance-sensitive
contexts with Q_UNLIKELY. Actual benefits unknown,
but it cannot hurt.

Change-Id: If3d2375965e6342b5d7a0b46ad2c8a41fd3567d0
Reviewed-by: Joerg Bornemann

Remove structural redundancy in project and product classes.

2013-04-12T12:19:12+00:00

We had two project classes, each holding a list of products
that was structurally identical, except that no BuildProduct
object existed for a disabled ResolvedProduct. The same kind
of duplication also happened for product dependencies.
This patch gets rid of these parallel structures. BuildProject
and BuildProduct are largely being demoted to data holders and are
aggregated by ResolvedProject and ResolvedProduct, respectively.
The resulting project structure should be easier to understand
and maintain.

Change-Id: I68beef60b9e0d62258f6a8337c9015864e18bd80
Reviewed-by: Joerg Bornemann

Make a process' environment configurable in a prepare script.

2013-04-03T10:13:05+00:00

This is needed for applications that are configured via environment
variables rather than command-line arguments.

Change-Id: I114f2bcd65d8551175b35ffbd4912bd37699dc89
Reviewed-by: Joerg Bornemann

replace Q_ASSERT with QBS_ASSERT / QBS_CHECK

2013-02-14T15:29:33+00:00

Change-Id: I9f4d43b9b7be7e4ce814963d1feee81605a9d2eb
Reviewed-by: Christian Kandeler