You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2020/09/08 18:51:41 UTC

[GitHub] [incubator-nuttx-apps] btashton commented on pull request #378: interpreters: Initial wasm3 WebAssembly runtime support

btashton commented on pull request #378:
URL: https://github.com/apache/incubator-nuttx-apps/pull/378#issuecomment-689069427


   > How about we mark the license in Kconfig like this:
   > 
   > ```
   > config INTERPRETERS_WASM3
   > 	tristate "WASM3 Webassembly Runtime(MIT)"
   > 	default n
   > ```
   > 
   > So the user know the license impact before enabling the component.
   
   Another option in this vain would be to have an allowed licences Kconfig where we would fail the build if a non-allowed license was enabled.  We could also make the Kconfig dependent, the downside being it might be hard to find features behind a license gate. 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org