fox icon indicating copy to clipboard operation
fox copied to clipboard

support additional architectures with autoconf

Open bernstei opened this issue 5 years ago • 9 comments

It would be nice if FoX's autoconf stuff was updated to support newer architectures, e.g. aarch64 and ppc64le. I don't really know much about autoconf, but would hope that it's as simple as updating autoconf itself, rerunning whatever generates files like config.sub, and pushing those back in. Is there anyone who already knows how to do this?

bernstei avatar Feb 10 '21 16:02 bernstei

I guess maybe #4 is the reason this hasn't been done before :(

bernstei avatar Feb 10 '21 16:02 bernstei

I have a tentative patch that makes FoX compatible with modern autoconf 2.69, at the expense of going back to relying entirely on the autoconf-distributed lang.m4 and fortran.m4, and therefore probably losing compatibility with some old compilers, e.g. ones that cannot do their own source preprocessing.

@andreww is that of interest, or would you rather not break that level of backward compatibility, or have another reason to avoid it?

bernstei avatar Feb 10 '21 19:02 bernstei

Yes - #4 (and my lack of time and fading ability with m4) has been the barrier to this kind of thing.

I think the exchange of the loss of some backward compatibility for a more updatable build system is fair. So yes @bernstei, a patch to update to a modern autoconf would be awesome. I need to merge a bunch of other patches and push out a new version, but I don't think this interacts with with any of that.

andreww avatar Feb 12 '21 09:02 andreww

Can this be closed now or is additional work/testing needed?

andreww avatar Feb 23 '21 17:02 andreww

On Feb 23, 2021, at 12:10 PM, Andrew Walker <[email protected]mailto:[email protected]> wrote:

Can this be closed now or is additional work/testing needed?

If you don't mind waiting until the downstream things to update and see if that problem really is fixed, I hope it won't take long, and I'll follow up here once we know.

bernstei avatar Feb 23 '21 17:02 bernstei

Looks like we may also need newer versions of config.sub and/or config.guess. Should I start a new PR for that?

bernstei avatar Feb 23 '21 19:02 bernstei

Yes please. I wonder if it's just a case of using new ones from the autoconf source or if it will need more thought?

andreww avatar Feb 24 '21 08:02 andreww

I think this is now fixed on master.

andreww avatar Mar 04 '21 11:03 andreww

On Mar 4, 2021, at 6:29 AM, Andrew Walker <[email protected]mailto:[email protected]> wrote:

I think this is now fixed on master.

Thanks. I've propagated the changes, and we should have CI results on various architectures in a few days.

Noam

bernstei avatar Mar 04 '21 14:03 bernstei