tmedicci opened a new pull request, #2992:
URL: https://github.com/apache/nuttx-apps/pull/2992

   ## Summary
   
   * interpreters/python: fix patch to set `_PyRuntime` attribute
   
   This commit also adds the check for the `__NuttX__` macro to the patch file 
that allows setting an attribute to the `_PyRuntime` structure. The `__NuttX__` 
macro is guaranteed to be present when building any application for NuttX.
   
   ## Impact
   <!-- Please fill the following sections with YES/NO and provide a brief 
explanation -->
   
   Impact on user: YES. Local changes to the Python application can use the 
`__NuttX__` macro.
   
   Impact on build: YES.  Local changes to the Python application can use the 
`__NuttX__` macro to enable/disable features.
   
   Impact on hardware: NO
   
   Impact on documentation: NO
   
   Impact on security: NO
   
   Impact on compatibility: NO
   
   ## Testing
   
   It can be tested with `rv-virt:python` (the only board currently supporting 
Python) by checking if the attribute is set to the `_PyRuntime` structure.
   
   ### Building
   Build it with:
   ```
   make -j distclean && ./tools/configure.sh rv-virt:python && make -j$(nproc)
   ```
   
   ### Results
   
   Check the generated `nuttx.map` file: it should shows that the `_PyRuntime` 
structure is at section `.PyRuntime`:
   ```
    .PyRuntime     0x80d70078    0x247a8 
/home/tiago/Documents/work/espressif/projects/nuttx/nuttxspace_alfa/apps/interpreters/python/install/target/libpython3.13.a(pylifecycle.o)
                   0x80d70078                _PyRuntime
   ```
   
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org

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

Reply via email to