On Mon, Sep 8, 2014 at 9:08 AM, Carlos Rafael Giani <[email protected]> wrote: > On 09/08/14 13:18, Richard Purdie wrote: >> >> On Mon, 2014-09-08 at 13:04 +0200, Carlos Rafael Giani wrote: >>> >>> A few firmware files use this filename extension, like korg/k1212.dsp . >>> Create respective packages. >>> >>> Signed-off-by: Carlos Rafael Giani <[email protected]> ... >> Perhaps its time to improve the regex slightly here and use something >> like file_regex='^(.*)\.(bin|fw|cis|dsp)$ rather than multiple calls to >> do_split_packages? > > I had the same thought. However, I figured that perhaps it would be better > to do this in a followup patch, to keep the two distinct changes separate > (add .dsp support + modify regex to the new and improved version).
Agreed. -- Otavio Salvador O.S. Systems http://www.ossystems.com.br http://code.ossystems.com.br Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750 -- _______________________________________________ Openembedded-core mailing list [email protected] http://lists.openembedded.org/mailman/listinfo/openembedded-core
