Re: [Bug-gnupod] mktunes.pl creates corrupt iTunesDB ?

[H. Langos]

On Fri, Jun 19, 2009 at 10:49:56AM +0200, Richard van den Berg wrote:
> On Fri, June 19, 2009 10:27, H. Langos wrote:
> > Good thing you mention tunes2pod. Please try to make your changes
> > optional.
> > Something like a "--merge" option to express that you are going to merge
> > iTunesDB and GNUtunesDB.xml
> 
> I thought about this myself, but we would have to make this a parameter in
> .gnupodrc instead because of the automatic tunes2pod run by the other
> tools.

Are you sure tunes2pod is run automatically anywhere? I know it is run by
gnupod_INIT but does gnupod_otgsync really call it? Hmm, on second thought 
it surely is. Otherwise people who use gnupod with iTunes would constanly
lose their changes from iTunes.

> The reason that I didn't implement it, is that there is no real
> downside to do the merging. The attributes in iTunesDB always overwrite
> those in GNUtunesDB.xml. If they are the same, no problem. 
> If a file is in iTunesDB and not in GNUtunesDB.xml, no problem either.

One downside: If you don't use the shrink feature (most people will not) you
will not be able to delete attributes (like annoying stuff in the comment
tags) from your other iPod managing application because tunes2pod will 
merge those back in every time.

> Let me know if you absolutely want it to be optional and I'll make it so.

Maybe the merge code should only be active if your memory saving feature is
active?

cheers
-henrik