rock: drop external tuple merger and tuple keydef installation

Crud requires a tuple merger to work, and it currently works with three
kinds of mergers: an embedded Tarantool one, an external one, and a crud
embedded one. The latter implementation is suboptimal but does not depend
on any additional symbols.

The external merger replicates Tarantool's embedded merger, and serves two
purposes:
1. Bring tuple merging to Tarantool 1.10 which does not have an embedded
   merger.
2. Allows for versioning the merger module independently from Tarantool.

Because of potential symbol dependency conflicts, the external merger is
installed in a hacky way, which didn't actually work when crud is built as
part of another rockspec.

Since Tarantool 1.10 can work with the merger embedded to crud, there is no
need in pulling an external merger, so let's drop this dependency.

Instead, add a CI matrix case with external merger installation to test
against it.

All written above also applies to the tuple keydef module.

Needed for tarantool/tarantool#8147

Author: CuriousGeorgiy

.github/workflows/test_on_push.yaml

@@ -20,13 +20,13 @@ jobs:
         tarantool-version: ["1.10.6", "1.10", "2.2", "2.3", "2.4", "2.5", "2.6", "2.7", "2.8", "2.10", "2.11"]
         metrics-version: [""]
         cartridge-version: ["2.8.0"]
-        remove-merger: [false]
+        external-tuple-merger-version: [""]
+        external-tuple-keydef-version: [""]
         include:
           - tarantool-version: "1.10"
             metrics-version: "1.0.0"
             cartridge-version: "2.8.0"
           - tarantool-version: "2.7"
-            remove-merger: true
             cartridge-version: "2.8.0"
           - tarantool-version: "2.10"
             metrics-version: "0.10.0"
@@ -38,6 +38,9 @@ jobs:
           - tarantool-version: "2.11"
             metrics-version: "1.0.0"
             vshard-version: "0.1.24"
+          - tarantool-version: "2.11"
+            external-merger-version: "0.0.5"
+            external-keydef-version: "0.0.4"
           - tarantool-version: "master"
             metrics-version: "1.0.0"
             vshard-version: "0.1.24"
@@ -102,9 +105,13 @@ jobs:
         if: matrix.metrics-version != ''
         run: tt rocks install metrics ${{ matrix.metrics-version }}
 
-      - name: Remove external merger if needed
-        if: ${{ matrix.remove-merger }}
-        run: rm .rocks/lib/tarantool/tuple/merger.so
+      - name: Install external tuple-merger
+        if: matrix.external-tuple-merger-version != ''
+        run: tt rocks install tuple-merger ${{ matrix.external-tuple-merger-version }}
+
+      - name: Install external tuple-keydef
+        if: matrix.external-tuple-keydef-version != ''
+        run: tt rocks install tuple-keydef ${{ matrix.external-tuple-keydef-version }}
 
       # This server starts and listen on 8084 port that is used for tests
       - name: Stop Mono server

CHANGELOG.md

@@ -5,6 +5,12 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+## Unreleased
+
+### Changed
+* Dropped external tuple merger and tuple keydef modules installation from the
+  package build (#390).
+
 ## [1.4.0] - 16-10-23
 
 ### Added

CMakeLists.txt

@@ -95,13 +95,3 @@ install(
   DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}/cartridge
   DESTINATION ${TARANTOOL_INSTALL_LUADIR}
 )
-
-# Don't include to rockspec as some Tarantool versions (e.g. 2.2 and 2.3)
-# don't have symbols required by "tuple-merger" and "tuple-keydef" modules.
-execute_process(
-    COMMAND bash "-c" "tarantoolctl rocks install tuple-keydef 0.0.2"
-)
-
-execute_process(
-    COMMAND bash "-c" "tarantoolctl rocks install tuple-merger 0.0.2"
-)