Bug 531

Summary: ffmpeg improved conditionals proposal - vdpau / suffix
Product: Fedora Reporter: Nicolas Chauvet <kwizart>
Component: ffmpegAssignee: Dominik 'Rathann' Mierzejewski <dominik>
Status: RESOLVED FIXED    
Severity: normal CC: alexl
Priority: P5    
Version: 14   
Hardware: All   
OS: GNU/Linux   
namespace:
Attachments: build suffix patch

Description Nicolas Chauvet 2009-04-08 19:21:31 CEST
Since vdpau is probably not stable enough for "linking" within a binary repository such our. I would suggest to add the related --with vdpau option

furthermore, it could be interesting to have the same kind of conditionnals (non-default) to build a ffmpeg libraries with a special SONAME. So we (rpmfusion packagers/developpers) can always have the lastest ffmpeg snapshoot without having everything to recompile.

For the record , I was using:
    %{?_with_rpmfusion:--disable-vhook --build-suffix=rpmfusion} \\\

thx
Comment 1 Nicolas Chauvet 2009-04-08 19:29:06 CEST
--disable-vhook can be disabled on recent snapshot
But I don't know if it would matter to pass a special suffix in a non harcoded form (so headers can get installed in a dedicated place, ffmpeg binaries disabled, cpu compatibility optim disabled)

Comment 2 Nicolas Chauvet 2009-05-21 16:46:09 CEST
Created attachment 180 [details]
build suffix patch

This patch active a rpmfusion developer option which make easier to use various ffmpeg libraries at the same time.

When ffmpegsuffix is defined such as:
mock -r fedora-rawhide-x86_64-rpmfusion_free ffmpeg-*.src.rpm --define 'ffmpegsuffix rawhide'
ffmpeg libs SONAME is prefixed by the choosen ffmpegsuffix , simd optimization are also disabled to whatever is the default in ffmpeg.
Comment 3 Nicolas Chauvet 2009-05-21 16:52:56 CEST
Please also activate the --with vdpau conditional.
It should be trivial to implement the same as --with amr despite it will leave the ffmpeg license in the free side. It could be activated by default as the vdpau wrapper is open source, but if ever we need to update the nvidia driver (vdpau backend) to a version incompatible with the current state of the wrapper, we will break the functionality. 

About dirac: I really think it should be activated by default. That would allow it to be usable by transcode, but if you disagree, please at least use any --with dirac build conditional.


Comment 4 Nicolas Chauvet 2009-05-26 14:22:14 CEST
There is a typo on 
@@ -181,17 +186,20 @@
 make %{?_smp_mflags}
 %endif
 popd
+endif

should be read

@@ -181,17 +186,20 @@
 make %{?_smp_mflags}
 %endif
 popd
+%endif



Rathann !?!