[FFmpeg-cvslog] r21226 - in trunk: Makefile common.mak subdir.mak

[Baptiste Coudurier]

On 1/15/10 11:26 AM, Reimar D?ffinger wrote:
> On Fri, Jan 15, 2010 at 08:16:28PM +0100, ramiro wrote:
>> Author: ramiro
>> Date: Fri Jan 15 20:16:28 2010
>> New Revision: 21226
>>
>> Log:
>> Get one step closer to world domination.
>> Remove "make uninstall".
>>
>> Modified:
>>     trunk/Makefile
>>     trunk/common.mak
>>     trunk/subdir.mak
>>
>> Modified: trunk/Makefile
>> ==============================================================================
>> --- trunk/Makefile	Fri Jan 15 19:50:53 2010	(r21225)
>> +++ trunk/Makefile	Fri Jan 15 20:16:28 2010	(r21226)
>> @@ -110,16 +110,8 @@ install-man: $(MANPAGES)
>>   	install -d "$(MANDIR)/man1"
>>   	install -m 644 $(MANPAGES) "$(MANDIR)/man1"
>>
>> -uninstall: uninstall-progs uninstall-data uninstall-man
>> -
>> -uninstall-progs:
>> -	rm -f $(addprefix "$(BINDIR)/", $(ALLPROGS))
>> -
>> -uninstall-data:
>> -	rm -rf "$(DATADIR)"
>> -
>> -uninstall-man:
>> -	rm -f $(addprefix "$(MANDIR)/man1/",$(ALLMANPAGES))
>> +uninstall:
>> +	@echo "I'm sorry, Dave. I'm afraid I can't do that"
>
> I'm all for a joke, but how about at least printing something useful in addition
> (if you really want after a sleep 1) like "Actually this way of uninstalling is
> just too unreliable, consider using a tool like (whatever Mans mentioned) and
> look in .... for files to manually remove").

Well, I'm all for a joke as well. However I do use make uninstall and I 
know why I'm using it. For example I want to remove the .a from a static 
compilation because I want to test shared libs, or vice versa.

So far it has worked decently for me.

This removes the feature and I certainly don't want to rm the files 
manually.

Printing the message is fine with me, but keep the uninstall working.
Thanks for your understanding.

-- 
Baptiste COUDURIER
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
FFmpeg maintainer                                  http://www.ffmpeg.org