Bugzilla Bug 42414

Hi!

I patched media-video/mjpegtools-1.6.2.ebuild to do the following:

1. new USE flag 'yv12' to enable DV PAL YV12 read support
2. better CFLAGS/CXXFLAGS handling (-march=i686 problem)
   now compiles only with specified -mcpu and -march flags!

take my included patch and incorporate it into official portage. It compiles and works great!

thanks in advance.

Reproducible: Always
Steps to Reproduce:




--- /usr/portage/media-video/mjpegtools/mjpegtools-1.6.2.ebuild 2004-02-07 
21:09:41.000000000 +0100 
+++ mjpegtools-1.6.2.ebuild     2004-02-21 22:25:15.251619448 +0100 
@@ -11,7 +11,7 @@ 
 LICENSE="as-is" 
 SLOT="1" 
 KEYWORDS="~x86 ~ppc ~amd64" 
-IUSE="gtk avi dv quicktime sdl X 3dnow mmx sse" 
+IUSE="gtk avi dv yv12 quicktime sdl X 3dnow mmx sse" 
 
 DEPEND="media-libs/jpeg 
        >=sys-apps/sed-4 
@@ -52,6 +52,9 @@ 
 
        if [ "`use dv`" ] ; then 
                myconf="${myconf} --with-dv=/usr" 
+        if [ "`use yv12`" ] ; then 
+               myconf="${myconf} --with-dv-yv12" 
+           fi 
        fi 
 
        if [ "`use x86`" ]; then 
@@ -74,7 +77,7 @@ 
                done 
        fi 
 
-       emake -j1 || die "compile problem" 
+       emake -j1 CFLAGS="$CFLAGS" CXXFLAGS="$CXXFLAGS" || die "compile 
problem" 
 } 
 
 src_install() {

Created an attachment (id=26064) [edit]
mjpegtools-1.6.2.ebuild.patch

any chance to get this into portage?

thanks!

Quite good chance, I'd say. Have a look at the buglist assigned to
media-video... please be patient.

please don't forget this "bug". ;-)

remind, remind, remind...

+       emake -j1 CFLAGS="$CFLAGS" CXXFLAGS="$CXXFLAGS" || die "compile 
problem" 

why do we need that?

> emake -j1 CFLAGS="$CFLAGS" CXXFLAGS="$CXXFLAGS" || die "compile problem" 

this is needed to "fix" the CFLAGS. The Configure/Makefile makes some stupid additions (ie -march) to your supplied CFLAGS. This way I enforce *MY* CFLAGS (from /etc/make.conf).

believe me, it works! ;-)

ok, I try to explain the "emake hack".

there're some stupid less intelligent configure-scripts out there. Not only the one of mjpegtools. You say CFLAGS="-march=athlon ..." and the configure makes: CFLAGS="-march=pentium -O3 ... -march=athlon ..." bla blub. ;-)

If I see such stupid habbit, I check the created Makefile. Very often, you can just overwrite it, sometimes you have to add some other stuff. But you can always overwrite the XYZ=ABC settings in the Makefile by simply provide them as options to 'make'. mjpegtools is easy, no futher options needed, you can really overwrite the CFLAGS/CXXFLAGS in the created Makefile with the flags YOU want (yours in /etc/make.conf).

so: make CFLAGS="$CFLAGS" enforces our CFLAGS.

just try it! try to emerge with and without this little trick and watch the compiler output. it really works like a charm!

hmmm... no answer? what now?

No time next week, sorry. Perhaps in May... I know this must be frustrating for
you.

remind again :-/

*sigh* moved out of my focus again. Will check on monday.

Finally. -r2 should be there soon. Thanks for input and patience :-)