Opened 5 months ago

Closed 4 weeks ago

#68720 closed defect (fixed)

ffmpeg6 @6.1_0: error: use of undeclared identifier 'kVTCompressionPropertyKey_EncoderID'

Reported by: tehcog (tehcog) Owned by: mascguy (Christopher Nielsen)
Priority: Normal Milestone:
Component: ports Version: 2.8.1
Keywords: mavericks Cc:
Port: ffmpeg6

Description

please see attached main.log

Attachments (1)

ffmpeg6_main.log​ (3.8 MB) - added by tehcog (tehcog) 5 months ago.
main.log

Change History (11)

Changed 5 months ago by tehcog (tehcog)

Attachment: ffmpeg6_main.log​ added

main.log

comment:1 Changed 5 months ago by ryandesign (Ryan Carsten Schmidt)

Keywords: mavericks added
Summary: ffmpeg6 6.0_4 < 6.1_0 fails to upgrade on mavericks → ffmpeg6 @6.1_0: error: use of undeclared identifier 'kVTCompressionPropertyKey_EncoderID' 
libavcodec/videotoolboxenc.c:1185:40: error: use of undeclared identifier 'kVTCompressionPropertyKey_EncoderID'; did you mean 'kVTCompressionPropertyKey_Depth'?
                                       kVTCompressionPropertyKey_EncoderID,
                                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                       kVTCompressionPropertyKey_Depth
/System/Library/Frameworks/VideoToolbox.framework/Headers/VTCompressionProperties.h:314:29: note: 'kVTCompressionPropertyKey_Depth' declared here
VT_EXPORT const CFStringRef kVTCompressionPropertyKey_Depth VT_AVAILABLE_STARTING(10_8); // Read/write, CFNumber (CMPixelFormatType), Optional
                            ^
1 error generated.

Indeed kVTCompressionPropertyKey_EncoderID was introduced in macOS 10.13.

comment:3 Changed 5 months ago by mascguy (Christopher Nielsen)

Cc: mascguy removed
Owner: set to mascguy
Status: new → assigned

comment:4 Changed 5 months ago by aeiouaeiouaeiouaeiouaeiouaeiou

Reported in upstream FFmpeg trac: ​https://trac.ffmpeg.org/ticket/10695

comment:5 Changed 5 months ago by aeiouaeiouaeiouaeiouaeiouaeiou

The master branch has recently pushed these two commits:

​https://github.com/FFmpeg/FFmpeg/commit/cb049d377f54f6b747667a93e4b719380c3e9475.patch

​https://github.com/FFmpeg/FFmpeg/commit/d526a34c20647f54a2cd8f5871b7dc24214578fe.patch

Can someone add these patches to portfile and see what happens? I'm not using the affected systems at this time.

comment:6 in reply to:  5 Changed 4 months ago by tehcog (tehcog)

Replying to aeiouaeiouaeiouaeiouaeiouaeiou:

Can someone add these patches to portfile and see what happens? I'm not using the affected systems at this time.

Sure, just tell me how to do it. Will the diff's work as they exists in their commit, or do they need to be modified to work with our port file?

Thanks

comment:7 Changed 4 months ago by tehcog (tehcog)

FYI this is still an issue when upgrading: 

ffmpeg6  6.0_4 < 6.1_1

:info:build libavcodec/videotoolboxenc.c:1185:40: error: use of undeclared identifier 'kVTCompressionPropertyKey_EncoderID'; did you mean 'kVTCompressionPropertyKey_Depth'?
:info:build                                        kVTCompressionPropertyKey_EncoderID,
:info:build                                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
:info:build                                        kVTCompressionPropertyKey_Depth
:info:build /System/Library/Frameworks/VideoToolbox.framework/Headers/VTCompressionProperties.h:314:29: note: 'kVTCompressionPropertyKey_Depth' declared here
:info:build VT_EXPORT const CFStringRef kVTCompressionPropertyKey_Depth VT_AVAILABLE_STARTING(10_8); // Read/write, CFNumber (CMPixelFormatType), Optional
:info:build                             ^
:info:build 1 error generated.

thank you

comment:8 in reply to:  7 ; Changed 4 months ago by mascguy (Christopher Nielsen)

Replying to tehcog:

FYI this is still an issue when upgrading: 

ffmpeg6  6.0_4 < 6.1_1

:info:build libavcodec/videotoolboxenc.c:1185:40: error: use of undeclared identifier 'kVTCompressionPropertyKey_EncoderID'; did you mean 'kVTCompressionPropertyKey_Depth'?
:info:build                                        kVTCompressionPropertyKey_EncoderID,
:info:build                                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
:info:build                                        kVTCompressionPropertyKey_Depth
:info:build /System/Library/Frameworks/VideoToolbox.framework/Headers/VTCompressionProperties.h:314:29: note: 'kVTCompressionPropertyKey_Depth' declared here
:info:build VT_EXPORT const CFStringRef kVTCompressionPropertyKey_Depth VT_AVAILABLE_STARTING(10_8); // Read/write, CFNumber (CMPixelFormatType), Optional
:info:build                             ^
:info:build 1 error generated.

Yes indeed, this is still on my to-do list. Thanks for the reminder!

comment:9 in reply to:  8 Changed 5 weeks ago by aeiouaeiouaeiouaeiouaeiouaeiou

Replying to mascguy:

Yes indeed, this is still on my to-do list.

Bump: these commits still haven't backported to the stable version, but I definitely think they will fix this issue. I can't deploy pre-Sierra VM for tests because it takes too long with my HDD.

comment:10 Changed 4 weeks ago by Christopher Nielsen <mascguy@…>

Resolution: → fixed
Status: assigned → closed

In d2f9f6bb49a41d643b1c975e2bcfd6805e0ca2d5/macports-ports (master):

ffmpeg6: add upstream patch, for earlier macOS releases

  • Failure Specifics: error: use of undeclared identifier 'kVTCompressionPropertyKey_EncoderID'
  • Note: Since this patch is applied unconditionally, rev-bump to rebuild across-the-board.

Fixes: #68720

Note: See TracTickets for help on using tickets.