Skip to content
This repository has been archived by the owner on Jan 15, 2025. It is now read-only.

(maint) only force libxml2/libxslt internal mirror if necessary #765

Closed

Conversation

h0tw1r3
Copy link
Contributor

@h0tw1r3 h0tw1r3 commented Nov 18, 2023

unmolested source mirror should be added to artifactory

@h0tw1r3 h0tw1r3 force-pushed the libxml-external-url branch 2 times, most recently from 925c002 to ebe83a0 Compare November 20, 2023 20:26
@h0tw1r3 h0tw1r3 marked this pull request as ready for review January 7, 2024 20:41
@h0tw1r3 h0tw1r3 requested review from a team as code owners January 7, 2024 20:41
@h0tw1r3
Copy link
Contributor Author

h0tw1r3 commented Jan 7, 2024

one step closer to community being able to build foss packages without hacking up vanagon configs

@h0tw1r3
Copy link
Contributor Author

h0tw1r3 commented Feb 14, 2024

tldr; I think just patching out the automake version requirement will work for most (all?) platforms.

Did a little digging. I don't think the custom pre-autoreconf is necessary with the exception of possibly aix 7.1.
The automake 1.15 requirement was added in libxml2 2.10, and it looks like it was arbitrary, moved from the python automake to the base. Then later changed to 1.16.3 for fixing python 3.10 detection. From what I can tell, nothing in the autotools requires a version beyond 1.10.

autotools version patch required to support building on EL8
era operating systems and older.
@joshcooper
Copy link
Contributor

Thanks for looking into this @h0tw1r3! It turns out that autoreconf -fi ends up calling on libtool and we have some older platforms where that fails. I was able to instead switch to using the release libxml2 and libxslt tarballs (see #827) so I'm going to close this.

@joshcooper joshcooper closed this Apr 16, 2024
@h0tw1r3 h0tw1r3 deleted the libxml-external-url branch April 17, 2024 00:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants