On Thursday 30 December 2004 17:40, Michael Pyne wrote:
>
> Maybe there is a way to add the $(all_includes) thingy to Benjamin's
> KDE linting scripts?  

Good idea.

> Doesn't sound that hard to achieve.  The 
> following shell code seems to work for me (run in your source dir):
>
> for i in `find . -name 'Makefile.am'`; do
>  grep -l '^\(INCLUDES\|AM_CPPFLAGS\).*\$(all_includes) \+.' $i
> done

Shouldn't this be '...\$(all_includes) *$'. And what about multilines?

> When I ran it in kdepim (just as an example) it came back with this,
> I looked at the Makefile and I'm not really sure if it's actually a
> problem or not though.
>
> ./kpilot/lib/Makefile.am

Me neither but I fixed it a few hours ago.


Cheers,
André
 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<