Skip to content

Commit

Permalink
Fix GCC7 build error, this statement may fall through. 2.0.263
Browse files Browse the repository at this point in the history
  • Loading branch information
winlinvip committed Apr 5, 2019
1 parent 91c462b commit a554dd8
Show file tree
Hide file tree
Showing 4 changed files with 17 additions and 1 deletion.
2 changes: 2 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -335,6 +335,7 @@ Remark:

## History

* v2.0, 2019-04-05, Merge [#1312][bug #1312], Fix GCC7 build error, this statement may fall through. 2.0.263
* v2.0, 2019-04-05, Merge [#1339][bug #1339], Support HTTP-FLV params. 2.0.262
* v2.0, 2018-12-01, Merge [#1274][bug #1274], Upgrade to FFMPEG 4.1 and X264 157. 2.0.261
* v2.0, 2018-11-11, Merge [#1261][bug #1261], Support `_definst_` for Wowza. 2.0.260
Expand Down Expand Up @@ -1344,6 +1345,7 @@ Winlin
[bug #1261]: https://github.com/ossrs/srs/issues/1261
[bug #1274]: https://github.com/ossrs/srs/pull/1274
[bug #1339]: https://github.com/ossrs/srs/pull/1339
[bug #1312]: https://github.com/ossrs/srs/pull/1312
[bug #xxxxxxxxxx]: https://github.com/ossrs/srs/issues/xxxxxxxxxx

[exo #828]: https://github.com/google/ExoPlayer/pull/828
Expand Down
12 changes: 12 additions & 0 deletions trunk/3rdparty/patches/7.http.parser.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
--- Makefile 2019-04-05 19:43:32.000000000 +0800
+++ ../http-parser-2.1-patch/Makefile 2019-04-05 19:50:26.000000000 +0800
@@ -7,7 +7,8 @@ CPPFLAGS_DEBUG += $(CPPFLAGS_DEBUG_EXTRA
CPPFLAGS_FAST = $(CPPFLAGS) -DHTTP_PARSER_STRICT=0
CPPFLAGS_FAST += $(CPPFLAGS_FAST_EXTRA)

-CFLAGS += -Wall -Wextra -Werror
+# patch by winlin
+CFLAGS += -Wall -Wextra

This comment has been minimized.

Copy link
@winlinvip

winlinvip Apr 5, 2019

Author Member
CFLAGS_DEBUG = $(CFLAGS) -O0 -g $(CFLAGS_DEBUG_EXTRA)
CFLAGS_FAST = $(CFLAGS) -O3 $(CFLAGS_FAST_EXTRA)
CFLAGS_LIB = $(CFLAGS_FAST) -fPIC
2 changes: 2 additions & 0 deletions trunk/auto/depends.sh
Original file line number Diff line number Diff line change
Expand Up @@ -482,6 +482,8 @@ if [ $SRS_HTTP_CORE = YES ]; then
rm -rf ${SRS_OBJS}/http-parser-2.1 && cd ${SRS_OBJS} && unzip -q ../3rdparty/http-parser-2.1.zip &&
cd http-parser-2.1 &&
patch -p0 < ../../3rdparty/patches/2.http.parser.patch &&
# Patch build error for https://github.com/ossrs/srs/pull/1312#issuecomment-480243404
patch -p0 < ../../3rdparty/patches/7.http.parser.patch &&
make package &&
cd .. && rm -rf hp && ln -sf http-parser-2.1 hp &&
cd .. && rm -f ${SRS_OBJS}/_flag.st.hp.tmp
Expand Down
2 changes: 1 addition & 1 deletion trunk/src/core/srs_core.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
// current release version
#define VERSION_MAJOR 2
#define VERSION_MINOR 0
#define VERSION_REVISION 262
#define VERSION_REVISION 263

// generated by configure, only macros.
#include <srs_auto_headers.hpp>
Expand Down

0 comments on commit a554dd8

Please sign in to comment.