Sorry, didn't see it before.  I'll take a look at it in the next few days and
likely merge it or commit it early next week.

--Mark

On 4/1/21 12:33 PM, Ricardo Salveti wrote:
> Any comments/reviews?
> 
> Tried sending a similar patch via github pr but also got no response.
> 
> Thanks,
> 
> Ricardo
> 
> On Wed, Mar 3, 2021 at 12:28 PM Ricardo Salveti <[email protected]> wrote:
>>
>> Fixes a packaging error when using 'usrmerge' in DISTRO_FEATURES
>> (nonarch_base_libdir is set to /usr/lib in this case, and to /lib when
>>  usrmerge is not used).
>>
>> Signed-off-by: Ricardo Salveti <[email protected]>
>> ---
>>  classes/fpgamanager_custom.bbclass                   | 10 +++++-----
>>  classes/fpgamanager_dtg.bbclass                      | 10 +++++-----
>>  .../fpga-manager-util/fpga-manager-util_1.0.bb       | 12 ++++++------
>>  3 files changed, 16 insertions(+), 16 deletions(-)
>>
>> diff --git a/classes/fpgamanager_custom.bbclass 
>> b/classes/fpgamanager_custom.bbclass
>> index 733f403..ba1f79c 100644
>> --- a/classes/fpgamanager_custom.bbclass
>> +++ b/classes/fpgamanager_custom.bbclass
>> @@ -63,14 +63,14 @@ python devicetree_do_compile_append() {
>>  }
>>
>>  do_install() {
>> -    install -d ${D}/lib/firmware/xilinx/${PN}/
>> -    install -Dm 0644 *.dtbo ${D}/lib/firmware/xilinx/${PN}/${PN}.dtbo
>> -    install -Dm 0644 ${PN}.bit.bin 
>> ${D}/lib/firmware/xilinx/${PN}/${PN}.bit.bin
>> +    install -d ${D}${nonarch_base_libdir}/firmware/xilinx/${PN}/
>> +    install -Dm 0644 *.dtbo 
>> ${D}${nonarch_base_libdir}/firmware/xilinx/${PN}/${PN}.dtbo
>> +    install -Dm 0644 ${PN}.bit.bin 
>> ${D}${nonarch_base_libdir}/firmware/xilinx/${PN}/${PN}.bit.bin
>>      if ls ${WORKDIR}/${XCL_PATH}/*.xclbin >/dev/null 2>&1; then
>> -        install -Dm 0644 ${WORKDIR}/${XCL_PATH}/*.xclbin 
>> ${D}/lib/firmware/xilinx/${PN}/${PN}.xclbin
>> +        install -Dm 0644 ${WORKDIR}/${XCL_PATH}/*.xclbin 
>> ${D}${nonarch_base_libdir}/firmware/xilinx/${PN}/${PN}.xclbin
>>      fi
>>  }
>>
>>  do_deploy[noexec] = "1"
>>
>> -FILES_${PN} += "/lib/firmware/xilinx/{PN}"
>> +FILES_${PN} += "${nonarch_base_libdir}/firmware/xilinx/{PN}"
>> diff --git a/classes/fpgamanager_dtg.bbclass 
>> b/classes/fpgamanager_dtg.bbclass
>> index 048bde0..61f57ba 100644
>> --- a/classes/fpgamanager_dtg.bbclass
>> +++ b/classes/fpgamanager_dtg.bbclass
>> @@ -84,14 +84,14 @@ python devicetree_do_compile_append() {
>>  }
>>
>>  do_install() {
>> -    install -d ${D}/lib/firmware/xilinx/${PN}/
>> -    install -Dm 0644 pl-final.dtbo ${D}/lib/firmware/xilinx/${PN}/${PN}.dtbo
>> -    install -Dm 0644 ${PN}.bit.bin 
>> ${D}/lib/firmware/xilinx/${PN}/${PN}.bit.bin
>> +    install -d ${D}${nonarch_base_libdir}/firmware/xilinx/${PN}/
>> +    install -Dm 0644 pl-final.dtbo 
>> ${D}${nonarch_base_libdir}/firmware/xilinx/${PN}/${PN}.dtbo
>> +    install -Dm 0644 ${PN}.bit.bin 
>> ${D}${nonarch_base_libdir}/firmware/xilinx/${PN}/${PN}.bit.bin
>>      if ls ${WORKDIR}/${XCL_PATH}/*.xclbin >/dev/null 2>&1; then
>> -        install -Dm 0644 ${WORKDIR}/${XCL_PATH}/*.xclbin 
>> ${D}/lib/firmware/xilinx/${PN}/${PN}.xclbin
>> +        install -Dm 0644 ${WORKDIR}/${XCL_PATH}/*.xclbin 
>> ${D}${nonarch_base_libdir}/firmware/xilinx/${PN}/${PN}.xclbin
>>      fi
>>  }
>>
>>  do_deploy[noexec] = "1"
>>
>> -FILES_${PN} += "/lib/firmware/xilinx/${PN}"
>> +FILES_${PN} += "${nonarch_base_libdir}/firmware/xilinx/${PN}"
>> diff --git a/recipes-bsp/fpga-manager-util/fpga-manager-util_1.0.bb 
>> b/recipes-bsp/fpga-manager-util/fpga-manager-util_1.0.bb
>> index 53a041c..6c2a062 100644
>> --- a/recipes-bsp/fpga-manager-util/fpga-manager-util_1.0.bb
>> +++ b/recipes-bsp/fpga-manager-util/fpga-manager-util_1.0.bb
>> @@ -130,23 +130,23 @@ do_compile() {
>>  }
>>
>>  do_install() {
>> -        install -d ${D}/lib/firmware/xilinx/base
>> +        install -d ${D}${nonarch_base_libdir}/firmware/xilinx/base
>>          if [ -e "base.dtbo" ]; then
>>              #install base hdf artifacts
>> -            install -Dm 0644 base.dtbo 
>> ${D}/lib/firmware/xilinx/base/base.dtbo
>> +            install -Dm 0644 base.dtbo 
>> ${D}${nonarch_base_libdir}/firmware/xilinx/base/base.dtbo
>>              if [ "${SOC_FAMILY}" != "versal" ]; then
>>                  newname=`basename *.bit.bin_base | awk -F '.bit.bin_' 
>> '{print $1}'`
>> -                install -Dm 0644 *.bit.bin_base 
>> ${D}/lib/firmware/xilinx/base/${newname}.bit.bin
>> +                install -Dm 0644 *.bit.bin_base 
>> ${D}${nonarch_base_libdir}/firmware/xilinx/base/${newname}.bit.bin
>>              else
>>                  #partial pdi in xsa is not yet supported, will need to 
>> modify this part once supported
>>                  echo "TODO"
>>              fi
>>          fi
>>          for hdf in ${HDF_LIST}; do
>> -                install -Dm 0644 ${hdf}.dtbo 
>> ${D}/lib/firmware/xilinx/${hdf}/${hdf}.dtbo
>> +                install -Dm 0644 ${hdf}.dtbo 
>> ${D}${nonarch_base_libdir}/firmware/xilinx/${hdf}/${hdf}.dtbo
>>                  if [ "${SOC_FAMILY}" != "versal" ]; then
>>                      newname=`basename *.bit.bin_${hdf} | awk -F '.bit.bin_' 
>> '{print $1}'`
>> -                    install -Dm 0644 *.bit.bin_${hdf} 
>> ${D}/lib/firmware/xilinx/${hdf}/${newname}.bit.bin
>> +                    install -Dm 0644 *.bit.bin_${hdf} 
>> ${D}${nonarch_base_libdir}/firmware/xilinx/${hdf}/${newname}.bit.bin
>>                  else
>>                      #partial pdi in xsa is not yet supported, will need to 
>> modify this part once supported
>>                      echo "TODO"
>> @@ -160,7 +160,7 @@ python () {
>>          if d.getVar('FPGA_MNGR_RECONFIG_ENABLE') == '1':
>>                  extra = d.getVar('EXTRA_HDF')
>>                  pn = d.getVar('PN')
>> -                baselib = d.getVar('base_libdir')
>> +                baselib = d.getVar('nonarch_base_libdir')
>>                  packages = d.getVar('PACKAGES').split()
>>
>>                  #package base hdf
>> --
>> 2.30.1
>>
> 
> 
> 
> 
> 
> 
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#4877): 
https://lists.yoctoproject.org/g/meta-xilinx/message/4877
Mute This Topic: https://lists.yoctoproject.org/mt/81053937/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/meta-xilinx/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to