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

brew 1.10.2 upgrade failure #493

Closed
chenrui333 opened this issue Mar 10, 2020 · 6 comments
Closed

brew 1.10.2 upgrade failure #493

chenrui333 opened this issue Mar 10, 2020 · 6 comments

Comments

@chenrui333
Copy link

@chenrui333 chenrui333 commented Mar 10, 2020

relates to Homebrew/homebrew-core#51435

01:56:44 Making all in tests/unit
01:56:44   CC       switch_event.o
01:56:44   CC       switch_hash.o
01:56:44   CC       switch_ivr_originate.o
01:56:44   CC       switch_utils.o
01:56:44   CC       switch_core.o
01:56:44   CC       switch_console.o
01:56:44   CC       switch_vpx.o
01:56:44   CC       switch_core_file.o
01:56:44   CC       switch_ivr_play_say.o
01:56:44   CC       switch_core_codec.o
01:56:44   CC       switch_rtp.o
01:56:44   CC       switch_xml.o
01:56:44 switch_rtp.c:18:1: error: unknown type name 'uint'; did you mean 'int'?
01:56:44 uint datalen;
01:56:44 ^~~~
01:56:44 int
01:56:44 1 error generated.
01:56:44 make[2]: *** [switch_rtp.o] Error 1
01:56:44 make[2]: *** Waiting for unfinished jobs....
01:56:44 make[1]: *** [all-recursive] Error 1
01:56:44 make: *** [all] Error 2
01:56:44 

full jenkins build log, https://jenkins.brew.sh/job/Homebrew%20Core%20Pull%20Requests/58857/version=catalina/console

@seven1240

This comment has been minimized.

Copy link
Contributor

@seven1240 seven1240 commented Mar 11, 2020

sounds like it's already fixed in the latest master.

@mjerris

This comment has been minimized.

Copy link
Contributor

@mjerris mjerris commented Mar 11, 2020

switch_rtp.c line 18 is in the middle of the license header comment. This does not appear to be code as exists in freeswitch.

@mjerris mjerris closed this Mar 11, 2020
@chenrui333

This comment has been minimized.

Copy link
Author

@chenrui333 chenrui333 commented Mar 11, 2020

sounds like it's already fixed in the latest master.

Sounds like I just need to wait for the next release?

@mjerris

This comment has been minimized.

Copy link
Contributor

@mjerris mjerris commented Mar 11, 2020

it was in the unit test, here is where its fixed:

commit 4bef44b
Author: Dragos Oancea dragos@signalwire.com
Date: Mon Jan 13 18:23:13 2020 +0000

[unit-tests] fix build switch_rtp on MacOS
@mjerris

This comment has been minimized.

Copy link
Contributor

@mjerris mjerris commented Mar 11, 2020

As he said the fix is alreay in tree now.

@chenrui333

This comment has been minimized.

Copy link
Author

@chenrui333 chenrui333 commented Mar 11, 2020

Great, thanks for clarifying it.

Also I just tested with the HEAD, it works well for me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.