Tarantool development patches archive
 help / color / mirror / Atom feed
* [Tarantool-patches] [PATCH v1] Remove autoconf tools from packages spec
@ 2020-11-02  6:24 Alexander V. Tikhonov
  2020-11-09  4:44 ` Alexander Turenko
  0 siblings, 1 reply; 2+ messages in thread
From: Alexander V. Tikhonov @ 2020-11-02  6:24 UTC (permalink / raw)
  To: Kirill Yukhin, Alexander Turenko; +Cc: tarantool-patches

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
 
-- 
2.25.1

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [Tarantool-patches] [PATCH v1] Remove autoconf tools from packages spec
  2020-11-02  6:24 [Tarantool-patches] [PATCH v1] Remove autoconf tools from packages spec Alexander V. Tikhonov
@ 2020-11-09  4:44 ` Alexander Turenko
  0 siblings, 0 replies; 2+ messages in thread
From: Alexander Turenko @ 2020-11-09  4:44 UTC (permalink / raw)
  To: Alexander V. Tikhonov; +Cc: tarantool-patches

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

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-11-09  4:44 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-02  6:24 [Tarantool-patches] [PATCH v1] Remove autoconf tools from packages spec Alexander V. Tikhonov
2020-11-09  4:44 ` Alexander Turenko

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox