Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 144551 - media-video/mjpegtools-1.8.0-r1 does not create png2yuv
Summary: media-video/mjpegtools-1.8.0-r1 does not create png2yuv
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo Media-video project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-08-20 13:12 UTC by Matthias Schwarzott
Modified: 2006-08-25 05:23 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Matthias Schwarzott gentoo-dev 2006-08-20 13:12:08 UTC 
The Commit 1.16 to mjpegtools-1.8.0-r1.ebuild  broke the ebuild / the configure script of mjpegtools to no longer correctly check for libpng and no longer install /usr/bin/png2yuv.
Comment 1 Matthias Schwarzott gentoo-dev 2006-08-20 13:17:52 UTC 
The associated changelog entry is:
  30 Jul 2006; Diego Petten
Comment 2 Matthias Schwarzott gentoo-dev 2006-08-20 13:17:52 UTC 
The associated changelog entry is:
  30 Jul 2006; Diego Pettenò <flameeyes@gentoo.org>
  +files/mjpegtools-1.8.0-libquicktime.patch, mjpegtools-1.8.0-r1.ebuild:
  Add patch from bug #128619 when using libquicktime 0.9.9, thanks to Mario
  Fetka; also fix quicktime useflag, thanks to Eldad Zack (eldad), both in bug
  #128619.
Comment 3 Matthias Schwarzott gentoo-dev 2006-08-20 14:02:40 UTC 
for latest mjpegtools-1.8.0-r1.ebuild png2yuv is installed when USE="quicktime" (regardless wheather quicktime is really installed or only fooled with --nodeps).

When USE="-quicktime" png2yuv is not installed.
Comment 4 Matthias Schwarzott gentoo-dev 2006-08-23 14:37:58 UTC 
The problem is within this part of configure.ac:
---cut---
have_libquicktime=false
if test x$with_libquicktime != xno ; then
  PKG_CHECK_MODULES(LIBQUICKTIME, [libquicktime >= 0.9.4],
                    [have_libquicktime=true
                     AC_DEFINE(HAVE_LIBQUICKTIME, 1, [libquicktime >= 0.9.4 present])],
                    [have_libquicktime=false])
fi

dnl *********************************************************************
dnl Check for PNG library
dnl   (creates LIBPNG_CFLAGS, LIBPNG_LIBS)
dnl *********************************************************************
have_libpng=false
if test x$with_libpng != xno ; then
  PKG_CHECK_MODULES(LIBPNG, [libpng], [have_libpng=true], [have_libpng=false])
  if test x$have_libpng = xfalse ; then
    PKG_CHECK_MODULES(LIBPNG, [libpng12], [have_libpng=true], [have_libpng=false])
  fi
fi
---cut---

The PKG_CHECK_MODULES-startement for quicktime is the first occurrence of pkg-config related things in configure.ac.
Thus it leads to automagic inclusion of PKG_PROG_PKG_CONFIG-macro.
The problem is that in the created configure-script the expanded PKG_PROG_PKG_CONFIG is not in global-scope but contained inside the statement:

---cut---
if test x$with_libquicktime != xno ; then
...
fi
---cut---

And that means it is not executed when --without-libquicktime is on configure-command-line.

This is the reason that the last commit from flameeyes triggered the problem as before that parameter was not there.
Comment 5 Matthias Schwarzott gentoo-dev 2006-08-25 05:23:31 UTC 
Fixed by adding PKG_PROG_PKG_CONFIG in global scope to configure.ac.
Also added use-flag png and dependency to libpng.