On Mon, Jun 22, 2015 at 02:03:43PM +0200, Klaus Aehlig wrote:
On Mon, Jun 22, 2015 at 01:54:16PM +0200, 'Petr Pudlak' via ganeti-devel wrote:
From: Niklas Hambuechen <[email protected]>

This renames the deprecated `printTestCase` to its replacement
`counterexample`, add provides a CPP-guarded fallback for QuickCheck < 2.7.

Signed-off-by: Niklas Hambuechen <[email protected]>
Reviewed-by: Klaus Aehlig <[email protected]>

Conflicts:
        test/hs/Test/Ganeti/JQScheduler.hs
          - removed file not present in 2.12
        test/hs/Test/Ganeti/SlotMap.hs
          - removed file not present in 2.12
        test/hs/Test/Ganeti/TestCommon.hs
          - added definition for MIN_VERSION_QuickCheck; needs to be
            removed when merging to 2.14
        test/hs/Test/Ganeti/Utils/Statistics.hs
          - added qualified imports to pull counterexample from
            TestCommon

Cherry-picked-from: 077c415a09f8c381ce788ebe6c065d8ccab60564
Signed-off-by: Petr Pudlak <[email protected]>
---
 Makefile.am                                    |   7 ++
 doc/dev-codestyle.rst                          |   6 +-
 test/hs/Test/Ganeti/BasicTypes.hs              |  10 +--
 test/hs/Test/Ganeti/Confd/Utils.hs             |   8 +-
 test/hs/Test/Ganeti/HTools/Backend/Text.hs     |   8 +-
 test/hs/Test/Ganeti/HTools/Cluster.hs          |  14 ++--
 test/hs/Test/Ganeti/HTools/Container.hs        |   2 +-
 test/hs/Test/Ganeti/HTools/Node.hs             |   2 +-
 test/hs/Test/Ganeti/HTools/Types.hs            |   2 +-
 test/hs/Test/Ganeti/Hypervisor/Xen/XmParser.hs |   4 +-
 test/hs/Test/Ganeti/JQueue.hs                  |  28 +++----
 test/hs/Test/Ganeti/Locking/Allocation.hs      |  30 +++----
 test/hs/Test/Ganeti/Locking/Locks.hs           |   4 +-
 test/hs/Test/Ganeti/Locking/Waiting.hs         |  46 +++++------
 test/hs/Test/Ganeti/Network.hs                 |  10 +--
 test/hs/Test/Ganeti/Objects.hs                 |   6 +-
 test/hs/Test/Ganeti/OpCodes.hs                 |   2 +-
 test/hs/Test/Ganeti/Query/Filter.hs            |   8 +-
 test/hs/Test/Ganeti/Query/Language.hs          |   2 +-
 test/hs/Test/Ganeti/Query/Query.hs             | 104 ++++++++++++-------------
 test/hs/Test/Ganeti/Ssconf.hs                  |   2 +-
 test/hs/Test/Ganeti/TestCommon.hs              |  29 ++++++-
 test/hs/Test/Ganeti/Utils.hs                   |  54 ++++++-------
 test/hs/Test/Ganeti/Utils/MultiMap.hs          |   4 +-
 test/hs/Test/Ganeti/Utils/Statistics.hs        |   5 +-
 25 files changed, 214 insertions(+), 183 deletions(-)

diff --git a/Makefile.am b/Makefile.am
index 29c06d8..c9fd066 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -1211,6 +1211,13 @@ hs-pkg-versions:
           -DMONAD_CONTROL_MAJOR=\1 -DMONAD_CONTROL_MINOR=\2 
-DMONAD_CONTROL_REV=\3/'\
          -e 's/^\s*//' \
        >> $@
+       ghc-pkg list --simple-output QuickCheck \
+       | sed -r -e '$$!d' \
+         -e 's/^QuickCheck-([0-9]+(\.[0-9]+)*)/\1 0 0 0/' \
+         -e 's/\./ /g' -e 's/([0-9]+) *([0-9]+) *([0-9]+) .*/\
+          -DQUICKCHECK_MAJOR=\1 -DQUICKCHECK_MINOR=\2 -DQUICKCHECK_REV=\3/'\
+         -e 's/^\s*//' \
+       >> $@

NACK.

This is not a cherry pick, as patch 077c415a09f8c381ce788ebe6c065d8ccab60564 
does not
touch the Makefile.am

If you need that change to the Makefile.am cherry-pick the patch that 
introduced the
change there. Same for the other changes not part of the cherry-picked patch.

--
Klaus Aehlig
Google Germany GmbH, Dienerstr. 12, 80331 Muenchen
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschaeftsfuehrer: Graham Law, Christine Elizabeth Flores

ACK. Since this patch will be replaced by 3 separate ones, I'll resend the whole series.

Reply via email to