[Asterisk-code-review] BuildSystem: Raise autoconf version requirement from 2.59 to... (asterisk[master])

[Alexander Traud]

Alexander Traud has posted comments on this change. ( https://gerrit.asterisk.org/7965 )

Change subject: BuildSystem: Raise autoconf version requirement from 2.59 to 2.60a.
......................................................................


Patch Set 1:

Short: No, no, no.

Long explanation:
In the Makefile, that flag indicates the lowest possible version. It is 2.60a and not 2.63 or 2.69. If somebody wants to backport Asterisk to a platform with a terrible old autoconf, he should not be hindered by doing so. The idea of that flag is to give the lowest possible version. Same for menuselect/configure.ac

In Asterisk development, this flag was used to create a lowest common denominator for submissions. Regarding this concern, I do not see much change between 2.61, 2.63 and 2.69. By the way, you are able to test different versions in BSD based Linux distributions like OpenBSD 6.2. There, you can install all version side-by-side, directly via the package manager and the standard distribution, and simply change the taken version via the script bootstrap.sh.

That tracks me to the bootstrap.sh. No, that should not be updated either. Actually, it is a complete mess because all different versions are used there by now. It should have used "2.61" throughout all platforms. However, I am not going to change it to "2.61" (or "2.63" or "2.69") because that is going to disturb developers; they have to install another autoconf version for no reason. That just wastes time (and therefore money) of highly qualified personal, without no reason.

Those platforms which still use 2.60 because of bootstrap.sh (and do not have 2.60a actually) have to create a new issue report. They were using the wrong autoconf version for seven years and ignored the error message. Nevertheless, that is a separate issue.


-- 
To view, visit https://gerrit.asterisk.org/7965
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibca850e2fe0e77d935207bd959bacf7197d7f637
Gerrit-Change-Number: 7965
Gerrit-PatchSet: 1
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Comment-Date: Tue, 16 Jan 2018 12:53:43 +0000
Gerrit-HasComments: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180116/bec84dd4/attachment-0001.html>