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

SRT: error in libsrt build options #2936

Closed
everything411 opened this issue Feb 24, 2022 · 4 comments · Fixed by #2943
Closed

SRT: error in libsrt build options #2936

everything411 opened this issue Feb 24, 2022 · 4 comments · Fixed by #2943
Assignees
Labels
good first issue Easy to fix issues, good for newcomers SRT It's about SRT protocol. TransByAI Translated by AI/GPT.
Milestone

Comments

@everything411
Copy link
Contributor

everything411 commented Feb 24, 2022

LIBSRT_OPTIONS="--disable-app --enable-static --enable-c++11=0"

The correct option to disable apps is --enable-apps=0, while the --disable-app option is ineffective.
Previously, it was working properly because --enable-c++11=0 set ENABLE_APPS=0 internally in libsrt, disabling apps.

TRANS_BY_GPT3

@winlinvip
Copy link
Member

👍

@winlinvip winlinvip self-assigned this Feb 25, 2022
@winlinvip winlinvip added the SRT It's about SRT protocol. label Feb 25, 2022
@winlinvip winlinvip added this to the 4.0 milestone Feb 25, 2022
@winlinvip winlinvip changed the title error in libsrt build options SRT: error in libsrt build options Feb 25, 2022
@winlinvip winlinvip added the good first issue Easy to fix issues, good for newcomers label Feb 26, 2022
@zhouxiaojun2008
Copy link
Contributor

zhouxiaojun2008 commented Mar 1, 2022

--disable-app' option is equivalent to '--enable-app=0'.

# - you can specify --disable-option instead of --enable-option=0

app is written in C++11, so removing app also disables the compilation of libsrt with C++11. No problem at all.

TRANS_BY_GPT3

@everything411
Copy link
Contributor Author

everything411 commented Mar 2, 2022

The --disable-app option is equivalent to --enable-app=0

# - you can specify --disable-option instead of --enable-option=0

The app is written in C++11, so removing the app also disables the compilation of libsrt with C++11. No problem at all.

The correct option is --enable-apps=0 or --disable-apps. The original option was misspelled, missing an 's', which made the option invalid. The original option was able to successfully disable apps due to the side effect of disabling C++11 compilation, so it should still be modified for the better.

set(ENABLE_APPS OFF)

TRANS_BY_GPT3

@zhouxiaojun2008
Copy link
Contributor

zhouxiaojun2008 commented Mar 3, 2022

The option --disable-app is equivalent to --enable-app=0.

# - you can specify --disable-option instead of --enable-option=0

The app is written in C++11, so removing the app also disables the compilation of libsrt with C++11. There is no problem with that.

The correct options are --enable-apps=0 or --disable-apps. The original option was misspelled, which made it ineffective. The original option was able to successfully disable apps due to the side effect of disabling compilation with C++11. Therefore, it should still be modified for the better.

set(ENABLE_APPS OFF)

Sorry, my mistake. Can I submit a pull request?

TRANS_BY_GPT3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Easy to fix issues, good for newcomers SRT It's about SRT protocol. TransByAI Translated by AI/GPT.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants