GO_IMPORT referenced by at least go-mod and go class, doesn't come with any default value. There's also no sanity check in case the currently undefined variable is not set at all
As per several other bugs opened of this type, we are not adding default values for the sake of it. For the sanity check, a patch would be welcome.
hi Richard I can send a patch for sanity check for this. Couple of questions: - in case sanity check fails, shall we generate a warning or abort building? - shall we check in every build ? Br Christos
Christos, Richard may not reply so I will. Start with making it an error and send the patch to get feedback. Do any go-based recipes currently not set this variable?
Hi Randy I am not in a hurry :)! Its assigned only in go-helloworld_0.1.bb as an example as far as I can see! That's why I asked if we make it only warning! I will do as you said! Br Christos
hi I send a patch! Testing with poky is generating error and aborting the build. If I assign a value in local.conf then is ok. https://lists.openembedded.org/g/openembedded-core/topic/patch_sanity_check/111381998
The patch is merged in the master: https://git.openembedded.org/openembedded-core/commit/?id=374a91204bdaf44067f6b0ae89ed60934751efaa
@Christos Gavros - very nice solution - really like it - thanks for the patch