dev builds
themes manual
device status forums
mailing lists
IRC bugs
dev guide

Rockbox mail archive

Subject: RE: Makefile cleanup

RE: Makefile cleanup

From: Tony Lenox <>
Date: Sun, 4 Dec 2005 13:55:59 -0000

I also noticed that FAAD (and only FAAD) is being made every time there is a
significant change in apps/firmware.
Is it really needed? (my guess that its referencing some headers it
shouldn't have referenced)


> -----Original Message-----
> From:
> [] On Behalf Of Magnus Holmgren
> Sent: Sunday, December 04, 2005 1:42 PM
> To: Rockbox development
> Subject: Makefile cleanup
> Hi,
> I've been rummaging around the Makefiles a bit lately (mainly the for
> codecs) and I've noticed a couple of things that could be improved:
> * Several codec makefiles contain a rule like this:
> $(OBJDIR)/<codec>/%.o: $(APPSDIR)/codecs/<codec>/%.c
> But it doesn't match, so the build rule in is
> used instead.
> Thus, all the "extra" stuff in the $(CC) command of the rule isn't
> needed and can be removed. It also means that the only thing the
> rule really adds is allowing for a different "echo" line. It also
> means that at least some codecs (e.g., liba52) are built using -O
> rather than the intended -O2.
> So, what to do? Correct the rules and remove the uneccessary stuff,
> or remove the rules completely?
> * codecs/Makefile creates a dependency file, but it isn't used,
> afaict. Should it be used or removed completely?
> * codecs/Makefile define OUTPUT when invoking the codec makefiles.
> Many of the codec makefiles define OUTPUT too (and to the same
> value). Clearly, two definitions is one too many, but which should
> be removed? I'd say the ones in the codec makefiles.
> * I've found one thing that makes "make clean" slow on Cygwin:
> dependency generation. Adding a "ifneq ($(MAKECMDGOALS),clean)"
> around each depenecy file include speeds things up. Is it okay if I
> add this? (After all, there's no point in updating a
> dependency file
> that is about to be deleted anyway.)
> Magnus
Received on 2005-12-04

Page template was last modified "Tue Sep 7 00:00:02 2021" The Rockbox Crew -- Privacy Policy