Re: [PATCH v3 0/3] conf: topology: Add the conf for SKL-rt286

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 21 Apr 2016 12:53:04 +0200,
Vinod Koul wrote:
> 
> On Thu, Apr 21, 2016 at 09:19:56AM +0200, Takashi Iwai wrote:
> > On Thu, 21 Apr 2016 08:27:35 +0200,
> > Vinod Koul wrote:
> > > 
> > > This series adds SKL-i2c conf for simple PCM  topology graph
> > > 
> > > Changes in v3:
> > > 
> > >  - remove bin blobs, now they are generated in install step
> > 
> > This isn't good in general.  Usually all installed files are generated
> > in the normal make step, and make-install only copies them.  There are
> > a few reasons:
> > - Invoking a random program as root is a security risk.
> > - This may create a file by root in the working directory, and either
> >   it's not always possible (e.g. on NFS) or it may be left undeletable
> >   after make install.
> 
> Right, that is good point indeed.
> 
> So what is your suggestion then?

Just put the normal make rule and generate it at the normal make
step (only when needed).


Takashi
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux