[meta-xilinx] [meta-xilinx-bsp][PATCH 1/2] recipes-multimedia/vcu: Removed .exe extension from ctrlsw/omx applications

Manjukumar Matha manjukumar.harthikote-matha at xilinx.com
Wed Dec 19 17:10:01 PST 2018


From: Dhaval Shah <dhaval.shah at xilinx.com>

Updated name as per below list in ctrlsw/omx component's:
AL_Encoder.exe -> ctrlsw_encoder
AL_Decoder.exe -> ctrlsw_decoder
omx_encoder.exe -> omx_encoder
omx_decoder.exe -> omx_decoder

-It is unconventional and unnecessary for Petalinux executables to
have a .exe extension, as .exe is standard convention for windows
applications, but not for Linux applications.

-This has created confusion among users of AL_Encoder.exe/AL_Decoder.exe
and omx_encoder.exe/omx_decoder.exe. The only reason to have .exe would
be for cross-platform compatibility with MicroSoft Windows, but Xilinx
will never support these sample applications on Windows, so it is
recommend to change the names to customary Linux names.

Signed-off-by: Dhaval Shah <dhaval.shah at xilinx.com>
Acked-by: Varunkumar Allagadapa <varunkumar.allagadapa at xilinx.com>
Signed-off-by: Manjukumar Matha <manjukumar.harthikote-matha at xilinx.com>
---
 meta-xilinx-bsp/recipes-multimedia/vcu/libomxil-xlnx.bb | 4 ++--
 meta-xilinx-bsp/recipes-multimedia/vcu/libvcu-xlnx.bb   | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/meta-xilinx-bsp/recipes-multimedia/vcu/libomxil-xlnx.bb b/meta-xilinx-bsp/recipes-multimedia/vcu/libomxil-xlnx.bb
index 76fdb9a..f17da4e 100644
--- a/meta-xilinx-bsp/recipes-multimedia/vcu/libomxil-xlnx.bb
+++ b/meta-xilinx-bsp/recipes-multimedia/vcu/libomxil-xlnx.bb
@@ -37,8 +37,8 @@ do_install() {
 
     install -m 0644 ${S}/omx_header/*.h ${D}${includedir}/vcu-omx-il
 
-    install -Dm 0755 ${S}/bin/omx_decoder.exe ${D}/${bindir}/omx_decoder.exe
-    install -Dm 0755 ${S}/bin/omx_encoder.exe ${D}/${bindir}/omx_encoder.exe
+    install -Dm 0755 ${S}/bin/omx_decoder.exe ${D}/${bindir}/omx_decoder
+    install -Dm 0755 ${S}/bin/omx_encoder.exe ${D}/${bindir}/omx_encoder
 
     oe_libinstall -C ${S}/bin/ -so libOMX.allegro.core ${D}/${libdir}/
     oe_libinstall -C ${S}/bin/ -so libOMX.allegro.video_decoder ${D}/${libdir}/
diff --git a/meta-xilinx-bsp/recipes-multimedia/vcu/libvcu-xlnx.bb b/meta-xilinx-bsp/recipes-multimedia/vcu/libvcu-xlnx.bb
index 8e2369b..c56bb92 100644
--- a/meta-xilinx-bsp/recipes-multimedia/vcu/libvcu-xlnx.bb
+++ b/meta-xilinx-bsp/recipes-multimedia/vcu/libvcu-xlnx.bb
@@ -29,8 +29,8 @@ do_install() {
     install -d ${D}${libdir}
     install -d ${D}${includedir}/vcu-ctrl-sw/include
 
-    install -Dm 0755 ${S}/bin/AL_Encoder.exe ${D}/${bindir}/AL_Encoder.exe
-    install -Dm 0755 ${S}/bin/AL_Decoder.exe ${D}/${bindir}/AL_Decoder.exe
+    install -Dm 0755 ${S}/bin/AL_Encoder.exe ${D}/${bindir}/ctrlsw_encoder
+    install -Dm 0755 ${S}/bin/AL_Decoder.exe ${D}/${bindir}/ctrlsw_decoder
 
     oe_runmake install_headers INSTALL_HDR_PATH=${D}${includedir}/vcu-ctrl-sw/include
     oe_libinstall -C ${S}/bin/ -so liballegro_decode ${D}/${libdir}/
-- 
2.7.4



More information about the meta-xilinx mailing list