Re: [libav-devel] [PATCH] build: Remove dependency and editor backup files also in the doc/ subdirectory.

2011-06-23 Thread Reinhard Tartler
On Thu, Jun 23, 2011 at 19:29:08 (CEST), Diego Biurrun wrote:

> On Thu, Jun 23, 2011 at 03:51:55PM +0100, Måns Rullgård wrote:
>> Reinhard Tartler  writes:
>> 
>> > On Thu, Jun 23, 2011 at 12:41:55 (CEST), Diego Biurrun wrote:
>> >
>> >> On Thu, Jun 23, 2011 at 12:38:03PM +0200, Diego Biurrun wrote:
>> >>> ---
>> >>>  Makefile |2 +-
>> >>>  1 files changed, 1 insertions(+), 1 deletions(-)
>> >>> 
>> >>> --- a/Makefile
>> >>> +++ b/Makefile
>> >>> @@ -172,7 +172,7 @@ testclean:
>> >>>  clean:: testclean
>> >>>  $(RM) $(ALLPROGS)
>> >>>  $(RM) $(CLEANSUFFIXES)
>> >>> -$(RM) doc/*.html doc/*.pod doc/*.1
>> >>> +$(RM) doc/*.html doc/*.pod doc/*.1 doc/*.d doc/*~
>> >>>  $(RM) $(TOOLS)
>> >>>  $(RM) $(CLEANSUFFIXES:%=tools/%)
>> >>
>> >> This is for the 0.7 branch - git send-email does not note the branch.
>> >
>> > Is this commit in master? The commit message does not seem to reference
>> > any commit so I guess no. Shouldn't this be applied in master first,
>> > then?
>> 
>> It's not in master, nor is it applicable there.
>
> Yes.  Mans fixed this bug while splitting off doc/Makefile from the
> top-level Makefile.  I have had this locally for a while, just did
> not send it because it would have conflicted with Mans' patch.
>
> I found it quite annoying while switching branches when testing 0.7
> and it's rather trivial, so I thought it might go in the branch.

In this case, please apply to 0.7

-- 
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel


Re: [libav-devel] [PATCH] build: Remove dependency and editor backup files also in the doc/ subdirectory.

2011-06-23 Thread Diego Biurrun
On Thu, Jun 23, 2011 at 03:51:55PM +0100, Måns Rullgård wrote:
> Reinhard Tartler  writes:
> 
> > On Thu, Jun 23, 2011 at 12:41:55 (CEST), Diego Biurrun wrote:
> >
> >> On Thu, Jun 23, 2011 at 12:38:03PM +0200, Diego Biurrun wrote:
> >>> ---
> >>>  Makefile |2 +-
> >>>  1 files changed, 1 insertions(+), 1 deletions(-)
> >>> 
> >>> --- a/Makefile
> >>> +++ b/Makefile
> >>> @@ -172,7 +172,7 @@ testclean:
> >>>  clean:: testclean
> >>>   $(RM) $(ALLPROGS)
> >>>   $(RM) $(CLEANSUFFIXES)
> >>> - $(RM) doc/*.html doc/*.pod doc/*.1
> >>> + $(RM) doc/*.html doc/*.pod doc/*.1 doc/*.d doc/*~
> >>>   $(RM) $(TOOLS)
> >>>   $(RM) $(CLEANSUFFIXES:%=tools/%)
> >>
> >> This is for the 0.7 branch - git send-email does not note the branch.
> >
> > Is this commit in master? The commit message does not seem to reference
> > any commit so I guess no. Shouldn't this be applied in master first,
> > then?
> 
> It's not in master, nor is it applicable there.

Yes.  Mans fixed this bug while splitting off doc/Makefile from the
top-level Makefile.  I have had this locally for a while, just did
not send it because it would have conflicted with Mans' patch.

I found it quite annoying while switching branches when testing 0.7
and it's rather trivial, so I thought it might go in the branch.

Diego
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel


Re: [libav-devel] [PATCH] build: Remove dependency and editor backup files also in the doc/ subdirectory.

2011-06-23 Thread Måns Rullgård
Reinhard Tartler  writes:

> On Thu, Jun 23, 2011 at 12:41:55 (CEST), Diego Biurrun wrote:
>
>> On Thu, Jun 23, 2011 at 12:38:03PM +0200, Diego Biurrun wrote:
>>> ---
>>>  Makefile |2 +-
>>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>> 
>>> --- a/Makefile
>>> +++ b/Makefile
>>> @@ -172,7 +172,7 @@ testclean:
>>>  clean:: testclean
>>> $(RM) $(ALLPROGS)
>>> $(RM) $(CLEANSUFFIXES)
>>> -   $(RM) doc/*.html doc/*.pod doc/*.1
>>> +   $(RM) doc/*.html doc/*.pod doc/*.1 doc/*.d doc/*~
>>> $(RM) $(TOOLS)
>>> $(RM) $(CLEANSUFFIXES:%=tools/%)
>>
>> This is for the 0.7 branch - git send-email does not note the branch.
>
> Is this commit in master? The commit message does not seem to reference
> any commit so I guess no. Shouldn't this be applied in master first,
> then?

It's not in master, nor is it applicable there.

-- 
Måns Rullgård
m...@mansr.com
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel


Re: [libav-devel] [PATCH] build: Remove dependency and editor backup files also in the doc/ subdirectory.

2011-06-23 Thread Reinhard Tartler
On Thu, Jun 23, 2011 at 12:41:55 (CEST), Diego Biurrun wrote:

> On Thu, Jun 23, 2011 at 12:38:03PM +0200, Diego Biurrun wrote:
>> ---
>>  Makefile |2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>> 
>> --- a/Makefile
>> +++ b/Makefile
>> @@ -172,7 +172,7 @@ testclean:
>>  clean:: testclean
>>  $(RM) $(ALLPROGS)
>>  $(RM) $(CLEANSUFFIXES)
>> -$(RM) doc/*.html doc/*.pod doc/*.1
>> +$(RM) doc/*.html doc/*.pod doc/*.1 doc/*.d doc/*~
>>  $(RM) $(TOOLS)
>>  $(RM) $(CLEANSUFFIXES:%=tools/%)
>
> This is for the 0.7 branch - git send-email does not note the branch.

Is this commit in master? The commit message does not seem to reference
any commit so I guess no. Shouldn't this be applied in master first,
then?

-- 
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel


Re: [libav-devel] [PATCH] build: Remove dependency and editor backup files also in the doc/ subdirectory.

2011-06-23 Thread Diego Biurrun
On Thu, Jun 23, 2011 at 12:38:03PM +0200, Diego Biurrun wrote:
> ---
>  Makefile |2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> --- a/Makefile
> +++ b/Makefile
> @@ -172,7 +172,7 @@ testclean:
>  clean:: testclean
>   $(RM) $(ALLPROGS)
>   $(RM) $(CLEANSUFFIXES)
> - $(RM) doc/*.html doc/*.pod doc/*.1
> + $(RM) doc/*.html doc/*.pod doc/*.1 doc/*.d doc/*~
>   $(RM) $(TOOLS)
>   $(RM) $(CLEANSUFFIXES:%=tools/%)

This is for the 0.7 branch - git send-email does not note the branch.

Diego
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel