Change-Id: Icc4dfd681017ff07ab34444004d0259579fe9949
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com>

messages like "Adding big file (49KiB > 50kB)" are a tad confusing.

Task-number: QTQAINFRA-485
Change-Id: If3ce78b47be8ba2e64075553abab7236044fb57a
Reviewed-by: Sergio Ahumada <sergio.ahumada@digia.com>

Change-Id: Ic15a8917b8b09d45f608123eab83b08079b837d6
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com>

Do not add "-skip qtwebkit" as one single item to the array, since that
will effectively run configure like this:
.\configure.exe "-skip qtwebkit"

This gave the error:
Unknown option -skip qtwebkit

This was found to be a problem with Windows Powershell, not sure if this
is a problem under command.com

Change-Id: Id2a2e8d4705fc46905450e6a7ac503d6c27710aa
Reviewed-by: Andras Becsi <andras.becsi@digia.com>

Building the qtwebkit module works the same way as any other module
these days (i.e. qmake && make), thus using the build-webkit script
for this purpose is obsolete.
In particular when using the qt5_tool the module is actually built
twice: first time with the top-level Makefile, and for the second
time in a final step with build-webkit which fails because the other
build is present. Since the build-webkit script is meant for upstream
developer builds the option can be removed from qt5_tool.
Configure with '-skip qtwebkit' when a webkit build was not requested.

Task-number: QTBUG-30858
Change-Id: Ib6b51b188c9f97064efb834da46de4d03e2a72b7
Reviewed-by: Tor Arne Vestbø <tor.arne.vestbo@digia.com>

Change-Id: Idb5f1f618fe692f2e59266c2d8dfe93722b7037a
Reviewed-by: Jan Arve Sæther <jan-arve.saether@digia.com>
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@digia.com>

It's more or less pointless to complain about these.

Change-Id: I85099cc3a267ab41915552a73c6b6c8c7fcb9ae8
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com>

Change-Id: Ia3640b91ad063da0d719172ae264c903b898135a
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@digia.com>

the priority argument is not the gerrit score, not even inverted ...

Change-Id: Iec9081449bc9113715b68d2f7268939f6b568fef
Reviewed-by: Sergio Ahumada <sergio.ahumada@digia.com>

Detect and complain about various English spelling errors/typos.
The spelling error dataset comes from the Lingua::EN::CommonMistakes
CPAN module; the feature is silently disabled if that module is not
installed.

Change-Id: Ie67984e43803683477549926cb39c61b96549966
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com>

allows specifying a group to get config data from.

Change-Id: I80e8816e73d4690634fc558390cb3601a255f255
Reviewed-by: Sergio Ahumada <sergio.ahumada@digia.com>

It has the standard branches now.

Change-Id: I958d47be0ed3bbe9336b6610cf1ecb06b5e096b2
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@digia.com>

The change is needed by qdoc to work after
https://codereview.qt-project.org/#change,51141



Change-Id: I4e97ef5957d2afc15aa314b73a49f4a22b01029e
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com>
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@digia.com>

Change-Id: I8e2d3f2fa3a914d53cd84b93c32ae5c4424b63d0
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@digia.com>

Change-Id: Ibae5d3655a9a5c76d21fee94b0c63efaba7a4996
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com>
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@digia.com>

Change-Id: I9af54a1c6a75159a5f18f318f5e99332d8437dcc
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@digia.com>

Change-Id: I635f50881c9ef146b6154016105dfe0ccebf1ed4
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@digia.com>

I find myself switching branches too often in different
checkouts.

Change-Id: I59a133a3ac4d88b6c9f08a3a326605a78b8eb36d
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@digia.com>

Change-Id: Idb35f67f54287ab27884fae3bd567b395aab5528
Reviewed-by: Lars Knoll <lars.knoll@digia.com>
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@digia.com>
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com>

Change-Id: Id413cc1ce60e4421cecec612942243ede5280a91
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@digia.com>

