Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#1750 closed defect (fixed)

Compilation error with newer ffmpeg version

Reported by: ismangil Owned by: nanang
Priority: normal Milestone: release-2.3
Component: pjmedia Version: trunk
Keywords: Cc:
Backport to 1.x milestone: Backported: no

Description (last modified by ismangil)

Newer ffmpeg version changed variable names, resulting in build error:

In file included from ../src/pjmedia/ffmpeg_util.c:26:
../src/pjmedia/ffmpeg_util.h:50:12: warning: declaration of 'enum CodecID' will
      not be visible outside of this function [-Wvisibility]
                                         enum CodecID *codec_id);
                                              ^
../src/pjmedia/ffmpeg_util.h:52:47: warning: declaration of 'enum CodecID' will
      not be visible outside of this function [-Wvisibility]
pj_status_t CodecID_to_pjmedia_format_id(enum CodecID codec_id,
                                              ^
../src/pjmedia/ffmpeg_util.c:65:18: error: field has incomplete type
      'enum CodecID'
    enum CodecID        codec_id;
                        ^

Reported in http://lists.pjsip.org/pipermail/pjsip_lists.pjsip.org/2014-March/017286.html

Attachments (1)

ffmpeg_fix.diff (1.4 KB) - added by ismangil 5 years ago.
Patch to check ffmpeg version

Download all attachments as: .zip

Change History (6)

Changed 5 years ago by ismangil

Patch to check ffmpeg version

comment:1 Changed 5 years ago by ismangil

Contributed patch from original reporter attached

Last edited 5 years ago by ismangil (previous) (diff)

comment:2 Changed 5 years ago by ismangil

  • Description modified (diff)

comment:3 Changed 5 years ago by ismangil

In 4801:

Re #1750 fix for detecting definition of CodecID in newer ffmpeg

comment:4 Changed 5 years ago by ismangil

  • Resolution set to fixed
  • Status changed from new to closed

comment:5 Changed 5 years ago by nanang

In 4808:

Re #1750:

  • Replaced "enum CodecID" with "unsigned" (previous fix raised compile error for some older ffmpeg versions).
  • Handled PIX_FMT_GBR24P hassle.
Note: See TracTickets for help on using tickets.