[MPlayer-dev-eng] Add demux_nut to libmpdemux

Rich Felker dalias at aerifal.cx
Sat Sep 16 21:47:00 CEST 2006


On Sat, Sep 16, 2006 at 12:51:15AM +0200, Luca Barbato wrote:
> Diego Biurrun wrote:
> > On Fri, Sep 15, 2006 at 09:57:33PM +0300, Oded Shimon wrote:
> >> On Fri, Sep 15, 2006 at 08:39:03PM +0200, Luca Barbato wrote:
> >>> Oded Shimon wrote:
> >>>> "libnut" is a grand total of 2 source files and 2 header files.., I 
> >>>> consider even the Makefile redundant, let alone an installation routine...
> >>> just take the ffmpeg makefiles/configure and clean them up, if nobody
> >>> will I'll do during the weekend.
> >> Heh, WAY WAY overkill. I can write the install routine right now, it's 2 
> >> lines...
> >>
> >> install:
> >> 	cp libnut.so /usr/local/lib
> >> 	cp nut.h /usr/local/include
> > 
> > make that
> > 
> > PREFIX = '$(DESTDIR)/usr/local'

No, PREFIX should be just the prefix. DESTDIR should be added
separately so that it still applies even if you override PREFIX. See
how other Makefiles do it..

> > install:
> > 	cp libnut.so $(PREFIX)/lib
> > 	cp nut.h $(PREFIX)/include
> > 
> 
> s/cp/install

Indeed! Using cp to install is not a good idea at all because
permissions are likely to be bogus.

> install -d $(libdir)

Yes.

Rich




More information about the MPlayer-dev-eng mailing list