Re: [meta-arago][master][PATCH v 1/1] tensorflow-lite: Fix file packaging for headers and symlinks

Pratham Deshmukh <[email protected]> Mon, 4 May 2026 16:53:34 +0530
Newsgroups org.yoctoproject.lists.meta-arago
Message-ID <[email protected]>
--------------k1QhjioOuquynYsFLLEqbQ2N
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 8bit

On 04/05/26 4:50 pm, Pratham Deshmukh wrote:
>
> On 02/05/26 4:44 am, Denys Dmytriyenko wrote:
>> On Wed, Apr 29, 2026 at 02:05:38PM +0530, Pratham Deshmukh via lists.yoctoproject.org wrote:
>>> On 27/04/26 9:05 pm, Denys Dmytriyenko wrote:
>>>> On Thu, Apr 23, 2026 at 01:24:08PM +0530, Pratham Deshmukh via lists.yoctoproject.org wrote:
>>>>> The TensorFlow Lite recipe was missing proper packaging directives for
>>>>> development files, causing two issues:
>>>>>
>>>>> 1. Headers installed to ${includedir} were not being packaged into any
>>>>>     specific package, resulting in missing headers in the final rootfs
>>>>>     despite being built and installed correctly.
>>>>>
>>>>> 2. Unversioned .so symlinks (libtensorflow-lite.so, libtensorflowlite.so)
>>>>>     were incorrectly placed in the main runtime package instead of the
>>>>>     development package, triggering QA error [dev-so].
>>>>>
>>>>> Add FILES:${PN}-dev to properly assign development files:
>>>>>     - ${includedir}/* for TensorFlow Lite headers
>>>>>     - libdir}/lib*.so for unversioned shared library symlinks
>>>>>
>>>>> Signed-off-by: Pratham Deshmukh<[email protected]>
>>>>> ---
>>>>>   .../tensorflow-lite/tensorflow-lite_2.20.0.bb                | 5 +++++
>>>>>   1 file changed, 5 insertions(+)
>>>>>
>>>>> diff --git a/meta-arago-extras/recipes-framework/tensorflow-lite/tensorflow-lite_2.20.0.bb b/meta-arago-extras/recipes-framework/tensorflow-lite/tensorflow-lite_2.20.0.bb
>>>>> index 559ec5ef..75889b7e 100644
>>>>> --- a/meta-arago-extras/recipes-framework/tensorflow-lite/tensorflow-lite_2.20.0.bb
>>>>> +++ b/meta-arago-extras/recipes-framework/tensorflow-lite/tensorflow-lite_2.20.0.bb
>>>>> @@ -229,6 +229,11 @@ FILES:${PN}-tools = " \
>>>>>       /opt/* \
>>>>>   "
>>>>> +FILES:${PN}-dev += " \
>>>>> +    ${includedir}/* \
>>>>> +    ${libdir}/lib*.so \
>>>> These are part of the defaults:
>>>> https://git.openembedded.org/openembedded-core/tree/meta/conf/bitbake.conf#n356
>>>>
>>>> I'm not sure your change even does anything here...
>>>>
>>> Hi Denys
>>>
>>> Thanks for the insights.
>>> Yes there's no need for these changes as it is already
>>>
>>> part of defaults.  Also, without these changes the headers are
>>>
>>> included in the rootfs. So this patch can be ignored/rejected.
>> This part I did not understand - normally you don't want the header files to
>> be included in the rootfs, only into the SDK...
>>
>> Can you please double-check this works as expected?
>>
> The reason is to support developing user-space applications related
>
> to TFLite directly on the target device. This provides additional 
> flexibility
>
> for users who want to compile and test TFLite-based applications
>
> on-target without solely relying on the SDK.
>
> The approach is implemented in the meta-tisdk layer [0]
>
> [0] https://github.com/TexasInstruments/meta-tisdk/pull/179
>
>
Attached incorrect patch link in the previous message. Please check the 
correct one [0]


[0] 
https://github.com/TexasInstruments/meta-tisdk/pull/173/changes#diff-50da8e5c6055ea370e9556e0ffed6bf9ac1e1331b5eecd6a3bd42f63f23294a6R5 

>>> Thanks & Regards,
>>> Pratham
>>>
>>>>> +"
>>>>> +
>>>>>   FILES:python3-${PN} = " \
>>>>>       ${PYTHON_SITEPACKAGES_DIR}/tflite_runtime \
>>>>>       ${PYTHON_SITEPACKAGES_DIR}/tflite_runtime.egg-info \
>>>>> -- 
>>>>> 2.34.1
--------------k1QhjioOuquynYsFLLEqbQ2N
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: 8bit

<!DOCTYPE html>
<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
  </head>
  <body>
    <p>
</p>
    <div class="moz-cite-prefix">On 04/05/26 4:50 pm, Pratham Deshmukh
      wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:[email protected]">
      <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
      <p>
</p>
      <div class="moz-cite-prefix">On 02/05/26 4:44 am, Denys
        Dmytriyenko wrote:<br>
      </div>
      <blockquote type="cite" cite="mid:[email protected]">
        <pre wrap="" class="moz-quote-pre">On Wed, Apr 29, 2026 at 02:05:38PM +0530, Pratham Deshmukh via lists.yoctoproject.org wrote:
</pre>
        <blockquote type="cite">
          <pre wrap="" class="moz-quote-pre">On 27/04/26 9:05 pm, Denys Dmytriyenko wrote:
</pre>
          <blockquote type="cite">
            <pre wrap="" class="moz-quote-pre">On Thu, Apr 23, 2026 at 01:24:08PM +0530, Pratham Deshmukh via lists.yoctoproject.org wrote:
</pre>
            <blockquote type="cite">
              <pre wrap="" class="moz-quote-pre">The TensorFlow Lite recipe was missing proper packaging directives for
development files, causing two issues:

1. Headers installed to ${includedir} were not being packaged into any
   specific package, resulting in missing headers in the final rootfs
   despite being built and installed correctly.

2. Unversioned .so symlinks (libtensorflow-lite.so, libtensorflowlite.so)
   were incorrectly placed in the main runtime package instead of the
   development package, triggering QA error [dev-so].

Add FILES:${PN}-dev to properly assign development files:
   - ${includedir}/* for TensorFlow Lite headers
   - libdir}/lib*.so for unversioned shared library symlinks

Signed-off-by: Pratham Deshmukh <a class="moz-txt-link-rfc2396E"
              href="mailto:[email protected]" moz-do-not-send="true">&lt;[email protected]&gt;</a>
---
 .../tensorflow-lite/tensorflow-lite_2.20.0.bb                | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/meta-arago-extras/recipes-framework/tensorflow-lite/tensorflow-lite_2.20.0.bb b/meta-arago-extras/recipes-framework/tensorflow-lite/tensorflow-lite_2.20.0.bb
index 559ec5ef..75889b7e 100644
--- a/meta-arago-extras/recipes-framework/tensorflow-lite/tensorflow-lite_2.20.0.bb
+++ b/meta-arago-extras/recipes-framework/tensorflow-lite/tensorflow-lite_2.20.0.bb
@@ -229,6 +229,11 @@ FILES:${PN}-tools = " \
     /opt/* \
 "
+FILES:${PN}-dev += " \
+    ${includedir}/* \
+    ${libdir}/lib*.so \
</pre>
            </blockquote>
            <pre wrap="" class="moz-quote-pre">These are part of the defaults:
<a class="moz-txt-link-freetext"
href="https://git.openembedded.org/openembedded-core/tree/meta/conf/bitbake.conf#n356"
            moz-do-not-send="true">https://git.openembedded.org/openembedded-core/tree/meta/conf/bitbake.conf#n356</a>

I'm not sure your change even does anything here...

</pre>
          </blockquote>
          <pre wrap="" class="moz-quote-pre">Hi Denys

Thanks for the insights.
Yes there's no need for these changes as it is already

part of defaults.  Also, without these changes the headers are

included in the rootfs. So this patch can be ignored/rejected.
</pre>
        </blockquote>
        <pre wrap="" class="moz-quote-pre">This part I did not understand - normally you don't want the header files to 
be included in the rootfs, only into the SDK...

Can you please double-check this works as expected?

</pre>
      </blockquote>
      <p
style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;">The reason is to support developing user-space applications related </p>
      <p
style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;">to TFLite directly on the target device. This provides additional flexibility </p>
      <p
style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;">for users who want to compile and test TFLite-based applications </p>
      <p
style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;">on-target without solely relying on the SDK.</p>
      <p
style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;">
</p>
      <p
style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;">The approach is implemented in the meta-tisdk layer [0]</p>
      <p
style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;">
</p>
      <p
style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px;">[0] <a
      class="moz-txt-link-freetext"
      href="https://github.com/TexasInstruments/meta-tisdk/pull/179"
      moz-do-not-send="true">https://github.com/TexasInstruments/meta-tisdk/pull/179</a></p>
      <style type="text/css">p, li { white-space: pre-wrap; }</style><br>
    </blockquote>
    <p>Attached incorrect patch link in the previous message. Please check the correct one [0]</p>
    <br>
    [0]
<a class="moz-txt-link-freetext" href="https://github.com/TexasInstruments/meta-tisdk/pull/173/changes#diff-50da8e5c6055ea370e9556e0ffed6bf9ac1e1331b5eecd6a3bd42f63f23294a6R5">https://github.com/TexasInstruments/meta-tisdk/pull/173/changes#diff-50da8e5c6055ea370e9556e0ffed6bf9ac1e1331b5eecd6a3bd42f63f23294a6R5</a>
    <blockquote type="cite"
      cite="mid:[email protected]">
      <blockquote type="cite" cite="mid:[email protected]">
        <blockquote type="cite">
          <pre wrap="" class="moz-quote-pre">Thanks &amp; Regards,
Pratham

</pre>
          <blockquote type="cite">
            <blockquote type="cite">
              <pre wrap="" class="moz-quote-pre">+"
+
 FILES:python3-${PN} = " \
     ${PYTHON_SITEPACKAGES_DIR}/tflite_runtime \
     ${PYTHON_SITEPACKAGES_DIR}/tflite_runtime.egg-info \
-- 
2.34.1
</pre>
            </blockquote>
          </blockquote>
        </blockquote>
      </blockquote>
    </blockquote>
  </body>
</html>

--------------k1QhjioOuquynYsFLLEqbQ2N--