Skip to content

Update files for Autotools#1020

Merged
RossBencina merged 7 commits into
PortAudio:masterfrom
gruenich:feature/update-autotools
Aug 21, 2025
Merged

Update files for Autotools#1020
RossBencina merged 7 commits into
PortAudio:masterfrom
gruenich:feature/update-autotools

Conversation

@gruenich
Copy link
Copy Markdown
Contributor

Increase required Autotools version to 2.69.
Fixes #994

Please do intense testing, this might have side effects as my tested was limited to my machine.

@RossBencina
Copy link
Copy Markdown
Collaborator

Thanks for this.

Please do intense testing,

Phil and I will test on Mac and Ubuntu prior to merge.

this might have side effects

Is there any documentation regarding backward compatibility for the version of autotools that you used?

as my tested was limited to my machine.

What is your machine?

@RossBencina
Copy link
Copy Markdown
Collaborator

If anyone else can test this now that would be helpful.

@RossBencina RossBencina added this to the V19.8 milestone May 1, 2025
@gruenich
Copy link
Copy Markdown
Contributor Author

gruenich commented May 2, 2025

  • I tested this change on openSuse Tumbleweed x86-64.
  • I don't really expect side effects as autotools are mainly in maintenance mode picking up with recent compilers and more strict interpretation of C. They are emphasizing backward-compatibility and there was no major version change of any tool included.

@philburk philburk added the P1 Priority: Highest label Jun 12, 2025
@philburk
Copy link
Copy Markdown
Collaborator

I tested this on my new M4 Mac Mini.

git fetch origin pull/1020/head:auto_1020
git checkout auto_1020
make clean && ./configure && make

but got this error:

libtool: Version mismatch error.  This is libtool 2.4.7, but the
libtool: definition of this LT_INIT comes from libtool 2.4.2.
libtool: You should recreate aclocal.m4 with macros from libtool 2.4.7
libtool: and run autoconf again.
make: *** [clean] Error 63

So I changed the order and did the "make clean" AFTER the ./configure.

./configure && make clean && make

That worked fine. So the new configure seems GOOD on Mac OS.
Also the CI tests are passing.

@philburk
Copy link
Copy Markdown
Collaborator

I also tested it on my Raspberry Pi. It was GOOD.

Copy link
Copy Markdown
Collaborator

@philburk philburk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested on Mac OS and Raspbian. It built fine.

@RossBencina
Copy link
Copy Markdown
Collaborator

Hi. I was testing this while working on some new CI scripts in draft PR #1020. The good news is that the updated files seemed to work fine on all tested CI platforms. However I did note a couple of strange warnings when running autoreconf against the files in this (#1020) PR:

autoreconf-2.72: running: /usr/bin/autoconf-2.72 --force
configure.ac:80: warning: back quotes and double quotes must not be escaped in: unknown Windows API \"$api\" (do you need --help?)
configure.ac:499: warning: AC_C_BIGENDIAN should be used with AC_CONFIG_HEADERS

See lines 106 and 107 of the "autoreconf" CI step here: https://github.com/PortAudio/portaudio/actions/runs/16820548423/job/47646396982?pr=1054

Note that the same warnings are not present when running autoreconf against master. See autoreconf step here:
https://github.com/PortAudio/portaudio/actions/runs/16821083986/job/47647942802?pr=1055

I'm not sure what this means yet. What do you think?

@RossBencina
Copy link
Copy Markdown
Collaborator

It would be nice to incorporate this patch: #1056 before the 19.8 release. But perhaps it can be done after this PR is merged, then we autoreconf again. Or you could bring the change into this PR.

@gruenich gruenich force-pushed the feature/update-autotools branch from b906b56 to 715e9f5 Compare August 8, 2025 17:55
@gruenich
Copy link
Copy Markdown
Contributor Author

gruenich commented Aug 8, 2025

Fixed some more warnings, rebased, re-run with automake 1.18.1

@RossBencina
Copy link
Copy Markdown
Collaborator

@gruenich we have merged an edit to configure.in could you please update this PR then we will merge it.

Get rid of some warings as autoconf prefers .ac over .in
Do not increase required autoconf version to 2.72
Autoconf 2.69 was released in 2012.
Fix a warning by more recent versions of Autoconf
Using aclocal 1.18.1, automake 1.18.1, autoconf-archive
@gruenich gruenich force-pushed the feature/update-autotools branch from 715e9f5 to ee6b6ef Compare August 14, 2025 22:50
@gruenich
Copy link
Copy Markdown
Contributor Author

Rebased.

@RossBencina RossBencina merged commit 4d1eb3c into PortAudio:master Aug 21, 2025
11 of 13 checks passed
@RossBencina
Copy link
Copy Markdown
Collaborator

Thank you!

@gruenich gruenich deleted the feature/update-autotools branch October 8, 2025 22:12
RossBencina pushed a commit that referenced this pull request Dec 4, 2025
…d1eb3c (#1098)

As reported in issue #1091, PR #1020, merged as 4d1eb3c, broke ./configure for the C++ bindings. This patch fixes it.

Running autoreconf for cpp bindings modified Makefile.in files. The subtle changes in the paths are required to find required auxiliary files: config.guess config.sub ltmain.sh compile missing install-sh

Fixes #1091
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build-autoconf Autoconf build system P1 Priority: Highest

Projects

None yet

Development

Successfully merging this pull request may close these issues.

configure script and related files generated with outdated autotools

3 participants