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

groff: patch for config.h bug #64842

Closed
wants to merge 1 commit into from
Closed

groff: patch for config.h bug #64842

wants to merge 1 commit into from

Conversation

jonchang
Copy link
Contributor

#64785

https://github.com/Homebrew/homebrew-core/runs/1401165131?check_suite_focus=true

  • Have you followed the guidelines for contributing?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

@carlocab
Copy link
Member

Are you working on this actively at the moment? Because I had a look at the formula and can try to work on fixing it. I can make a PR on your fork so you can add the commits here if it helps.

@carlocab
Copy link
Member

carlocab commented Nov 17, 2020

It works! ...after about 100 extra lines of diff patches. It's essentially all the same error.

I decided to just open a new PR to save you the trouble of merging it onto your fork. Hope that's ok. Happy to close my newer PR if you'd rather do it here.

#65042

@carlocab carlocab mentioned this pull request Nov 17, 2020
5 tasks
@jonchang jonchang closed this Nov 18, 2020
BrewTestBot pushed a commit that referenced this pull request Nov 18, 2020
* groff: fix build
  See #65000 (comment)

  Builds on #64842

  - [x] Have you followed the [guidelines for contributing](https://github.com/Homebrew/homebrew-core/blob/HEAD/CONTRIBUTING.md)?
  - [x] Have you checked that there aren't other open [pull requests](https://github.com/Homebrew/homebrew-core/pulls) for the same formula update/change?
  - [x] Have you built your formula locally with `brew install --build-from-source <formula>`, where `<formula>` is the name of the formula you're submitting?
  - [x] Is your test running fine `brew test <formula>`, where `<formula>` is the name of the formula you're submitting?
  - [x] Does your build pass `brew audit --strict <formula>` (after doing `brew install <formula>`)?

  Newly added dependencies are for creating PDF and HTML files, and for
  recognising file encoding.
* Update groff.rb
* Update groff.rb

Closes #65042.

Co-authored-by: Jonathan Chang <[email protected]>
Signed-off-by: Jonathan Chang <[email protected]>
Signed-off-by: BrewTestBot <[email protected]>
@BrewTestBot BrewTestBot added the outdated PR was locked due to age label Dec 18, 2020
@Homebrew Homebrew locked as resolved and limited conversation to collaborators Dec 18, 2020
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.

3 participants