On Sat, 2012-06-30 at 06:20 -0400, Robert P. J. Day wrote: > On Sat, 30 Jun 2012, Martin Jansa wrote: > > > http://git.openembedded.org/meta-openembedded/commit/?id=040f75eca217c79fed7b881589d9bb36358cffe1 > > http://git.openembedded.org/openembedded-core/commit/?id=d122343362669c683acc4af295971a62cbc823fc > > > > so can be dropped from bitbake.conf too AFAIK > > actually, there are still a number of lines that refer to that > variable: > > $ grep -rw PRIORITY * > meta/conf/bitbake.conf:PRIORITY = "optional" > meta/conf/documentation.conf:PRIORITY[doc] = "Importance of package, default > values are 'optional' or 'needed'." > meta/classes/package_deb.bbclass: fields.append(["Priority: %s\n", > ['PRIORITY']]) > meta/classes/package_ipk.bbclass: fields.append(["Priority: > %s\n", ['PRIORITY']]) > meta/classes/oelint.bbclass: # Test for valid PRIORITY > meta/classes/oelint.bbclass: s = testVar("PRIORITY") > meta/classes/oelint.bbclass: bb.note("PRIORITY '%s' > is not recommended" % s) > meta/classes/oelint.bbclass: bb.note("PRIORITY '%s' > is not recommended, better use '%s'" % (s, newprio)) > meta/classes/oelint.bbclass: bb.note("PRIORITY '%s' is not > recommended" % s) > meta/classes/oelint.bbclass: bb.error("PRIORITY should only > use lower case") > $ > > if i can assume that all of that can be removed, i can do that as a > single patch, again unless someone else wants to handle it if there > are subtleties involved.
I think this one is best left alone as the code stands. We did remove the variable from standard usage, it doesn't obsolete the above code which someone could in theory still use from say a distro configuration perspective. The line in documentation.conf look wrong though, I think "needed" should be "required". Cheers, Richard _______________________________________________ Openembedded-core mailing list [email protected] http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
