Andraž 'ruskie' Levstik | 10 Nov 2007 19:58

Support for Ogg images

Is there any way to add support for ogg images to easytag?

--
Andraž "ruskie" Levstik
Source Mage GNU/Linux Games grimoire guru
Geek/Hacker/Tinker

Be sure brain is in gear before engaging mouth.

Key id = F4C1F89C
Key fingerprint = 6FF2 8F20 4C9D DB36 B5B6  F134 884D 72CC F4C1 F89C

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
Easytag-mailing mailing list
Easytag-mailing <at> lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/easytag-mailing
Johann Franz | 22 Nov 2007 07:36

Bugfix, changed signature of MP4GetMetadataCoverArt() in libmp4v2-1.6.1

Well, if this hasn't been fixed before - patch applies to easytag 2.1 and 2.1.3:

--- src/mp4_tag.c.orig	2007-11-19 19:27:42.000000000 +0100
+++ src/mp4_tag.c
 <at>  <at>  -185,7 +185,7  <at>  <at>  gboolean Mp4tag_Read_File_Tag (gchar *fi
      * Picture *
      ***********/
     // There is only one picture!
-    if ( MP4GetMetadataCoverArt( mp4file, &coverArt, &coverSize ) )
+    if ( MP4GetMetadataCoverArt( mp4file, &coverArt, &coverSize, 0 ) )
     {
         Picture *pic = Picture_Allocate();
         pic->size = coverSize;
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Easytag-mailing mailing list
Easytag-mailing <at> lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/easytag-mailing
Jérôme COUDERC | 22 Nov 2007 21:00
Picon

Re: Bugfix, changed signature of MP4GetMetadataCoverArt() in libmp4v2-1.6.1

Hi,

    Thanks again for this new patch to enable compilation with the last 
version of mpeg4ip 1.6.1.
But also, I need a solution to be compatible with the previous version.

There is a way to detect the version of mpeg4ip or libmp4v2 to include 
some "#if" in the code?

Regards,
Jerome

Johann Franz wrote, the 22/11/2007 07:36 :
> Well, if this hasn't been fixed before - patch applies to easytag 2.1 and 2.1.3:
>
> --- src/mp4_tag.c.orig	2007-11-19 19:27:42.000000000 +0100
> +++ src/mp4_tag.c
>  <at>  <at>  -185,7 +185,7  <at>  <at>  gboolean Mp4tag_Read_File_Tag (gchar *fi
>       * Picture *
>       ***********/
>      // There is only one picture!
> -    if ( MP4GetMetadataCoverArt( mp4file, &coverArt, &coverSize ) )
> +    if ( MP4GetMetadataCoverArt( mp4file, &coverArt, &coverSize, 0 ) )
>      {
>          Picture *pic = Picture_Allocate();
>          pic->size = coverSize;
>   
>

--

-- 
(Continue reading)

Götz Waschk | 22 Nov 2007 22:45
Picon

Re: Bugfix, changed signature of MP4GetMetadataCoverArt() in libmp4v2-1.6.1

On Nov 22, 2007 9:00 PM, Jérôme COUDERC <easytag <at> gmail.com> wrote:
>  Thanks again for this new patch to enable compilation with the last
> version of mpeg4ip 1.6.1.
> But also, I need a solution to be compatible with the previous version.

Hi Jéròme,

I remember I had promised something like that. I wrote a little
configure check, but maybe some autoconf guru can improve it.

Regards, Götz

--

-- 
AL I:40: Do what thou wilt shall be the whole of the Law.
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Easytag-mailing mailing list
Easytag-mailing <at> lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/easytag-mailing
Jérôme COUDERC | 23 Nov 2007 00:45
Picon

Re: Bugfix, changed signature of MP4GetMetadataCoverArt() in libmp4v2-1.6.1

Thanks Götz,

I've tried the attached patch. It detects correctly the version of the 
library, but I've noted that now it doesn't add "-lmp4v2" to the 
compilator flags....

So the compilation fails...

Do you have the same problem?

Regards,
Jerome

Götz Waschk wrote, the 22.11.2007 22:45 :
> On Nov 22, 2007 9:00 PM, Jérôme COUDERC <easytag <at> gmail.com> wrote:
>   
>>  Thanks again for this new patch to enable compilation with the last
>> version of mpeg4ip 1.6.1.
>> But also, I need a solution to be compatible with the previous version.
>>     
>
> Hi Jéròme,
>
> I remember I had promised something like that. I wrote a little
> configure check, but maybe some autoconf guru can improve it.
>
> Regards, Götz
>
>
>   
(Continue reading)

Götz Waschk | 23 Nov 2007 08:04
Picon

Re: Bugfix, changed signature of MP4GetMetadataCoverArt() in libmp4v2-1.6.1

On Nov 23, 2007 12:45 AM, Jérôme COUDERC <easytag <at> gmail.com> wrote:
> I've tried the attached patch. It detects correctly the version of the
> library, but I've noted that now it doesn't add "-lmp4v2" to the
> compilator flags....
Hi Jérôme,

oh right, autoheader didn't pick up the mp4v2 definition, don't ask me
why. But it is working fine if you rearrange it a bit, see the
attached patch.

Regards, Götz

--

-- 
AL I:40: Do what thou wilt shall be the whole of the Law.
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Easytag-mailing mailing list
Easytag-mailing <at> lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/easytag-mailing
Götz Waschk | 23 Nov 2007 09:01
Picon

Re: Bugfix, changed signature of MP4GetMetadataCoverArt() in libmp4v2-1.6.1

Hi,

a much simpler solution is to just check for the mpeg4ip version, then
you don't need my configure hack at all:

#if MPEG4IP_HEX_VERSION >= 0x10601
    if ( MP4GetMetadataCoverArt( mp4file, &coverArt, &coverSize,0 ) )
#else
    if ( MP4GetMetadataCoverArt( mp4file, &coverArt, &coverSize ) )
#endif

Regards, Götz

--

-- 
AL I:40: Do what thou wilt shall be the whole of the Law.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
EasyTAG | 23 Nov 2007 12:48
Picon

Re: Bugfix, changed signature of MP4GetMetadataCoverArt() in libmp4v2-1.6.1

But this shouldn't work if you install only libmp4v2, and not mpeg4ip...

Personally, the first solution seems good for me...

About the version 1.6.1 : the new API allow to read multiple images but you can save only one picture when writing the tag. Isn't it?

Jerome


2007/11/23, Götz Waschk <goetz.waschk <at> gmail.com >:
Hi,

a much simpler solution is to just check for the mpeg4ip version, then
you don't need my configure hack at all:

#if MPEG4IP_HEX_VERSION >= 0x10601
    if ( MP4GetMetadataCoverArt( mp4file, &coverArt, &coverSize,0 ) )
#else
    if ( MP4GetMetadataCoverArt( mp4file, &coverArt, &coverSize ) )
#endif

Regards, Götz

--
AL I:40: Do what thou wilt shall be the whole of the Law.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Easytag-mailing mailing list
Easytag-mailing <at> lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/easytag-mailing

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Easytag-mailing mailing list
Easytag-mailing <at> lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/easytag-mailing
Götz Waschk | 23 Nov 2007 12:52
Picon

Re: Bugfix, changed signature of MP4GetMetadataCoverArt() in libmp4v2-1.6.1

On Nov 23, 2007 12:48 PM, EasyTAG <easytag <at> gmail.com> wrote:
> But this shouldn't work if you install only libmp4v2, and not mpeg4ip...

Is that even possible? Here, mp4.h includes mpeg4ip.h.

> About the version 1.6.1 : the new API allow to read multiple images but you
> can save only one picture when writing the tag. Isn't it?

I don't think so. Maybe you should try to call
MP4SetMetadataCoverArt() several times?

Regards, Götz

--

-- 
AL I:40: Do what thou wilt shall be the whole of the Law.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
EasyTAG | 23 Nov 2007 14:22
Picon

Re: Bugfix, changed signature of MP4GetMetadataCoverArt() in libmp4v2-1.6.1



2007/11/23, Götz Waschk <goetz.waschk <at> gmail.com>:
On Nov 23, 2007 12:48 PM, EasyTAG <easytag <at> gmail.com> wrote:
> But this shouldn't work if you install only libmp4v2, and not mpeg4ip...

Is that even possible? Here, mp4.h includes mpeg4ip.h.


Some distributions as Fedora use the package from http://resare.com/libmp4v2/
It was build by keeping only the library mp4v2 from mpeg4ip
 

> About the version 1.6.1 : the new API allow to read multiple images but you
> can save only one picture when writing the tag. Isn't it?

I don't think so. Maybe you should try to call
MP4SetMetadataCoverArt() several times?


Already tested... it fails :-(
 

Regards, Götz

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Easytag-mailing mailing list
Easytag-mailing <at> lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/easytag-mailing

Gmane