At some point "make docs" command started to need qhelpgenerator which
doesn't exist in qtbase, so it can not be used. That was causing
the qdoc bot to fail with a cryptic message:

    make[3]: *** Waiting for unfinished jobs....

Change-Id: I2bf2dd421fc55720a68a4659ddffbd3b789facf1
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com>
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@digia.com>

Change-Id: I670423c6001e2a504df8be80b42ee88566ad3baa
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com>

Change-Id: I964f24da68b08893d775b04a3d6885175f1d34f5
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@digia.com>

getcwd is already provided by POSIX.
If we need other functions from Cwd, we should import only those.

Change-Id: I6f85e4db5670b36355123accc17e4bc7b5573a55
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@digia.com>

this is in line with the other tools, and it allows us to use a sane
copyright header.

Revert "Add a proper license header qdoc-bot"

This reverts commit 30a8d896

.

Change-Id: I2ef4c1cde7fb2ca6ccbe2a8f84b3cb99c47a653c
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com>

It has dev/stable branches now.

Change-Id: I95f367c60469efac875aae2abd451ef582ccb23a
Reviewed-by: Kai Koehne <kai.koehne@digia.com>

Change-Id: I03622bdd124a4d3a116fc7235d27c17636787a43
Reviewed-by: Kai Koehne <kai.koehne@digia.com>

otherwise the inactive login is used

Change-Id: I0545a7bb2ab27e5fa9aad304540ffc97d87657b1
Reviewed-by: Alan Alpert <aalpert@rim.com>

Change-Id: I285d66824f135017f4880607af9f10b33b5ceb2d
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@digia.com>

Use a single variable for the current line number of the commit message
and of the current file, in preparation for new checks which use this information.

Note that $lineno always counts from 1. Previously, $ln counted from 0 while
$lineno counted from 1.

Change-Id: I3a87d29df1378e304a27a7672630f0319dc863a2
Reviewed-by: Sergio Ahumada <sergio.ahumada@digia.com>
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@digia.com>

warn_on is already added by default_pre.prf.

Change-Id: I0da03963edf7f3196638a238b66bd5bfde492795
Reviewed-by: hjk <qthjk@ovi.com>

Change-Id: I6f3eacc4064dbb2a815508f7445441153ebcd82d
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@digia.com>

Change-Id: I5b4d63eb3defbce52f2358b44ec48ecfd7286a07
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@digia.com>

Introduce new config key 'branch' and modify checkout to
create tracking branches.

Change-Id: I4cdafbedda36141de4c15ff6cfb06fb437fecde4
Reviewed-by: Kai Koehne <kai.koehne@digia.com>

Change-Id: I5f9faf58e133cac4cad09d6407d0e61fa5224a40
Reviewed-by: Kai Koehne <kai.koehne@digia.com>

Change-Id: I1cec04e8763c6102b485638058df37d9093ed859
Reviewed-by: Lars Knoll <lars.knoll@digia.com>

git.exe is now in the cmd folder.

Change-Id: I84892a88a79d2dbc97bcf2419cfff095bfc33638
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@digia.com>

Change-Id: I1a88ac25571ad0e23672098dc03c2bb85062f390
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@digia.com>

qdoc-bot should only provide a -1 if something can be staged.
When it is work in progress (WIP or ***) there is no risk of
it getting into Qt, so it should only provide a score of 0
and just tell what the (possible) problems are.

Otherwise it can be misread as a human -1 review to an idea.

Change-Id: Ica2b664eecf375cd0262007a423aafe379c607fd
Reviewed-by: Jędrzej Nowacki <jedrzej.nowacki@digia.com>
Reviewed-by: Stephen Kelly <stephen.kelly@kdab.com>

Change-Id: Ibaec7723a155c760ccafa89fe09c67d6410f06b5
Reviewed-by: Frederik Gladhorn <frederik.gladhorn@digia.com>