transcode 1.1.0 beta1 portfile with quicktime and lzo

[Ryan Schmidt]

Thanks for the update, David. I put your new Portfile into the  
repository in r39706.

http://trac.macports.org/changeset/39706

Yes, you should submit Portfile changes attached to tickets in the  
issue tracker in the future. Pasting submissions into an email is not  
optimal as email transport can sometimes add whitespace or rewrap  
lines, plus emails can easily get forgotten. At least in the issue  
tracker, they can be searched for and found.

It's also best to submit changes as unified diffs, not as full  
Portfiles. That way it's easier for both you and the committer to see  
what you're changing. In the case of this port, you inadvertently  
removed a crucial closing curly bracket in the imagemagick variant. I  
fixed this before committing.

I also dropped the "revision 1" line, since the initial revision of  
any given version of a port should be 0, not 1, and 0 is the default.

You added pkgconfig as a library dependency (as was given in ticket  
#15547), but pkgconfig does not provide any libraries; it's only a  
program used at build time to find dependencies. Thus pkgconfig is  
properly a build dependency, not a library dependency.


I cannot explain the difficulty of the developer of transcode in  
fixing the software to link with ImageMagick since the name change of  
its core library. transcode's configure script does not need code to  
test for both the old name and the new name; all it needs to do is to  
use pkgconfig or Magick-config to discover the library's correct  
name, as I explained earlier:

http://lists.macosforge.org/pipermail/macports-users/2008-August/ 
011149.html


May I also enable the port's parallel build, as requested earlier?

http://lists.macosforge.org/pipermail/macports-users/2008-August/ 
011141.html



On Aug 30, 2008, at 12:30 AM, David Liontooth wrote:

> Transcode 1.1.0 beta1 builds successfully with lzo2 and quicktime,  
> the latter adding among other things the mov import module, which  
> enables transcode to process mp4 files.
>
> I still haven't found an elegant solution to transcode's inability  
> to locate the renamed ImageMagick libraries; upstream is looking at  
> config.log of the latest failure. In the meantime, adding a symlink  
> works:
>
> cd /opt/local/lib
> ln -s libMagickCore.1.dylib libMagick.dylib
>
> Chuck Remes has signaled he is no longer available as co-maintaner  
> of this port; he made all the essential contributions to get the  
> port off the ground.
>
> I enclose an updated portfile; let me know if I should be using  
> trac for this.
>
> Dave
>
> # $Id: Portfile 37840 2008-06-25 16:56:28Z ecronin at macports.org $
>
> PortSystem 1.0
>
> name        transcode
> version     1.1.0beta1
> revision    1
> categories  multimedia
> maintainers liontooth at cogweb.net
> description Video and audio processing tools
> long_description    transcode is a suite of tools, all of which are \
>             command line utilities, for transcoding various \
>             video, audio, and container formats, running on a \
>             platform that supports shared libraries and threads.\
>             \
>             Decoding and encoding is done by modules that are \
>             responsible for feeding transcode with raw video/ \
>             audio streams (import modules) and encoding the frames \
>             (export modules). \
>             \
>             It includes a variety of video and audio filters, \
>             including (but not limited to) video de-interlacing, \
>             audio resampling, framerate conversion, different \
>             resizing algorithms, smoothing, sharpening,  
> denoisifying, \
>             and cutting.
>
> homepage        http://www.transcoding.org
>
> master_sites    http://fromani.exit1.org/
> distname        ${name}-${version}
> use_bzip2       yes
>
> checksums       ${distname}${extract.suffix} \
>                  md5 b84566b618f2b667ce048993692e69db \
>                  sha1 e68f65b6558c97ffc4fef3aaae9fc35a01af4390
>
> depends_lib     port:automake \
>                 port:autoconf \
>                 port:libtool \
> 		port:pkgconfig \
>                 port:ffmpeg \
>                 port:libmpeg2 \
>                 port:libdvdread \
>                 port:jpeg \
>                 port:lame
>
> pre-configure       {
>             system "cd ${worksrcpath} && \
>                     LIBTOOLIZE=glibtoolize autoreconf -i -f"
>             }
>
> platforms       darwin
>
> configure.args  --disable-mmx \
>                 --disable-altivec \
>                 --with-libavcodec-prefix=${prefix} \
>                 --with-libmpeg2-prefix=${prefix} \
>                 --with-libdvdread-prefix=${prefix} \
>                 --with-libjpeg-prefix=${prefix} \
>                 --with-lame-prefix=${prefix}
>
> # Optional components
>
> variant a52dec {
>     depends_lib-append port:a52dec
>     configure.args-append --enable-a52 --enable-a52-default-decoder  
> --with-a52-prefix=${prefix}
> }
>
> variant faac {
>     depends_lib-append port:faac
>     configure.args-append --enable-faac --with-faac-prefix=${prefix}
> }
>
> variant faad2 {
>     depends_lib-append port:faad2
>     configure.args-append --enable-faad --with-faad-prefix=${prefix}
> }
>
> variant freetype {
>     depends_lib-append port:freetype
>     configure.args-append --enable-freetype2 --with-freetype2- 
> prefix=${prefix}
> }
>
> variant imagemagick {
>     depends_lib-append port:ImageMagick
>     configure.args-append --enable-imagemagick --with-imagemagick- 
> prefix=${prefix
> }
>
> variant libdv {
>     depends_lib-append port:libdv
>     configure.args-append --enable-libdv --with-libdv-prefix=${prefix}
> }
>
> variant libogg {
>     depends_lib-append port:libogg
>     configure.args-append --enable-ogg --with-ogg-prefix=${prefix}
> }
>
> variant libquicktime {
>     depends_lib-append port:libquicktime
>     configure.args-append --enable-libquicktime --with-libquicktime- 
> prefix=${prefix}
> }
>
> variant libsdl {
>     depends_lib-append port:libsdl
>     configure.args-append --enable-sdl --with-sdl-prefix=${prefix}
> }
>
> variant libtheora {
>     depends_lib-append port:libtheora
>     configure.args-append --enable-theora --with-theora-prefix=$ 
> {prefix}
> }
>
> variant libvorbis {
>     depends_lib-append port:libvorbis
>     configure.args-append --enable-vorbis --with-vorbis-prefix=$ 
> {prefix}
> }
>
> variant libxml2 {
>     depends_lib-append port:libxml2
>     configure.args-append --enable-libxml2 --with-libxml2-prefix=$ 
> {prefix}
> }
>
> variant lzo {
>     depends_lib-append port:lzo2
>     configure.args-append --enable-lzo --with-lzo-prefix=${prefix}
> }
>
> variant x264 {
>     depends_lib-append port:x264
>     configure.args-append --enable-x264 --with-x264-prefix=${prefix}
> }
>
> variant xvid {
>     depends_lib-append port:XviD
>     configure.args-append --enable-xvid --with-xvid-prefix=${prefix}
> }