libsdl1.2_1.2.15-8_amd64.changes ACCEPTED into unstable

[Manuel A. Fernandez Montecelo]

2013/10/28 Felix Geyer <fgeyer at debian.org>:
> On 2013-10-28 14:18, Debian FTP Masters wrote:
>>
>>    * Build-depend on autotools-dev to update config.{sub,guess} and modify
>>      debian/rules to add support for new ports (needed for arm64)
>>    * Use "dh_autoreconf --as-needed" in debian/rules
>
>
> The dh-autoreconf man page says:
>
>> dh_autoreconf is a superset of the autotools-dev debhelper addons, so
>> you do not need --with=autotools_dev if you use --with=autoreconf. In
>> fact, in most cases they should not be used together, as it may lead to
>> unpredictable behaviour.
>
>
> We should probably try to call dh_autoreconf without telling it to use
> the upstream autogen.sh. That doesn't seem to actually regenerate the
> autoconf stuff, see libsdl2-image.

I am trying to follow the advice given in recent bug reports asking to
support arm64, doing it first in libsdl1.2 (and I intended libsdl2 as
well) before going with the modules like mixer. BTW, Reviewing it now,
I think that I put an error when saying "autotools-dev" instead of
using underscore, as recommended in the bug reports (like #727964).

I actually tried to not use autogen.sh but I got the error below, so I
didn't want to also start to dig into that and though that continuing
with autogen.sh for the time being was fine (especially for the 1.2
branches).

If you have a good solution for this, I can work into applying the
solution for all packages, different branches and modules (it will be
easier to maintain things if all modules work in the same way and the
Debian package is homogeneous, I think).

If you don't know or don't have time, I will try to review all this
more closely and see if I can get it working properly (I have a window
of about a week with more time than usual to work in Debian stuff, but
with patchy internet access).


Cheers.
-- 
Manuel A. Fernandez Montecelo <manuel.montezelo at gmail.com>


dh_autoreconf --as-needed
aclocal: warning: autoconf input should be named 'configure.ac', not
'configure.in'
configure.in:462: warning: macro 'AM_PATH_ESD' not found in library
libtoolize: putting auxiliary files in AC_CONFIG_AUX_DIR, `build-scripts'.
libtoolize: copying file `build-scripts/ltmain.sh'
libtoolize: Consider adding `AC_CONFIG_MACRO_DIR([m4])' to configure.in and
libtoolize: rerunning libtoolize, to keep the correct libtool macros in-tree.
libtoolize: Consider adding `-I m4' to ACLOCAL_AMFLAGS in Makefile.am.
aclocal: warning: autoconf input should be named 'configure.ac', not
'configure.in'
configure.in:462: warning: macro 'AM_PATH_ESD' not found in library
configure.in:463: error: possibly undefined macro: AM_PATH_ESD
      If this token and others are legitimate, please use m4_pattern_allow.
      See the Autoconf documentation.
autoreconf: /usr/bin/autoconf failed with exit status: 1
dh_autoreconf: autoreconf -f -i returned exit code 1
make[1]: *** [override_dh_autoreconf] Error 2