rra@debian.org writes: > - if (grep ($_ eq $target, qw(clean binary-arch build-arch))) { > + if (/^(\S+):(.*)/) { > + $target = $1; > + if (grep ($_ eq $target, @rules)) { > + push (@rules, split (' ', $2)); > + } > + } > + if (grep ($_ eq $target, @rules)) { That won't work for makefiles like this one: clean-bla: foofoo clean-foo: clean-bla barbar clean: clean-foo blabla clean-foo will be added to @rules when the actual clean-foo target was already parsed. Probably better to go over the makefile more than once (hey, lintian performance sucks anyway, and debian/rules is short). So let's simply restart parsing when we change the content of @rules (of cource, we need to check that the targets we want to add are not in there). I just woke up, so it's probably a better idea if someone else actually implements this and checks if I'm right. Marc, who was probably on bad crack when he wrote that simply make parser -- BOFH #311: transient bus protocol violation
Attachment:
pgpQ2L9k3njpl.pgp
Description: PGP signature