Diego Biurrun <[email protected]> writes:

> On Mon, Jul 11, 2011 at 10:40:43AM +0100, Måns Rullgård wrote:
>> Diego Biurrun <[email protected]> writes:
>> 
>> > On Sun, Jul 10, 2011 at 11:21:22PM +0100, Måns Rullgård wrote:
>> >> Diego Biurrun <[email protected]> writes:
>> >> 
>> >> > ---
>> >> >  Makefile             |    3 ++-
>> >> >  libavfilter/Makefile |    3 ++-
>> >> >  libavformat/Makefile |    1 +
>> >> >  subdir.mak           |    4 ++--
>> >> >  tests/Makefile       |    3 ---
>> >> >  5 files changed, 7 insertions(+), 7 deletions(-)
>> >> 
>> >> An alternative to this would be moving seek_test to libavformat/
>> >> There is some logic that as well.
>> >
>> > I disagree here.  It's convenient to have the seek_test line in the
>> > libavformat/ Makefile because it's the simplest way to construct the
>> > correct link command.  But logically seek_test is no more part of
>> > libavformat itself than some of the programs from tools/ that link
>> > against it.
>> 
>> What about all the other *-test programs?
>
> They are not part of the FATE infrastructure,

Sure they are.  Many of them are run as part of FATE, and I intend to
add as many of the remainder as possible.  What are they for if not for
testing?

> unlike seek_test.  This is moving into bikeshed territory.  Can I
> please just push this now and be done with it?

No.

> If you want to move files further around, be my guest afterwards.

You're making further cleanup difficult by making the mess bigger.

-- 
Måns Rullgård
[email protected]
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to