On Mon, 2014-05-12 at 23:31 +0000, Paul McGougan wrote: > > -----Original Message----- > > From: David Nyström [mailto:david.c.nyst...@gmail.com] > > > To me, Thats sound like a bug in the dependency parser. > > It should not parse for changes in comments, i.e. in this case > > do_install_append > > comments. > > > > Or perhaps in variable expansion, which could be avoided in commented > > sections. > > Yes, I agree that the dependency parser should be altered. > > I would suggest your 2nd suggestion would be the most appropriate, i.e. do > not perform variable expansion in commented lines.
Its easy to think that at a first glance. What if the variable being expanded expands to something multiline and some of those lines are not comments though? Bitbake's behaviour has always been to expand variables first, then follow with other things like execution. The dependency parsing follows the behaviour and includes them as dependencies because of the above multiline issue. So rightly or wrongly, the code does at least behave consistently at the moment and its not as easy to change as you'd first think. I make no claims about whether multiline expansion is a good thing, just that these issues are there and that they're hard to change as the impact isn't always as you'd expect. The behaviour has been like this for a long time too. Cheers, Richard -- _______________________________________________ yocto mailing list yocto@yoctoproject.org https://lists.yoctoproject.org/listinfo/yocto