Re: [PATCH] crec: Correct name of output binary to crec

[Richard Fitzgerald <rf@opensource.wolfsonmicro.com>]

On Wed, 2016-11-16 at 18:32 +0530, Vinod Koul wrote:
> On Wed, Nov 16, 2016 at 11:06:45AM +0000, Richard Fitzgerald wrote:
> > For some reason when the build system was converted to
> > automake by this patch
> > 
> >     "Convert the build system to autotools"
> > 
> > the name of the output binary for crec was changed
> > from 'crec' to 'crecord'.
> > 
> > This patch corrects it back to 'crec'
> 
> That implies crecord is incorrect, I do not think so.
> 
> Can you explain why you would want to rename this back.
> 
> The motivation for this was to make it proper like cplay. Also similar to
> aplay and arecord.
> 

I don't really see a need for that, or why "crec" is wrong.

In any case if you did want to change that name 

a) it should be an explicit patch, not sneaked in as an undocumented and
unexpected side-effect of some other patch

b) the source file should have been renamed to match, if the name "crec"
is a massive problem then it should also be a problem that
"cplay.c->cplay" but "crec.c->crecord". If it's ok to have the source
file called crec.c what's wrong with the binary it builds being called
crec?

c) it's a nuisance to have to change existing test systems and apps that
launch "crec", or to create symlinks from crec->crecord.