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

gettext 0.21 #58683

Closed
wants to merge 1 commit into from
Closed

gettext 0.21 #58683

wants to merge 1 commit into from

Conversation

miccal
Copy link
Contributor

@miccal miccal commented Jul 27, 2020

Created with brew bump-formula-pr.

@miccal
Copy link
Contributor Author

miccal commented Jul 27, 2020

The only failure for 10.15 and 10.14 was for agda, which is to be expected.

On 10.13 the failures were for agda and gopass; the gopass failure appears to merely be a time-out issue while running its brew test.

I think this can be merged?

Thanks.

@miccal miccal mentioned this pull request Jul 28, 2020
Rylan12
Rylan12 previously approved these changes Jul 28, 2020
Copy link
Member

@Rylan12 Rylan12 left a comment

Choose a reason for hiding this comment

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

LGTM.

The agda issue should be fine, but I just want to check with @chenrui333 about the gopass failure on 10.13.

Here's the output
==> FAILED
Testing gopass
/usr/bin/sandbox-exec -f /private/tmp/homebrew20200727-17762-yh0nlb.sb ruby -W0 -I $LOAD_PATH -- /usr/local/Homebrew/Library/Homebrew/test.rb /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/gopass.rb --verbose
==> /usr/local/Cellar/gopass/1.9.2_1/bin/gopass version
Killing child processes...
Error: gopass: failed
An exception occurred within a child process:
  Timeout::Error: execution expired
/usr/local/Homebrew/Library/Homebrew/formula_assertions.rb:11:in ``'
/usr/local/Homebrew/Library/Homebrew/formula_assertions.rb:11:in `shell_output'
/usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/gopass.rb:37:in `block in '
/usr/local/Homebrew/Library/Homebrew/formula.rb:1818:in `block (3 levels) in run_test'
/usr/local/Homebrew/Library/Homebrew/utils.rb:474:in `with_env'
/usr/local/Homebrew/Library/Homebrew/formula.rb:1817:in `block (2 levels) in run_test'
/usr/local/Homebrew/Library/Homebrew/formula.rb:900:in `with_logging'
/usr/local/Homebrew/Library/Homebrew/formula.rb:1816:in `block in run_test'
/usr/local/Homebrew/Library/Homebrew/formula.rb:2060:in `block in mktemp'
/usr/local/Homebrew/Library/Homebrew/mktemp.rb:57:in `block in run'
/usr/local/Homebrew/Library/Homebrew/mktemp.rb:57:in `chdir'
/usr/local/Homebrew/Library/Homebrew/mktemp.rb:57:in `run'
/usr/local/Homebrew/Library/Homebrew/formula.rb:2059:in `mktemp'
/usr/local/Homebrew/Library/Homebrew/formula.rb:1810:in `run_test'
/usr/local/Homebrew/Library/Homebrew/test.rb:35:in `block in '
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.6.3_2/lib/ruby/2.6.0/timeout.rb:108:in `timeout'
/usr/local/Homebrew/Library/Homebrew/test.rb:34:in `'

It's pretty clearly a timeout error so I think it's fine.

@miccal
Copy link
Contributor Author

miccal commented Jul 28, 2020

Thanks again @Rylan12.

@SMillerDev
Copy link
Member

It's pretty clearly a timeout error so I think it's fine.

That's not fine, it should be fixed.

@rfinnie rfinnie mentioned this pull request Jul 28, 2020
5 tasks
@Rylan12
Copy link
Member

Rylan12 commented Jul 29, 2020

That's not fine, it should be fixed.

Yes, you're right. Sorry for jumping the gun a little here.

In #58772, there were no timeout issues. I think this means it's either:

  1. A fluke for some reason
  2. An issue with gettext

I don't have a high sierra machine to test in so I'm not sure how to diagnose

@Rylan12 Rylan12 dismissed their stale review July 29, 2020 14:06

CI failures

@miccal
Copy link
Contributor Author

miccal commented Jul 30, 2020

@Rylan12 would you like me to rebase after #58772?

@Rylan12
Copy link
Member

Rylan12 commented Jul 30, 2020

Yeah, let's try that. I'm not really sure how else to diagnose it.

There's only one run going on at the moment so now's probably an okay time to start another long build.

@miccal
Copy link
Contributor Author

miccal commented Jul 30, 2020

@Rylan12 very strange -- the dependencies were not tested this time?

@Rylan12
Copy link
Member

Rylan12 commented Jul 30, 2020

Thanks for pointing this out. It should be fixed in Homebrew/brew#8141

@miccal
Copy link
Contributor Author

miccal commented Jul 31, 2020

@Rylan12 agda is now the only failure in all three runs -- gopass has passed.

@Rylan12
Copy link
Member

Rylan12 commented Jul 31, 2020

Thanks. I'm going to continue with the merge as there's an issue open that needs the new gettext (#58819)

@Bo98 can you at least open a PR with the agda changes, even if it's not ready yet? As mentioned in #58696 (comment), we need an open PR or issue that's specific to agda or we probably won't get around to fixing it.

@BrewTestBot BrewTestBot added the missing license Formula has a missing license which should be added label Jul 31, 2020
@dawidd6 dawidd6 removed the missing license Formula has a missing license which should be added label Jul 31, 2020
@fxcoudert
Copy link
Member

Agda issue reported #58984
Merging

@fxcoudert fxcoudert closed this in a273b28 Aug 1, 2020
@Rylan12
Copy link
Member

Rylan12 commented Aug 1, 2020

Thanks, @miccal (sorry it took so long...)

And thanks, @fxcoudert for creating the issue and merging 👍

@miccal miccal deleted the gettext-0.21 branch August 1, 2020 23:26
@miccal
Copy link
Contributor Author

miccal commented Aug 1, 2020

Thank you @Rylan12 and @fxcoudert.

@aconchillo aconchillo mentioned this pull request Aug 15, 2020
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants