this avoids that we blow up on submodule updates inside merges.

Change-Id: I5608554ae4991b379219199c32e2394c3a0b1518
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

git has a tendency to misdetect these files as text, which causes us to
produce quite a lot of noise.

Change-Id: I01e23b26ce3f7f13f606524f611d747ce8db30f1
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

Change-Id: I302a20bf5359a676f2f506010fba7ce30c2d6231
Reviewed-by: Orgad Shaneh <orgads@gmail.com>
Reviewed-by: Jake Petroules <jake.petroules@petroules.com>

this is needed for the digia-internal sanity bot.

on the downside, local sanity checking won't complain about url abuse
anymore (unless the user configures it), but that seems somewhat minor
to me.

Change-Id: Ibbcd2b8ef79668d7f7c7c3a9aa6702c7599c4db1
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

Change-Id: I6862970b6bb5ebb6a171c40a5ed4668d49778b28
Reviewed-by: Orgad Shaneh <orgads@gmail.com>
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com>

Change-Id: I65e0ce4381b98b632dee740f1a3da546691f83ff
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

Change-Id: I2657f34947790eb5ed71f160becbb01399147793
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

it will also be used for watching .ui files.

Change-Id: I74cddca1dcdc8661d397a3f6af814d97a13d4ebd
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

we need to use 'require' - 'use' is compile-time, regardless of
location.

Change-Id: Ie341dde2467206563e6995e1e7f8f3686fcdd8fa
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

Change-Id: Iac47b4480bf427c2524d79484dcf8a9fca22c5a2
Reviewed-by: Simo Fält <simo.falt@digia.com>
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

every repo should have refs/changes (because fetching changes is the
first thing the bot does after cloning a repo), but with the default
config, gc will pack the refs, and the directory vanishes if no refs
were created for too long (which also breaks our cleanup).
don't error out when that happens.

also document how to avoid that situation to start with.

Change-Id: I92d539785a9810c02263f954d9299169d2fa3f2c
Reviewed-by: Orgad Shaneh <orgads@gmail.com>
Reviewed-by: Simo Fält <simo.falt@digia.com>

no point in descending into the .git directories.

Change-Id: I6786ae611fe40498500e4260836c84cbe8359b39
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

otherwise dead mainlines accumulate, as for example when we changed
branching schemes.

Change-Id: Ia5bcb90a079dc62c3c848d139cef5f6d6e0ccbba
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

Change-Id: I8e71a3edd05d9d3c7b7736cd08460acfd8c64f9c
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com>

Change-Id: I22da050bf3da9db107332b1bcdb3c321dc7dd390
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

the pipe can be read in one swoop.

Change-Id: I72f87996d26663d0203404e77d47e130046845b7
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com>

for loops seem to have some really weird scoping rules.

Change-Id: I23d0ee41fb9035545b483c60ad71a9eda50d59b6
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

eat the trailing whitespace, to avoid that the other ws checks trigger
on it again.

Change-Id: Ibaafcf2bf4e0857b61e2f5cd044e4fac84865850
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

Change-Id: I79ded5e4a437e84d444f98e20c70bef023f66398
Reviewed-by: Jędrzej Nowacki <jedrzej.nowacki@digia.com>

produces JSON object that is accepted by gerrit >= 2.6.

Started-by: Orgad Shaneh <orgad.shaneh@audiocodes.com>
Change-Id: I7e9bf99d18beadac8a715e10e4f241c953442d16
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

Change-Id: Ia90528e1747c7f9ea46418c7814584cc9220d483
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

instead of dumping everything straight to stdout, buffer it in a hash
first.
there are multiple advantages:
- in some cases, we'd get line-less complaints only after ones with
  lines, which would look funny. the buffering allows us to sort the
  output.
- different output formats will be easier to implement.
- the output code is cleaner, as it can hold the state more locally.

Change-Id: I0b558ecb58cf4d44a7efbd5a13a79349c55283f9
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

Change-Id: I38ecc06bfc8d24f1df89812a8b86bd78970e095b
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

Change-Id: I19567aadfe2e094fb4d25d3302603e96a9b374fa
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

put it in line with how the style checking is reported.

Change-Id: I9a081a3a9f5c1a06aad30e9e487650333bd812c1
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

Change-Id: Ibf68632ea8a8d9216235b64bda1757d110cf24ad
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

there is no real reason to maintain the order of files, so we can just
use the sorted keys of ws_lines.

Change-Id: Id973fcf1fec6995a931dee665303fb58074bc320
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

Change-Id: I999434f1953896d0bd0d2328176df65b05d63855
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

it actually relies on $_ anyway

Change-Id: I17f48f1d50c20936ffd27a06bfb826be7c3bdd7f
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

they are generated, and we need to check them in.

Change-Id: Ife8bd4a820c5a29ac9e28ce6af102d30577df4cb
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

xcode always omits it.

Change-Id: I37b1e799d08d595aee4c227f9ed7af3eca69a6b8
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

Change-Id: I6ad9c682697e80e175cb19d67d3167c1756cd311
Reviewed-by: Orgad Shaneh <orgads@gmail.com>
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com>

Submodules that have no changes actually show up with space in front of
the sha-1 which would mess up the logic when splitting by spaces.

Change-Id: I15506c6e3780efb29d885ff456ecfac60526c03d
Reviewed-by: Jędrzej Nowacki <jedrzej.nowacki@digia.com>

Change-Id: Ia8fb489715e2ba0f0c29ad76ada20800a1ab5c54
Reviewed-by: Jędrzej Nowacki <jedrzej.nowacki@digia.com>

- Added LICENSE.LGPLv21, LICENSE.LGPLv3 & LICENSE.GPLv2

Change-Id: If5b04ddf25095ae1eb59e25361c88e570bb2b4df
Reviewed-by: Antti Kokko <antti.kokko@digia.com>
Reviewed-by: Tony Sarajärvi <tony.sarajarvi@digia.com>

These files are only edited in Xcode in practice, which uses tabs.

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

Change-Id: I3f12d4ce0653b31ca397167a40303b764bdf7472
Reviewed-by: Sergio Ahumada <sahumada@blackberry.com>

i considered making --draft the default. however, i didn't like the
change of default behavior.

then it occurred to me that automatically detecting WIP subjects like
the sanity bot does would actually fit our current workflow pretty
neatly.
we are looking only at the last commit, as that's likely to be the one
which is WIP. also, in "unbased" push mode, determining the start of the
range to look at would cause quite some overhead.

Change-Id: I70f6f397693f600b6a5d5791f04d9cae12fa8d02
Reviewed-by: Sergio Ahumada <sahumada@blackberry.com>

at the moment the primary effect of this is ensuring early that we are,
in fact, inside a git clone.

Change-Id: I265631a0f4a025b09556ccbb9bfd735973d66d78
Reviewed-by: Sergio Ahumada <sahumada@blackberry.com>

Change-Id: Id49de348f8808a9786616e512636c08eb4d6ef6d
Reviewed-by: Sergio Ahumada <sahumada@blackberry.com>