Opened 11 years ago

Closed 11 years ago

#42891 closed defect (fixed)

ffmpeg @2.1.4_1 Portfile syntax error on 10.6.8

Reported by: mopihopi Owned by: dbevans (David B. Evans)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: dbevans (David B. Evans), jeremyhu (Jeremy Huddleston Sequoia), cooljeanius (Eric Gallager)
Port: ffmpeg

Description

After a port selfupdate, ffmpeg is no longer recognized as a port on OS X 10.6.8.

$ port info ffmpeg
Error: Port ffmpeg not found
$ 

This is due to an error in the Portfile, introduced in r117917 (and in ffmpeg-devel r117918). Braces are used to group a subexpression, instead of parentheses:

if {${os.major} > 10 || {${os.major} == 10 && [lindex [split ${os.version} .] 1] >= 3}} { 

This corrects the error:

if {${os.major} > 10 || (${os.major} == 10 && [lindex [split ${os.version} .] 1] >= 3)} {

Change History (3)

comment:1 Changed 11 years ago by dbevans (David B. Evans)

Owner: changed from macports-tickets@… to devans@…
Status: newassigned
Version: 2.2.1

Yes, I just forced a build on the build bots (automatic builds still not working) and was able to reproduce this error on 10.6 but works ok on 10.7 and later. I assume this has to do with the version of tcl available on 10.6 as opposed to the others.

Also another build error on ffmpeg-devel after you get by this so will fix all the above ASAP.

Thanks for your report.

comment:2 Changed 11 years ago by cooljeanius (Eric Gallager)

Cc: egall@… added

Cc Me!

comment:3 Changed 11 years ago by dbevans (David B. Evans)

Resolution: fixed
Status: assignedclosed

Fix for ffmpeg committed in r117949.
Fix for ffmpeg-devel committed in r117951.

Both build successfully on all 4 build bots (10.6-10.9).

Note: See TracTickets for help on using tickets.