15763 2025-02-16 11:15:16 +0000 GO_IMPORT missing sanity check 2025-03-04 12:47:41 +0000 1 1 7 Build System, Metadata & Runtime OE-Core core unspecified x86 Multiple RESOLVED FIXED NEWCOMER Medium normal 5.99 1 kweihmann gavrosc gavrosc meta.mr.watcher meta.watcher randy.macleod richard.purdie --- --- No (bug/feature does not impact docs) oldest_to_newest 101102 0 kweihmann 2025-02-16 11:15:16 +0000 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 101113 1 richard.purdie 2025-02-17 12:51:19 +0000 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. 101226 2 gavrosc 2025-02-25 14:28:51 +0000 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 101227 3 randy.macleod 2025-02-25 15:24:33 +0000 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? 101228 4 gavrosc 2025-02-25 15:31:29 +0000 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 101229 5 gavrosc 2025-02-25 18:12:47 +0000 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 101275 6 gavrosc 2025-03-04 12:41:10 +0000 The patch is merged in the master: https://git.openembedded.org/openembedded-core/commit/?id=374a91204bdaf44067f6b0ae89ed60934751efaa 101276 7 kweihmann 2025-03-04 12:47:41 +0000 @Christos Gavros - very nice solution - really like it - thanks for the patch