[Tarantool-patches] [PATCH v1] Remove autoconf tools from	packages spec
    Alexander Turenko 
    alexander.turenko at tarantool.org
       
    Mon Nov  9 07:44:48 MSK 2020
    
    
  
On Mon, Nov 02, 2020 at 09:24:46AM +0300, Alexander V. Tikhonov wrote:
> Remove autotools from packages spec
Let's add some meaningful prefix. I would use 'build' here.
> After cURL was changed to use cmake in commit
> 
>   2b07601922138fee02554ba69c989bee567e812e ('build: enable cmake in curl build')
> 
> there is no need to set dependencies to autoconf tools. So removed it.
> ---
> 
> Github: https://github.com/tarantool/tarantool/tree/avtikhon/remove-old-curl-tools
> 
>  rpm/tarantool.spec | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/rpm/tarantool.spec b/rpm/tarantool.spec
> index 489b94df8..6049d0e61 100644
> --- a/rpm/tarantool.spec
> +++ b/rpm/tarantool.spec
> @@ -26,9 +26,6 @@ Requires(pre): %{_sbindir}/useradd
>  Requires(pre): %{_sbindir}/groupadd
>  
>  # libcurl dependencies (except ones we have already).
> -BuildRequires: autoconf
> -BuildRequires: automake
> -BuildRequires: libtool
>  BuildRequires: zlib-devel
>  Requires: zlib
How about debian/control?
I see, we unable to safely remove those packages from 'deps_debian'
target in .travis.mk. However, it seems, we can do it for OSX_PKGS,
STATIC_OSX_PKGS and 'deps_freebsd'. If it is so, I would remove them.
I would also ask you to add a docbot comment or file an issue against
tarantool/doc to remove autotools from the 'building from source'
instructions for tarantool 2.6.1 and 2.7+.
NB: The issue, where it was added is
https://github.com/tarantool/doc/issues/868
    
    
More information about the Tarantool-patches
mailing list