[FFmpeg-devel] [PATCH] Added an Adobe HTTP Dynamic Streaming (HDS) demuxer

[Michael Niedermayer]

On Fri, Nov 22, 2013 at 05:06:58PM -0700, CORY MCCARTHY wrote:
> Fix broken build when libxml2 is not installed

i think its easier for people to test if you would post a complete
patch instead of patches on top of patches


> 
> ----- Original Message -----
> From: "Michael Niedermayer" <michaelni at gmx.at>
> To: "FFmpeg development discussions and patches" <ffmpeg-devel at ffmpeg.org>
> Sent: Wednesday, November 20, 2013 6:54:56 AM
> Subject: Re: [FFmpeg-devel] [PATCH] Added an Adobe HTTP Dynamic	Streaming	(HDS) demuxer
> 
> On Tue, Nov 19, 2013 at 06:19:45PM -0700, CORY MCCARTHY wrote:
> > Added Changelog entry
> > 
> > Use the existing amf_get_string() and remove duplicate amf_metadata_read_string()
> > Use avio_get_str() and remove f4fbox_read_string()
> > Separated declarations and statements
> > Use sizeof() instead of hard coded literal numbers
> > Fixed typo with "libxml" includes
> > Use av_freep instead of av_free to prevent stale pointers
> > Pass output buffer length to av_strlcat() instead of input buffer length
> > Remove unneeded cast
> > Remove unneeded pix_fmt set
> > Use avpriv_set_pts_info
> > 
> > I attached a patch that shows these changes.
> > I'll work on addressing the rest of the issues outlined below.
> 
> seems failing to build here:
> ffmpeg/libavformat/f4mmanifest.c:32:27: fatal error: libxml/parser.h: No such file or directory
> 
> that is when cross compiling to mingw (or probably anything else that
> doesnt have libxml* installed)
> 
> 
> [...]
> 
> -- 
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
> 
> Concerning the gods, I have no means of knowing whether they exist or not
> or of what sort they may be, because of the obscurity of the subject, and
> the brevity of human life -- Protagoras
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

>  configure |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> dca086e7582c45b4af6f9c9b5ecab8f54784d54f  hdsdec_rev3.patch
>  configure | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/configure b/configure
> index 031a91f..3f0688a 100755
> --- a/configure
> +++ b/configure
> @@ -4203,7 +4203,7 @@ disabled bzlib || check_lib2 bzlib.h BZ2_bzlibVersion -lbz2 || disable bzlib
>  disabled xml2 || {
>      check_pkg_config libxml-2.0 libxml2/libxml/xmlversion.h xmlCheckVersion &&
>      require_pkg_config libxml-2.0 libxml2/libxml/xmlversion.h xmlCheckVersion
> -    } || disable xml
> +    } || disable xml2
>  
>  if ! disabled w32threads && ! enabled pthreads; then
>      check_func_headers "windows.h process.h" _beginthreadex && enable w32threads

> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

You can kill me, but you cannot change the truth.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20131127/7e632797/attachment.asc>