[Ffmpeg-devel] Makefile consolidation
Diego Biurrun
diego
Mon Feb 13 13:14:28 CET 2006
On Mon, Feb 13, 2006 at 12:17:41PM +0100, Michael Niedermayer wrote:
>
> On Mon, Feb 13, 2006 at 11:00:58AM -0000, M?ns Rullg?rd wrote:
> >
> > Diego Biurrun said:
> > > On Mon, Feb 13, 2006 at 12:18:56AM +0000, M?ns Rullg?rd wrote:
> > >> M?ns Rullg?rd <mru at inprovide.com> writes:
> > >>
> > >> > Diego Biurrun <diego at biurrun.de> writes:
> > >> >
> > >> >> Good. Will you commit the patch?
> > >> >>
> > >> >> I'm fine with seeing this committed. I'll have to adapt the MPlayer
> > >> >> build, though ...
> > >> >
> > >> > OK, I'll commit it shortly.
> > >>
> > >> The style checker is shouting at me about tabs in common.mak. Would
> > >> you please change it to allow tabs in *.mak as well as Makefile? I'd
> > >> do it myself, but seeing how many attempts it took to get it right in
> > >> the first place I'm afraid I'll break something.
> > >
> > > Nah, the trouble was just with the different CVS versions on mphq and my
> > > machine. But having a shell on mphq, this is much easier for me to
> > > test.
> > >
> > > Fixed. Go ahead and commit.
> >
> > It's still complaining about tabs in common.mak.
>
> i think follwoing should fix it ...
> -FILELIST="`echo $FILELIST | sed 's/Makefile\|common.mak\|clean-diff//'`"
> +FILELIST="`echo $FILELIST | sed 's/Makefile\|common.mak\|clean-diff\|style.sh//g'`"
Yes, but just
-FILELIST="`echo $FILELIST | sed 's/Makefile\|common.mak\|clean-diff//'`"
+FILELIST="`echo $FILELIST | sed 's/Makefile\|common.mak\|clean-diff//g'`"
will do as well...
Diego
More information about the ffmpeg-devel
mailing list