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

Mac/iOS: Include default brew-installed FFMPEG directory path #953

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kubark42
Copy link

This reduces build-environment complexity by including default paths for FFPEG as installed by brew. Tested to work both for macOS and iOS builds.

This reduces build-environment complexity by including default paths for
FFPEG as installed by brew. Tested to work both for macOS and iOS builds.
@kubark42
Copy link
Author

P.S. Seems that the CI is failing on the archiving step

wget http://sourceforge.net/projects/sshpass/files/sshpass/1.05/sshpass-1.05.tar.gz/download -O sshpass.tar.gz
--2017-09-26 22:27:53--  http://sourceforge.net/projects/sshpass/files/sshpass/1.05/sshpass-1.05.tar.gz/download
Resolving sourceforge.net... 52.84.125.208, 52.84.125.204, 52.84.125.163, ...
Connecting to sourceforge.net|52.84.125.208|:80... connected.
HTTP request sent, awaiting response... 302 Moved Temporarily
Location: https://sourceforge.net/#!/projects/sshpass/files/sshpass/1.05/sshpass-1.05.tar.gz/download [following]
--2017-09-26 22:27:53--  https://sourceforge.net/
Connecting to sourceforge.net|52.84.125.208|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 652 [text/html]
Saving to: ‘sshpass.tar.gz’
sshpass.tar.gz      100%[===================>]     652  --.-KB/s    in 0s      
2017-09-26 22:27:53 (88.8 MB/s) - ‘sshpass.tar.gz’ saved [652/652]
tar zxf sshpass.tar.gz
tar: Unrecognized archive format
tar: Error exit delayed from previous errors.
shell_session_update
/Users/travis/.travis/job_stages: line 57: shell_session_update: command not found

@wang-bin
Copy link
Owner

it's for macOS only, not iOS

@kubark42
Copy link
Author

kubark42 commented Sep 27, 2017

it's for macOS only, not iOS

@wang-bin I'm unclear if you're referring to the travis CI error, or the PR. FWIW, mac includes both macOS and iOS, whereas macx is exclusively macOS[1]. This code builds both iOS and macOS successfully, although I haven't tried to build and run an iOS app using the outputted QtAV binaries. If there is an easy way to test that, I'm happy to give it a shot.

[1] https://stackoverflow.com/a/18520177

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.

2 participants