Hi Alejandro,
Looks good, but some small points:
On 11 December 2014 at 22:40, Alejandro Hernandez <
alejandro.hernan...@linux.intel.com> wrote:
> - version-going-backwards \
> + version-going-backwards expanded_d \
>
+QAPATHTEST[expanded_d] = "package_qa_check_expanded_d"
>
Rename this to expanded-d for consistency with the other symbols that use -
instead of _.
> + # Variables are actually var_${PN}
>
No need to document idioms, remove this comment.
> + messages["expanded_d"] = "FILES should not
> contain the ${D} variable as it references the local build directory not
> the target filesystem, best solution is to remove the ${D} reference"
>
This doesn't name the package which makes it tricky to find in large builds.
Ross
--
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
