[alsa-devel] [PATCH] crec: Correct name of output binary to crec

[Richard Fitzgerald]

On Fri, 2016-11-18 at 09:26 +0530, Vinod Koul wrote:
> On Wed, Nov 16, 2016 at 01:14:21PM +0000, Richard Fitzgerald wrote:
> > 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
> 
> This part I agree. Unfortunately this was not done transparently...
> 
> > 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?
> 
> I dont mind renaming source as well..
> 
> I think bigger question is should we continue crec or crecord, my preference
> is latter.
> 
> > c) it's a nuisance to have to change existing test systems and apps that
> > launch "crec", or to create symlinks from crec->crecord.
> 
> Yeah agree, lets decide on one and stick to that, forever hopefully :)
> 

Ok let's change to crecord for consistency with arecord, and I'll do a
patch to rename the source to match.