https://bugs.gpodder.org/show_bug.cgi?id=585
--- Comment #8 from Caleb Clarke <[email protected]> 2010-03-13 21:12:07 GMT --- lambda512: I was unaware of faad (and hadn't noticed that tags were lost in conversion), but otherwise agree with your commentary. I did say, "the config and sync.py appear to use logic of 'convert all formats or none', so some modification there would be necessary for a proper patch." I was only offering a quick hack since there had been no activity on this ticket. Again, there probably also need to be new "Additional Component" dependencies. Near the top of sync.py shows how existing per-feature oggdec/lame/mplayer dependencies are handled in that regard. I suppose it gets a little complicated with the fallback decoder... it doesn't look like the dependency manager (in services.py) is setup to handle the concept of a fallback dependency. Seems like you would wind up adding an "Additional Component" entry for each individual encoder/decoder, including the fallback(s). That may work against the user-friendliness of the dialog. I like the ideas of flexible conversion and a fallback decoder. I would have preferred to see the Converters class enhanced, but I'll leave such stylistic concerns to more experienced python/gpodder developers. However, dependencies should probably be dynamically-checked (so installing oggdec/lame/etc doesn't require restarting gpodder...), especially since the "Additional Components" display *is* dynamically-checked. Ideally, it seems like things would be setup such that a single line adds full knowledge of a particular encoder/decoder, which is then automatically added to the dependency manager (in some magically-intuitive way), which is later the entity queried each time a conversion is attempted (rather than directly using find_command from the FileConverter class). But that's probably crazy-talk. -- Configure bugmail: https://bugs.gpodder.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes. _______________________________________________ gPodder-Bugs mailing list [email protected] https://lists.berlios.de/mailman/listinfo/gpodder-bugs
