Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

a2ps: work around ./configure issues with Xcode 12 #66293

Closed
wants to merge 1 commit into from

Conversation

mitchblank
Copy link
Contributor

I saw that a2ps had been having build issues and so disabled recently by @iMichka

It is true that the a2ps project itself is long-dormant, but it does seem that it mostly has been continuing to work. Anyway, I dug in to the build failure that we've been seeing with other old ./configure scripts we've been having with Xcode 12 (see #65954 for another example I recently fixed)

This compiles fine for me on Big Sur now; hopefully this will bottle OK now.

@mitchblank
Copy link
Contributor Author

I can't really tell if there is anything to that 10.15 failure... seems to be a ruby gem failure in the test infra itself?!? or am I looking in the wrong place?

@carlocab
Copy link
Member

carlocab commented Dec 5, 2020

It's not your commit. See thread starting from this comment: #66268 (comment)

@carlocab
Copy link
Member

carlocab commented Dec 5, 2020

Actually, no, it's probably this: #66290

I suspect if the test runs again now it should be fine.

@BrewTestBot
Copy link
Member

:shipit: @fxcoudert has triggered a merge.

@BrewTestBot BrewTestBot added the outdated PR was locked due to age label Jan 7, 2021
@Homebrew Homebrew locked as resolved and limited conversation to collaborators Jan 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants