All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/4] Resubmit TI DSP build issues related to installer
@ 2012-01-31 12:40 Peter A. Bigot
  2012-01-31 12:40 ` [PATCH 1/4] ti-eula-unpack: remove extraneous newline in cmd output Peter A. Bigot
                   ` (4 more replies)
  0 siblings, 5 replies; 14+ messages in thread
From: Peter A. Bigot @ 2012-01-31 12:40 UTC (permalink / raw)
  To: meta-ti

From: "Peter A. Bigot" <bigotp@acm.org>

This patch set re-submits the (updated) uncommitted parts of my 27Dec2011
patch set to get gstreamer-ti and its dependencies to build in a pristine
work environment.  It was left back on 29Dec2011 with:

> I'm going to wait for Roger to get back from holidays to comment on the installer issues, I'll push the CE and dmai patches later today.

The installer issues are:

ti-eula-unpack spawns the installer, then writes canned input to it to do
things like set the output directory.  The function that does this
explicitly tacks a second newline on, in addition to the one normally added
by Python's print statement.

For ti-cgt6x, the result is that a bare newline is fed to the installer at
the point where the installation directory is requested, instead of the
workdir.  Hence, the default directory gets used; for this package, that's
in /opt/TI.

For ti-xdctools, ti-edma3lld, and ti-dspbios, the bare newline also caused
the default installation directory to be requested, but for this package
that's ${HOME}/ti.  Since ti-eula-unpack overrides the HOME environment
variable to be the work directory, everything somehow happened to work out,
at least for other folks.  I don't think that's a great solution though, so
I updated the recipes to do it "right", by changing the commands to conform
to what the current installers expect.  There may be other installation
recipes for other tools that need the same fixes.

For the 31Jan2012 attempt, similar changes had to be made to
ti-codecs-omap3530.

For the 31Jan2012 attempt, the ti-dmai build required more than just
altering the striplevel: the ${S} value had to be changed since there was no
Makefile at its original value.  There is still an issue that prevents use
of where make 3.85 produces a "Entering an unknown directory" failure in
do_install, but that's an upstream problem: I just used make 3.81 to
complete the build.

Peter A. Bigot (4):
  ti-eula-unpack: remove extraneous newline in cmd output
  various: fix installation directories for corrected ti-eula-unpack
  ti-codecs-omap3530: fix for corrected ti-eula-unpack
  ti-dmai_svn: correct path within svn checkout

 recipes-ti/bios/ti-dspbios.inc                     |    3 ++-
 .../codec-engine/ti-codecs-omap3530_4.00.00.00.bb  |   20 ++++++++++----------
 recipes-ti/devtools/ti-xdctools.inc                |    3 ++-
 recipes-ti/dmai/ti-dmai_svn.bb                     |    9 ++++++---
 recipes-ti/edma3lld/ti-edma3lld.inc                |    3 ++-
 recipes-ti/includes/ti-eula-unpack.inc             |    2 +-
 6 files changed, 23 insertions(+), 17 deletions(-)

-- 
1.7.6.4



^ permalink raw reply	[flat|nested] 14+ messages in thread

* [PATCH 1/4] ti-eula-unpack: remove extraneous newline in cmd output
  2012-01-31 12:40 [PATCH 0/4] Resubmit TI DSP build issues related to installer Peter A. Bigot
@ 2012-01-31 12:40 ` Peter A. Bigot
  2012-02-15 15:11   ` Monk, Roger
  2012-01-31 12:40 ` [PATCH 2/4] various: fix installation directories for corrected ti-eula-unpack Peter A. Bigot
                   ` (3 subsequent siblings)
  4 siblings, 1 reply; 14+ messages in thread
From: Peter A. Bigot @ 2012-01-31 12:40 UTC (permalink / raw)
  To: meta-ti

From: "Peter A. Bigot" <bigotp@acm.org>

print adds one already; to add another causes the input to be off and the
default installation directory to be used, at least on Fedora 15.

Signed-off-by: Peter A. Bigot <bigotp@acm.org>
---
 recipes-ti/includes/ti-eula-unpack.inc |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/recipes-ti/includes/ti-eula-unpack.inc b/recipes-ti/includes/ti-eula-unpack.inc
index 3b79c24..7c6e7b7 100644
--- a/recipes-ti/includes/ti-eula-unpack.inc
+++ b/recipes-ti/includes/ti-eula-unpack.inc
@@ -53,7 +53,7 @@ python ti_bin_do_unpack() {
             wdext = bb.data.getVar('TI_BIN_UNPK_WDEXT', localdata)
             wdext = bb.data.expand(wdext, localdata)
             cmd = workdir+wdext
-        print >>f, "%s\n" % cmd
+        print >>f, cmd
     f.close()
 
     # Expand the tarball that was created if required
-- 
1.7.6.4



^ permalink raw reply related	[flat|nested] 14+ messages in thread

* [PATCH 2/4] various: fix installation directories for corrected ti-eula-unpack
  2012-01-31 12:40 [PATCH 0/4] Resubmit TI DSP build issues related to installer Peter A. Bigot
  2012-01-31 12:40 ` [PATCH 1/4] ti-eula-unpack: remove extraneous newline in cmd output Peter A. Bigot
@ 2012-01-31 12:40 ` Peter A. Bigot
  2012-02-15 15:11   ` Monk, Roger
  2012-01-31 12:40 ` [PATCH 3/4] ti-codecs-omap3530: fix " Peter A. Bigot
                   ` (2 subsequent siblings)
  4 siblings, 1 reply; 14+ messages in thread
From: Peter A. Bigot @ 2012-01-31 12:40 UTC (permalink / raw)
  To: meta-ti

From: "Peter A. Bigot" <bigotp@acm.org>

Signed-off-by: Peter A. Bigot <bigotp@acm.org>
---
 recipes-ti/bios/ti-dspbios.inc      |    3 ++-
 recipes-ti/devtools/ti-xdctools.inc |    3 ++-
 recipes-ti/edma3lld/ti-edma3lld.inc |    3 ++-
 3 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/recipes-ti/bios/ti-dspbios.inc b/recipes-ti/bios/ti-dspbios.inc
index 8519039..0e3fa62 100644
--- a/recipes-ti/bios/ti-dspbios.inc
+++ b/recipes-ti/bios/ti-dspbios.inc
@@ -10,7 +10,8 @@ require ../includes/ti-paths.inc
 require ../includes/ti-staging.inc
 require ../includes/ti-eula-unpack.inc
 
-S = "${WORKDIR}/bios_${PV}"
+TI_BIN_UNPK_WDEXT="/bios_${PV}"
+S = "${WORKDIR}${TI_BIN_UNPK_WDEXT}"
 
 SRC_URI = "http://software-dl.ti.com/dsps/dsps_public_sw/sdo_sb/targetcontent/bios/dspbios/${PV}/exports/bios_setuplinux_${PV}.bin;name=biosbin"
 
diff --git a/recipes-ti/devtools/ti-xdctools.inc b/recipes-ti/devtools/ti-xdctools.inc
index c76ad80..547a555 100644
--- a/recipes-ti/devtools/ti-xdctools.inc
+++ b/recipes-ti/devtools/ti-xdctools.inc
@@ -12,7 +12,8 @@ S = "${WORKDIR}/xdctools_${PV}"
 SRC_URI = "http://software-dl.ti.com/dsps/dsps_public_sw/sdo_sb/targetcontent/rtsc/${PV}/exports/xdctools_setuplinux_${PV}.bin;name=xdcbin"
 
 BINFILE="xdctools_setuplinux_${PV}.bin"
-TI_BIN_UNPK_CMDS="Y: qY:workdir:Y"
+TI_BIN_UNPK_CMDS="Y:workdir:Y"
+TI_BIN_UNPK_WDEXT="/ti"
 
 do_install() {
     install -d ${D}${XDC_INSTALL_DIR_RECIPE}
diff --git a/recipes-ti/edma3lld/ti-edma3lld.inc b/recipes-ti/edma3lld/ti-edma3lld.inc
index d4e8557..fb131b6 100644
--- a/recipes-ti/edma3lld/ti-edma3lld.inc
+++ b/recipes-ti/edma3lld/ti-edma3lld.inc
@@ -8,7 +8,8 @@ require ../includes/ti-paths.inc
 require ../includes/ti-staging.inc
 require ../includes/ti-eula-unpack.inc
 
-S = "${WORKDIR}/edma3_lld_${PV}"
+TI_BIN_UNPK_WDEXT="/edma3_lld_${PV}"
+S = "${WORKDIR}${TI_BIN_UNPK_WDEXT}"
 
 SRC_URI = "http://software-dl.ti.com/dsps/dsps_public_sw/${SRC_URI_edma3lldsite}/edma3_lld/edma3-lld-bios5/${PV}/exports/EDMA3_LLD_setuplinux_${PV}.bin;name=edma3lldbin"
 
-- 
1.7.6.4



^ permalink raw reply related	[flat|nested] 14+ messages in thread

* [PATCH 3/4] ti-codecs-omap3530: fix for corrected ti-eula-unpack
  2012-01-31 12:40 [PATCH 0/4] Resubmit TI DSP build issues related to installer Peter A. Bigot
  2012-01-31 12:40 ` [PATCH 1/4] ti-eula-unpack: remove extraneous newline in cmd output Peter A. Bigot
  2012-01-31 12:40 ` [PATCH 2/4] various: fix installation directories for corrected ti-eula-unpack Peter A. Bigot
@ 2012-01-31 12:40 ` Peter A. Bigot
  2012-02-15 15:11   ` Monk, Roger
  2012-01-31 12:40 ` [PATCH 4/4] ti-dmai_svn: correct path within svn checkout Peter A. Bigot
  2012-02-01  0:45 ` [PATCH 0/4] Resubmit TI DSP build issues related to installer Monk, Roger
  4 siblings, 1 reply; 14+ messages in thread
From: Peter A. Bigot @ 2012-01-31 12:40 UTC (permalink / raw)
  To: meta-ti

From: "Peter A. Bigot" <bigotp@acm.org>

Remove directory prefix for tar files that use TI_BIN_UNPK_CMDS that
specifies unpacking into workdir; formerly that specification was ignored
and these were unpacked into a package-specific subdirectory of
${HOME}=${WORKDIR}.  Note c64xplus_deinterlacer overrides TI_BIN_UNPK_CMDS
and does not specify workdir, so its prefix is correct.

Signed-off-by: Peter A. Bigot <bigotp@acm.org>
---
 .../codec-engine/ti-codecs-omap3530_4.00.00.00.bb  |   20 ++++++++++----------
 1 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/recipes-ti/codec-engine/ti-codecs-omap3530_4.00.00.00.bb b/recipes-ti/codec-engine/ti-codecs-omap3530_4.00.00.00.bb
index ae218b1..ea9f389 100644
--- a/recipes-ti/codec-engine/ti-codecs-omap3530_4.00.00.00.bb
+++ b/recipes-ti/codec-engine/ti-codecs-omap3530_4.00.00.00.bb
@@ -77,43 +77,43 @@ python do_unpack () {
     bb.build.exec_func('base_do_unpack', d)
 
     bb.data.setVar("BINFILE", "omap3530_h264enc_2_01_013_production.bin", d)
-    bb.data.setVar("TARFILE", "h264enc_dm6467_1_20_00_08/omap3530_h264enc_2_01_013_production.tar", d)
+    bb.data.setVar("TARFILE", "omap3530_h264enc_2_01_013_production.tar", d)
     bb.build.exec_func('ti_bin_do_unpack', d)
 
     bb.data.setVar("BINFILE", "omap3530_h264dec_2_01_007_production.bin", d)
-    bb.data.setVar("TARFILE", "omap3530_h264dec_2_01_007_production/omap3530_h264dec_2_01_007_production.tar", d)
+    bb.data.setVar("TARFILE", "omap3530_h264dec_2_01_007_production.tar", d)
     bb.build.exec_func('ti_bin_do_unpack', d)
 
     bb.data.setVar("BINFILE", "omap3530_jpegenc_02_01_01_00_production.bin", d)
-    bb.data.setVar("TARFILE", "omap3530_jpegenc_02_01_01_00_production/omap3530_jpegenc_02_01_01_00_production.tar", d)
+    bb.data.setVar("TARFILE", "omap3530_jpegenc_02_01_01_00_production.tar", d)
     bb.build.exec_func('ti_bin_do_unpack', d)
 
     bb.data.setVar("BINFILE", "c64xplus_jpegdec_02_00_01_01_production.bin", d)
-    bb.data.setVar("TARFILE", "c64xplus_jpegdec_02_00_01_01_production/c64xplus_jpegdec_02_00_01_01_production.tar", d)
+    bb.data.setVar("TARFILE", "c64xplus_jpegdec_02_00_01_01_production.tar", d)
     bb.build.exec_func('ti_bin_do_unpack', d)
 
     bb.data.setVar("BINFILE", "c64xplus_mpeg2dec_02_00_02_00_production.bin", d)
-    bb.data.setVar("TARFILE", "c64xplus_mpeg2dec_02_00_02_00_production/c64xplus_mpeg2dec_02_00_02_00_production.tar", d)
+    bb.data.setVar("TARFILE", "c64xplus_mpeg2dec_02_00_02_00_production.tar", d)
     bb.build.exec_func('ti_bin_do_unpack', d)
 
     bb.data.setVar("BINFILE", "omap3530_mpeg4enc_02_04_00_00_production.bin", d)
-    bb.data.setVar("TARFILE", "omap3530_mpeg4enc_02_04_00_00_production/omap3530_mpeg4enc_02_04_00_00_production.tar", d)
+    bb.data.setVar("TARFILE", "omap3530_mpeg4enc_02_04_00_00_production.tar", d)
     bb.build.exec_func('ti_bin_do_unpack', d)   
 
     bb.data.setVar("BINFILE", "c64xplus_mpeg4dec_02_01_00_00_production.bin", d)
-    bb.data.setVar("TARFILE", "h264enc_dm6467_1_20_00_08/c64xplus_mpeg4dec_02_01_00_00_production.tar", d)
+    bb.data.setVar("TARFILE", "c64xplus_mpeg4dec_02_01_00_00_production.tar", d)
     bb.build.exec_func('ti_bin_do_unpack', d)
 
     bb.data.setVar("BINFILE", "c64xplus_aachedec_01_30_03_00_production.bin", d)
-    bb.data.setVar("TARFILE", "c64xplus_aachedec_01_30_03_00_production/dm6446_aachedec_01_30_03_00_production.tar", d)
+    bb.data.setVar("TARFILE", "dm6446_aachedec_01_30_03_00_production.tar", d)
     bb.build.exec_func('ti_bin_do_unpack', d)
 
     bb.data.setVar("BINFILE", "c64xplus_g711_1_12_00_000_production.bin", d)
-    bb.data.setVar("TARFILE", "h264enc_dm6467_1_20_00_08/dm6446_g711enc_1_12_00_000_production.tar", d)
+    bb.data.setVar("TARFILE", "dm6446_g711enc_1_12_00_000_production.tar", d)
     bb.build.exec_func('ti_bin_do_unpack', d)
 
     bb.data.setVar("BINFILE", "c64xplus_g711_1_12_00_000_production.bin", d)
-    bb.data.setVar("TARFILE", "h264enc_dm6467_1_20_00_08/dm6446_g711dec_1_12_00_000_production.tar", d)
+    bb.data.setVar("TARFILE", "dm6446_g711dec_1_12_00_000_production.tar", d)
     bb.build.exec_func('ti_bin_do_unpack', d)
 
     bb.data.setVar("BINFILE", "c64xplus_deinterlacer_01_00_00_07_production.bin", d)
-- 
1.7.6.4



^ permalink raw reply related	[flat|nested] 14+ messages in thread

* [PATCH 4/4] ti-dmai_svn: correct path within svn checkout
  2012-01-31 12:40 [PATCH 0/4] Resubmit TI DSP build issues related to installer Peter A. Bigot
                   ` (2 preceding siblings ...)
  2012-01-31 12:40 ` [PATCH 3/4] ti-codecs-omap3530: fix " Peter A. Bigot
@ 2012-01-31 12:40 ` Peter A. Bigot
  2012-02-15 15:11   ` Monk, Roger
  2012-02-01  0:45 ` [PATCH 0/4] Resubmit TI DSP build issues related to installer Monk, Roger
  4 siblings, 1 reply; 14+ messages in thread
From: Peter A. Bigot @ 2012-01-31 12:40 UTC (permalink / raw)
  To: meta-ti

From: "Peter A. Bigot" <bigotp@acm.org>

At r642 as currently retrieved, the contents of ${S} are subdirectories dmai
and patches, so the subsequent attempt to run "make clean" in ${S} fails
because there is no Makefile there.  Append /dmai to the default ${S},
remove the now incorrect /dmai from LIC_FILES_CHKSUM, and correct the
strippath for the patches to reflect the new starting location.

Signed-off-by: Peter A. Bigot <bigotp@acm.org>
---
 recipes-ti/dmai/ti-dmai_svn.bb |    9 ++++++---
 1 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/recipes-ti/dmai/ti-dmai_svn.bb b/recipes-ti/dmai/ti-dmai_svn.bb
index 3c5595d..3ac7a81 100644
--- a/recipes-ti/dmai/ti-dmai_svn.bb
+++ b/recipes-ti/dmai/ti-dmai_svn.bb
@@ -1,11 +1,14 @@
 require ti-dmai.inc
 
+# Need to descend one level more to find source
+S = "${WORKDIR}/${DMAIBRANCH}/davinci_multimedia_application_interface/dmai_${PV}/dmai"
+
 # Hack to be able to use recent kernel headers from userspace
 TARGET_CC_ARCH += " -D__EXPORTED_HEADERS__"
 
 PV = "2_10_00_01+svnr${SRCPV}"
 
-LIC_FILES_CHKSUM = "file://dmai/dmai_${PV}_License.html;md5=3302f728a5a42f97cabc26a54d7fa607"
+LIC_FILES_CHKSUM = "file://dmai_${PV}_License.html;md5=3302f728a5a42f97cabc26a54d7fa607"
 
 # This package has high dependence on kernel, use kernel PR as base and append a local version
 PR = "${MACHINE_KERNEL_PR}"
@@ -29,7 +32,7 @@ SRCREV_omapl137       = "482"
 SRCREV_omapl138       = "570"
 SRCREV               ?= "UNDEFINED_SRCREV"
 
-SRC_URI_append = " file://omap3530-r642-remove-include-videodev.diff;striplevel=2 \
-                   file://r642-fix-config-bld.diff;striplevel=2 \
+SRC_URI_append = " file://omap3530-r642-remove-include-videodev.diff;striplevel=3 \
+                   file://r642-fix-config-bld.diff;striplevel=3 \
                  "
 
-- 
1.7.6.4



^ permalink raw reply related	[flat|nested] 14+ messages in thread

* Re: [PATCH 0/4] Resubmit TI DSP build issues related to installer
  2012-01-31 12:40 [PATCH 0/4] Resubmit TI DSP build issues related to installer Peter A. Bigot
                   ` (3 preceding siblings ...)
  2012-01-31 12:40 ` [PATCH 4/4] ti-dmai_svn: correct path within svn checkout Peter A. Bigot
@ 2012-02-01  0:45 ` Monk, Roger
  2012-02-01 20:41   ` Peter Bigot
  2012-02-11  4:03   ` Denys Dmytriyenko
  4 siblings, 2 replies; 14+ messages in thread
From: Monk, Roger @ 2012-02-01  0:45 UTC (permalink / raw)
  To: Peter A. Bigot, meta-ti@yoctoproject.org

Hi Peter,

On Tue, Jan 31, 2012 at 12:40:53, Peter A. Bigot wrote:
> Subject: [meta-ti] [PATCH 0/4] Resubmit TI DSP build issues related to
> installer
>
> From: "Peter A. Bigot" <bigotp@acm.org>
>
> This patch set re-submits the (updated) uncommitted parts of my
> 27Dec2011 patch set to get gstreamer-ti and its dependencies to build
> in a pristine work environment.  It was left back on 29Dec2011 with:
>
> > I'm going to wait for Roger to get back from holidays to comment on
> the installer issues, I'll push the CE and dmai patches later today.
>

Thanks for re-basing/re-submitting - sorry this has taken a while.  Your patches look good - however, I have some other failing installer scenarios like these for which I have a similar, but unfortunately, not the same patchset, in progress.  I hope to review yours in the coming days if ok.

> The installer issues are:
>
> ti-eula-unpack spawns the installer, then writes canned input to it to
> do things like set the output directory.  The function that does this
> explicitly tacks a second newline on, in addition to the one normally
> added by Python's print statement.
>
> For ti-cgt6x, the result is that a bare newline is fed to the
> installer at the point where the installation directory is requested,
> instead of the workdir.  Hence, the default directory gets used; for
> this package, that's in /opt/TI.
>
> For ti-xdctools, ti-edma3lld, and ti-dspbios, the bare newline also
> caused the default installation directory to be requested, but for
> this package that's ${HOME}/ti.  Since ti-eula-unpack overrides the
> HOME environment variable to be the work directory, everything somehow
> happened to work out, at least for other folks.  I don't think that's
> a great solution though, so I updated the recipes to do it "right", by
> changing the commands to conform to what the current installers expect.
> There may be other installation recipes for other tools that need the
> same fixes.
>
> For the 31Jan2012 attempt, similar changes had to be made to
> ti-codecs- omap3530.
>
> For the 31Jan2012 attempt, the ti-dmai build required more than just
> altering the striplevel: the ${S} value had to be changed since there
> was no Makefile at its original value.  There is still an issue that
> prevents use of where make 3.85 produces a "Entering an unknown
> directory" failure in do_install, but that's an upstream problem: I
> just used make 3.81 to complete the build.
>
> Peter A. Bigot (4):
>   ti-eula-unpack: remove extraneous newline in cmd output
>   various: fix installation directories for corrected ti-eula-unpack
>   ti-codecs-omap3530: fix for corrected ti-eula-unpack
>   ti-dmai_svn: correct path within svn checkout
>
>  recipes-ti/bios/ti-dspbios.inc                     |    3 ++-
>  .../codec-engine/ti-codecs-omap3530_4.00.00.00.bb  |   20 ++++++++++--
> --------
>  recipes-ti/devtools/ti-xdctools.inc                |    3 ++-
>  recipes-ti/dmai/ti-dmai_svn.bb                     |    9 ++++++---
>  recipes-ti/edma3lld/ti-edma3lld.inc                |    3 ++-
>  recipes-ti/includes/ti-eula-unpack.inc             |    2 +-
>  6 files changed, 23 insertions(+), 17 deletions(-)
>
> --
> 1.7.6.4
>
>
Texas Instruments Limited, 800 Pavilion Drive, Northampton, NN4 7YL. Registered in England & Wales under company number 00574102

_______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti





^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 0/4] Resubmit TI DSP build issues related to installer
  2012-02-01  0:45 ` [PATCH 0/4] Resubmit TI DSP build issues related to installer Monk, Roger
@ 2012-02-01 20:41   ` Peter Bigot
  2012-02-15 15:11     ` Monk, Roger
  2012-02-11  4:03   ` Denys Dmytriyenko
  1 sibling, 1 reply; 14+ messages in thread
From: Peter Bigot @ 2012-02-01 20:41 UTC (permalink / raw)
  To: Monk, Roger; +Cc: meta-ti@yoctoproject.org

On Tue, Jan 31, 2012 at 6:45 PM, Monk, Roger <r-monk@ti.com> wrote:
> Hi Peter,
>
> On Tue, Jan 31, 2012 at 12:40:53, Peter A. Bigot wrote:
>> Subject: [meta-ti] [PATCH 0/4] Resubmit TI DSP build issues related to
>> installer
>>
>> From: "Peter A. Bigot" <bigotp@acm.org>
>>
>> This patch set re-submits the (updated) uncommitted parts of my
>> 27Dec2011 patch set to get gstreamer-ti and its dependencies to build
>> in a pristine work environment.  It was left back on 29Dec2011 with:
>>
>> > I'm going to wait for Roger to get back from holidays to comment on
>> the installer issues, I'll push the CE and dmai patches later today.
>>
>
> Thanks for re-basing/re-submitting - sorry this has taken a while.  Your patches look good - however, I have some other failing installer scenarios like these for which I have a similar, but unfortunately, not the same patchset, in progress.  I hope to review yours in the coming days if ok.

That's fine.  Rather than bug everybody with update emails while the
two approaches are reconciled, these fixes are available at
https://github.com/pabigot/meta-ti/tree/ti-eula-unpack.  There you'll
find the corresponding change as required for omap3-sgx-modules.

Peter

>> The installer issues are:
>>
>> ti-eula-unpack spawns the installer, then writes canned input to it to
>> do things like set the output directory.  The function that does this
>> explicitly tacks a second newline on, in addition to the one normally
>> added by Python's print statement.
>>
>> For ti-cgt6x, the result is that a bare newline is fed to the
>> installer at the point where the installation directory is requested,
>> instead of the workdir.  Hence, the default directory gets used; for
>> this package, that's in /opt/TI.
>>
>> For ti-xdctools, ti-edma3lld, and ti-dspbios, the bare newline also
>> caused the default installation directory to be requested, but for
>> this package that's ${HOME}/ti.  Since ti-eula-unpack overrides the
>> HOME environment variable to be the work directory, everything somehow
>> happened to work out, at least for other folks.  I don't think that's
>> a great solution though, so I updated the recipes to do it "right", by
>> changing the commands to conform to what the current installers expect.
>> There may be other installation recipes for other tools that need the
>> same fixes.
>>
>> For the 31Jan2012 attempt, similar changes had to be made to
>> ti-codecs- omap3530.
>>
>> For the 31Jan2012 attempt, the ti-dmai build required more than just
>> altering the striplevel: the ${S} value had to be changed since there
>> was no Makefile at its original value.  There is still an issue that
>> prevents use of where make 3.85 produces a "Entering an unknown
>> directory" failure in do_install, but that's an upstream problem: I
>> just used make 3.81 to complete the build.
>>
>> Peter A. Bigot (4):
>>   ti-eula-unpack: remove extraneous newline in cmd output
>>   various: fix installation directories for corrected ti-eula-unpack
>>   ti-codecs-omap3530: fix for corrected ti-eula-unpack
>>   ti-dmai_svn: correct path within svn checkout
>>
>>  recipes-ti/bios/ti-dspbios.inc                     |    3 ++-
>>  .../codec-engine/ti-codecs-omap3530_4.00.00.00.bb  |   20 ++++++++++--
>> --------
>>  recipes-ti/devtools/ti-xdctools.inc                |    3 ++-
>>  recipes-ti/dmai/ti-dmai_svn.bb                     |    9 ++++++---
>>  recipes-ti/edma3lld/ti-edma3lld.inc                |    3 ++-
>>  recipes-ti/includes/ti-eula-unpack.inc             |    2 +-
>>  6 files changed, 23 insertions(+), 17 deletions(-)
>>
>> --
>> 1.7.6.4
>>
>>
> Texas Instruments Limited, 800 Pavilion Drive, Northampton, NN4 7YL. Registered in England & Wales under company number 00574102
>
> _______________________________________________
>> meta-ti mailing list
>> meta-ti@yoctoproject.org
>> https://lists.yoctoproject.org/listinfo/meta-ti
>
>
>


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 0/4] Resubmit TI DSP build issues related to installer
  2012-02-01  0:45 ` [PATCH 0/4] Resubmit TI DSP build issues related to installer Monk, Roger
  2012-02-01 20:41   ` Peter Bigot
@ 2012-02-11  4:03   ` Denys Dmytriyenko
  1 sibling, 0 replies; 14+ messages in thread
From: Denys Dmytriyenko @ 2012-02-11  4:03 UTC (permalink / raw)
  To: Monk, Roger; +Cc: meta-ti@yoctoproject.org

On Wed, Feb 01, 2012 at 12:45:03AM +0000, Monk, Roger wrote:
> Hi Peter,
> 
> On Tue, Jan 31, 2012 at 12:40:53, Peter A. Bigot wrote:
> > Subject: [meta-ti] [PATCH 0/4] Resubmit TI DSP build issues related to
> > installer
> >
> > From: "Peter A. Bigot" <bigotp@acm.org>
> >
> > This patch set re-submits the (updated) uncommitted parts of my
> > 27Dec2011 patch set to get gstreamer-ti and its dependencies to build
> > in a pristine work environment.  It was left back on 29Dec2011 with:
> >
> > > I'm going to wait for Roger to get back from holidays to comment on
> > the installer issues, I'll push the CE and dmai patches later today.
> >
> 
> Thanks for re-basing/re-submitting - sorry this has taken a while.  Your 
> patches look good - however, I have some other failing installer scenarios 
> like these for which I have a similar, but unfortunately, not the same 
> patchset, in progress.  I hope to review yours in the coming days if ok.

Roger,

Were you able to look into this issue and review Peter's patches and/or merge 
with your work? Please let me know if you are making progress and new more 
time. Thanks for your time and support!

-- 
Denys


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/4] ti-eula-unpack: remove extraneous newline in cmd output
  2012-01-31 12:40 ` [PATCH 1/4] ti-eula-unpack: remove extraneous newline in cmd output Peter A. Bigot
@ 2012-02-15 15:11   ` Monk, Roger
  0 siblings, 0 replies; 14+ messages in thread
From: Monk, Roger @ 2012-02-15 15:11 UTC (permalink / raw)
  To: Peter A. Bigot, meta-ti@yoctoproject.org

On Tue, Jan 31, 2012 at 12:40:54, Peter A. Bigot wrote:
> Subject: [meta-ti] [PATCH 1/4] ti-eula-unpack: remove extraneous
> newline in cmd output
>
> From: "Peter A. Bigot" <bigotp@acm.org>
>
> print adds one already; to add another causes the input to be off and
> the default installation directory to be used, at least on Fedora 15.
>
> Signed-off-by: Peter A. Bigot <bigotp@acm.org>

Acked-by: Roger Monk <r-monk@ti.com>

> ---
>  recipes-ti/includes/ti-eula-unpack.inc |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/recipes-ti/includes/ti-eula-unpack.inc b/recipes-
> ti/includes/ti-eula-unpack.inc index 3b79c24..7c6e7b7 100644
> --- a/recipes-ti/includes/ti-eula-unpack.inc
> +++ b/recipes-ti/includes/ti-eula-unpack.inc
> @@ -53,7 +53,7 @@ python ti_bin_do_unpack() {
>              wdext = bb.data.getVar('TI_BIN_UNPK_WDEXT', localdata)
>              wdext = bb.data.expand(wdext, localdata)
>              cmd = workdir+wdext
> -        print >>f, "%s\n" % cmd
> +        print >>f, cmd
>      f.close()
>
>      # Expand the tarball that was created if required
> --
> 1.7.6.4
>
>
Texas Instruments Limited, 800 Pavilion Drive, Northampton, NN4 7YL. Registered in England & Wales under company number 00574102

_______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti





^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 4/4] ti-dmai_svn: correct path within svn checkout
  2012-01-31 12:40 ` [PATCH 4/4] ti-dmai_svn: correct path within svn checkout Peter A. Bigot
@ 2012-02-15 15:11   ` Monk, Roger
  0 siblings, 0 replies; 14+ messages in thread
From: Monk, Roger @ 2012-02-15 15:11 UTC (permalink / raw)
  To: Peter A. Bigot, meta-ti@yoctoproject.org

On Tue, Jan 31, 2012 at 12:40:57, Peter A. Bigot wrote:
> Subject: [meta-ti] [PATCH 4/4] ti-dmai_svn: correct path within svn
> checkout
>
> From: "Peter A. Bigot" <bigotp@acm.org>
>
> At r642 as currently retrieved, the contents of ${S} are
> subdirectories dmai and patches, so the subsequent attempt to run
> "make clean" in ${S} fails because there is no Makefile there.  Append
> /dmai to the default ${S}, remove the now incorrect /dmai from
> LIC_FILES_CHKSUM, and correct the strippath for the patches to reflect the new starting location.
>
> Signed-off-by: Peter A. Bigot <bigotp@acm.org>

Acked-by: Roger Monk <r-monk@ti.com>

> ---
>  recipes-ti/dmai/ti-dmai_svn.bb |    9 ++++++---
>  1 files changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/recipes-ti/dmai/ti-dmai_svn.bb b/recipes-ti/dmai/ti-
> dmai_svn.bb index 3c5595d..3ac7a81 100644
> --- a/recipes-ti/dmai/ti-dmai_svn.bb
> +++ b/recipes-ti/dmai/ti-dmai_svn.bb
> @@ -1,11 +1,14 @@
>  require ti-dmai.inc
>
> +# Need to descend one level more to find source S =
> +"${WORKDIR}/${DMAIBRANCH}/davinci_multimedia_application_interface/dm
> +a
> i_${PV}/dmai"
> +
>  # Hack to be able to use recent kernel headers from userspace
> TARGET_CC_ARCH += " -D__EXPORTED_HEADERS__"
>
>  PV = "2_10_00_01+svnr${SRCPV}"
>
> -LIC_FILES_CHKSUM =
> "file://dmai/dmai_${PV}_License.html;md5=3302f728a5a42f97cabc26a54d7fa
> 6
> 07"
> +LIC_FILES_CHKSUM =
> "file://dmai_${PV}_License.html;md5=3302f728a5a42f97cabc26a54d7fa607"
>
>  # This package has high dependence on kernel, use kernel PR as base
> and append a local version  PR = "${MACHINE_KERNEL_PR}"
> @@ -29,7 +32,7 @@ SRCREV_omapl137       = "482"
>  SRCREV_omapl138       = "570"
>  SRCREV               ?= "UNDEFINED_SRCREV"
>
> -SRC_URI_append = " file://omap3530-r642-remove-include-
> videodev.diff;striplevel=2 \
> -                   file://r642-fix-config-bld.diff;striplevel=2 \
> +SRC_URI_append = " file://omap3530-r642-remove-include-
> videodev.diff;striplevel=3 \
> +                   file://r642-fix-config-bld.diff;striplevel=3 \
>                   "
>
> --
> 1.7.6.4
>
>
Texas Instruments Limited, 800 Pavilion Drive, Northampton, NN4 7YL. Registered in England & Wales under company number 00574102

_______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti





^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 2/4] various: fix installation directories for corrected ti-eula-unpack
  2012-01-31 12:40 ` [PATCH 2/4] various: fix installation directories for corrected ti-eula-unpack Peter A. Bigot
@ 2012-02-15 15:11   ` Monk, Roger
  0 siblings, 0 replies; 14+ messages in thread
From: Monk, Roger @ 2012-02-15 15:11 UTC (permalink / raw)
  To: Peter A. Bigot, meta-ti@yoctoproject.org

On Tue, Jan 31, 2012 at 12:40:55, Peter A. Bigot wrote:
> Subject: [meta-ti] [PATCH 2/4] various: fix installation directories
> for corrected ti-eula-unpack
>
> From: "Peter A. Bigot" <bigotp@acm.org>
>
> Signed-off-by: Peter A. Bigot <bigotp@acm.org>

Acked-by: Roger Monk <r-monk@ti.com>

> ---
>  recipes-ti/bios/ti-dspbios.inc      |    3 ++-
>  recipes-ti/devtools/ti-xdctools.inc |    3 ++-
>  recipes-ti/edma3lld/ti-edma3lld.inc |    3 ++-
>  3 files changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/recipes-ti/bios/ti-dspbios.inc b/recipes-ti/bios/ti-
> dspbios.inc index 8519039..0e3fa62 100644
> --- a/recipes-ti/bios/ti-dspbios.inc
> +++ b/recipes-ti/bios/ti-dspbios.inc
> @@ -10,7 +10,8 @@ require ../includes/ti-paths.inc  require
> ../includes/ti-staging.inc  require ../includes/ti-eula-unpack.inc
>
> -S = "${WORKDIR}/bios_${PV}"
> +TI_BIN_UNPK_WDEXT="/bios_${PV}"
> +S = "${WORKDIR}${TI_BIN_UNPK_WDEXT}"
>
>  SRC_URI = "http://software-
> dl.ti.com/dsps/dsps_public_sw/sdo_sb/targetcontent/bios/dspbios/${PV}/
> e xports/bios_setuplinux_${PV}.bin;name=biosbin"
>
> diff --git a/recipes-ti/devtools/ti-xdctools.inc b/recipes-
> ti/devtools/ti-xdctools.inc index c76ad80..547a555 100644
> --- a/recipes-ti/devtools/ti-xdctools.inc
> +++ b/recipes-ti/devtools/ti-xdctools.inc
> @@ -12,7 +12,8 @@ S = "${WORKDIR}/xdctools_${PV}"
>  SRC_URI = "http://software-
> dl.ti.com/dsps/dsps_public_sw/sdo_sb/targetcontent/rtsc/${PV}/exports/
> x dctools_setuplinux_${PV}.bin;name=xdcbin"
>
>  BINFILE="xdctools_setuplinux_${PV}.bin"
> -TI_BIN_UNPK_CMDS="Y: qY:workdir:Y"
> +TI_BIN_UNPK_CMDS="Y:workdir:Y"
> +TI_BIN_UNPK_WDEXT="/ti"
>
>  do_install() {
>      install -d ${D}${XDC_INSTALL_DIR_RECIPE} diff --git a/recipes-
> ti/edma3lld/ti-edma3lld.inc b/recipes-ti/edma3lld/ti-edma3lld.inc
> index d4e8557..fb131b6 100644
> --- a/recipes-ti/edma3lld/ti-edma3lld.inc
> +++ b/recipes-ti/edma3lld/ti-edma3lld.inc
> @@ -8,7 +8,8 @@ require ../includes/ti-paths.inc  require
> ../includes/ti-staging.inc  require ../includes/ti-eula-unpack.inc
>
> -S = "${WORKDIR}/edma3_lld_${PV}"
> +TI_BIN_UNPK_WDEXT="/edma3_lld_${PV}"
> +S = "${WORKDIR}${TI_BIN_UNPK_WDEXT}"
>
>  SRC_URI = "http://software-
> dl.ti.com/dsps/dsps_public_sw/${SRC_URI_edma3lldsite}/edma3_lld/edma3-
> lld-
> bios5/${PV}/exports/EDMA3_LLD_setuplinux_${PV}.bin;name=edma3lldbin"
>
> --
> 1.7.6.4
>
>
Texas Instruments Limited, 800 Pavilion Drive, Northampton, NN4 7YL. Registered in England & Wales under company number 00574102

_______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti





^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 3/4] ti-codecs-omap3530: fix for corrected ti-eula-unpack
  2012-01-31 12:40 ` [PATCH 3/4] ti-codecs-omap3530: fix " Peter A. Bigot
@ 2012-02-15 15:11   ` Monk, Roger
  0 siblings, 0 replies; 14+ messages in thread
From: Monk, Roger @ 2012-02-15 15:11 UTC (permalink / raw)
  To: Peter A. Bigot, meta-ti@yoctoproject.org

On Tue, Jan 31, 2012 at 12:40:56, Peter A. Bigot wrote:
> Subject: [meta-ti] [PATCH 3/4] ti-codecs-omap3530: fix for corrected
> ti-eula-unpack
>
> From: "Peter A. Bigot" <bigotp@acm.org>
>
> Remove directory prefix for tar files that use TI_BIN_UNPK_CMDS that
> specifies unpacking into workdir; formerly that specification was
> ignored and these were unpacked into a package-specific subdirectory
> of ${HOME}=${WORKDIR}.  Note c64xplus_deinterlacer overrides
> TI_BIN_UNPK_CMDS and does not specify workdir, so its prefix is
> correct.
>
> Signed-off-by: Peter A. Bigot <bigotp@acm.org>

Acked-by: Roger Monk <r-monk@ti.com>

> ---
>  .../codec-engine/ti-codecs-omap3530_4.00.00.00.bb  |   20 ++++++++++--
> --------
>  1 files changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/recipes-ti/codec-engine/ti-codecs-omap3530_4.00.00.00.bb
> b/recipes-ti/codec-engine/ti-codecs-omap3530_4.00.00.00.bb
> index ae218b1..ea9f389 100644
> --- a/recipes-ti/codec-engine/ti-codecs-omap3530_4.00.00.00.bb
> +++ b/recipes-ti/codec-engine/ti-codecs-omap3530_4.00.00.00.bb
> @@ -77,43 +77,43 @@ python do_unpack () {
>      bb.build.exec_func('base_do_unpack', d)
>
>      bb.data.setVar("BINFILE",
> "omap3530_h264enc_2_01_013_production.bin", d)
> -    bb.data.setVar("TARFILE",
> "h264enc_dm6467_1_20_00_08/omap3530_h264enc_2_01_013_production.tar",
> d)
> +    bb.data.setVar("TARFILE",
> + "omap3530_h264enc_2_01_013_production.tar", d)
>      bb.build.exec_func('ti_bin_do_unpack', d)
>
>      bb.data.setVar("BINFILE",
> "omap3530_h264dec_2_01_007_production.bin", d)
> -    bb.data.setVar("TARFILE",
> "omap3530_h264dec_2_01_007_production/omap3530_h264dec_2_01_007_produc
> t
> ion.tar", d)
> +    bb.data.setVar("TARFILE",
> + "omap3530_h264dec_2_01_007_production.tar", d)
>      bb.build.exec_func('ti_bin_do_unpack', d)
>
>      bb.data.setVar("BINFILE",
> "omap3530_jpegenc_02_01_01_00_production.bin", d)
> -    bb.data.setVar("TARFILE",
> "omap3530_jpegenc_02_01_01_00_production/omap3530_jpegenc_02_01_01_00_
> p
> roduction.tar", d)
> +    bb.data.setVar("TARFILE",
> + "omap3530_jpegenc_02_01_01_00_production.tar", d)
>      bb.build.exec_func('ti_bin_do_unpack', d)
>
>      bb.data.setVar("BINFILE",
> "c64xplus_jpegdec_02_00_01_01_production.bin", d)
> -    bb.data.setVar("TARFILE",
> "c64xplus_jpegdec_02_00_01_01_production/c64xplus_jpegdec_02_00_01_01_
> p
> roduction.tar", d)
> +    bb.data.setVar("TARFILE",
> + "c64xplus_jpegdec_02_00_01_01_production.tar", d)
>      bb.build.exec_func('ti_bin_do_unpack', d)
>
>      bb.data.setVar("BINFILE",
> "c64xplus_mpeg2dec_02_00_02_00_production.bin", d)
> -    bb.data.setVar("TARFILE",
> "c64xplus_mpeg2dec_02_00_02_00_production/c64xplus_mpeg2dec_02_00_02_0
> 0
> _production.tar", d)
> +    bb.data.setVar("TARFILE",
> + "c64xplus_mpeg2dec_02_00_02_00_production.tar", d)
>      bb.build.exec_func('ti_bin_do_unpack', d)
>
>      bb.data.setVar("BINFILE",
> "omap3530_mpeg4enc_02_04_00_00_production.bin", d)
> -    bb.data.setVar("TARFILE",
> "omap3530_mpeg4enc_02_04_00_00_production/omap3530_mpeg4enc_02_04_00_0
> 0
> _production.tar", d)
> +    bb.data.setVar("TARFILE",
> + "omap3530_mpeg4enc_02_04_00_00_production.tar", d)
>      bb.build.exec_func('ti_bin_do_unpack', d)
>
>      bb.data.setVar("BINFILE",
> "c64xplus_mpeg4dec_02_01_00_00_production.bin", d)
> -    bb.data.setVar("TARFILE",
> "h264enc_dm6467_1_20_00_08/c64xplus_mpeg4dec_02_01_00_00_production.ta
> r
> ", d)
> +    bb.data.setVar("TARFILE",
> + "c64xplus_mpeg4dec_02_01_00_00_production.tar", d)
>      bb.build.exec_func('ti_bin_do_unpack', d)
>
>      bb.data.setVar("BINFILE",
> "c64xplus_aachedec_01_30_03_00_production.bin", d)
> -    bb.data.setVar("TARFILE",
> "c64xplus_aachedec_01_30_03_00_production/dm6446_aachedec_01_30_03_00_
> p
> roduction.tar", d)
> +    bb.data.setVar("TARFILE",
> + "dm6446_aachedec_01_30_03_00_production.tar", d)
>      bb.build.exec_func('ti_bin_do_unpack', d)
>
>      bb.data.setVar("BINFILE",
> "c64xplus_g711_1_12_00_000_production.bin", d)
> -    bb.data.setVar("TARFILE",
> "h264enc_dm6467_1_20_00_08/dm6446_g711enc_1_12_00_000_production.tar",
> d)
> +    bb.data.setVar("TARFILE",
> + "dm6446_g711enc_1_12_00_000_production.tar", d)
>      bb.build.exec_func('ti_bin_do_unpack', d)
>
>      bb.data.setVar("BINFILE",
> "c64xplus_g711_1_12_00_000_production.bin", d)
> -    bb.data.setVar("TARFILE",
> "h264enc_dm6467_1_20_00_08/dm6446_g711dec_1_12_00_000_production.tar",
> d)
> +    bb.data.setVar("TARFILE",
> + "dm6446_g711dec_1_12_00_000_production.tar", d)
>      bb.build.exec_func('ti_bin_do_unpack', d)
>
>      bb.data.setVar("BINFILE",
> "c64xplus_deinterlacer_01_00_00_07_production.bin", d)
> --
> 1.7.6.4
>
>
Texas Instruments Limited, 800 Pavilion Drive, Northampton, NN4 7YL. Registered in England & Wales under company number 00574102

_______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti





^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 0/4] Resubmit TI DSP build issues related to installer
  2012-02-01 20:41   ` Peter Bigot
@ 2012-02-15 15:11     ` Monk, Roger
  2012-02-15 15:32       ` Peter Bigot
  0 siblings, 1 reply; 14+ messages in thread
From: Monk, Roger @ 2012-02-15 15:11 UTC (permalink / raw)
  To: Peter Bigot; +Cc: meta-ti@yoctoproject.org

On Wed, Feb 01, 2012 at 20:41:26, Peter Bigot wrote:
> Cc: meta-ti@yoctoproject.org
> Subject: Re: [meta-ti] [PATCH 0/4] Resubmit TI DSP build issues
> related to installer
>
> On Tue, Jan 31, 2012 at 6:45 PM, Monk, Roger <r-monk@ti.com> wrote:
> > Hi Peter,
> >
> > On Tue, Jan 31, 2012 at 12:40:53, Peter A. Bigot wrote:
> >> Subject: [meta-ti] [PATCH 0/4] Resubmit TI DSP build issues related
> >> to installer
> >>
> >> From: "Peter A. Bigot" <bigotp@acm.org>
> >>
> >> This patch set re-submits the (updated) uncommitted parts of my
> >> 27Dec2011 patch set to get gstreamer-ti and its dependencies to
> build
> >> in a pristine work environment.  It was left back on 29Dec2011 with:
> >>
> >> > I'm going to wait for Roger to get back from holidays to comment
> on
> >> the installer issues, I'll push the CE and dmai patches later today.
> >>
> >
> > Thanks for re-basing/re-submitting - sorry this has taken a while.
>  Your patches look good - however, I have some other failing installer
> scenarios like these for which I have a similar, but unfortunately,
> not the same patchset, in progress.  I hope to review yours in the
> coming days if ok.
>
> That's fine.  Rather than bug everybody with update emails while the
> two approaches are reconciled, these fixes are available at
> https://github.com/pabigot/meta-ti/tree/ti-eula-unpack.  There you'll
> find the corresponding change as required for omap3-sgx-modules.
>
> Peter
>

Thanks Peter - sorry for the delay.  Looks like I can add my patches separately.  I follow-up with those.

You're patchset looks good to go from me - thanks for the time and effort you put into this.

~roger


> >> The installer issues are:
> >>
> >> ti-eula-unpack spawns the installer, then writes canned input to it
> >> to do things like set the output directory.  The function that does
> >> this explicitly tacks a second newline on, in addition to the one
> >> normally added by Python's print statement.
> >>
> >> For ti-cgt6x, the result is that a bare newline is fed to the
> >> installer at the point where the installation directory is
> requested,
> >> instead of the workdir.  Hence, the default directory gets used;
> >> for this package, that's in /opt/TI.
> >>
> >> For ti-xdctools, ti-edma3lld, and ti-dspbios, the bare newline also
> >> caused the default installation directory to be requested, but for
> >> this package that's ${HOME}/ti.  Since ti-eula-unpack overrides the
> >> HOME environment variable to be the work directory, everything
> >> somehow happened to work out, at least for other folks.  I don't
> >> think that's a great solution though, so I updated the recipes to
> >> do it "right", by changing the commands to conform to what the
> >> current
> installers expect.
> >> There may be other installation recipes for other tools that need
> the
> >> same fixes.
> >>
> >> For the 31Jan2012 attempt, similar changes had to be made to
> >> ti-codecs- omap3530.
> >>
> >> For the 31Jan2012 attempt, the ti-dmai build required more than
> >> just altering the striplevel: the ${S} value had to be changed
> >> since
> there
> >> was no Makefile at its original value.  There is still an issue
> >> that prevents use of where make 3.85 produces a "Entering an
> >> unknown directory" failure in do_install, but that's an upstream
> >> problem: I just used make 3.81 to complete the build.
> >>
> >> Peter A. Bigot (4):
> >>   ti-eula-unpack: remove extraneous newline in cmd output
> >>   various: fix installation directories for corrected
> >> ti-eula-unpack
> >>   ti-codecs-omap3530: fix for corrected ti-eula-unpack
> >>   ti-dmai_svn: correct path within svn checkout
> >>
> >>  recipes-ti/bios/ti-dspbios.inc                     |    3 ++-
> >>  .../codec-engine/ti-codecs-omap3530_4.00.00.00.bb  |   20
> >> ++++++++++--
> >> --------
> >>  recipes-ti/devtools/ti-xdctools.inc                |    3 ++-
> >>  recipes-ti/dmai/ti-dmai_svn.bb                     |    9
> >> ++++++---
> >>  recipes-ti/edma3lld/ti-edma3lld.inc                |    3 ++-
> >>  recipes-ti/includes/ti-eula-unpack.inc             |    2 +-
> >>  6 files changed, 23 insertions(+), 17 deletions(-)
> >>
> >> --
> >> 1.7.6.4
> >>
> >>
> > Texas Instruments Limited, 800 Pavilion Drive, Northampton, NN4 7YL.
> > Registered in England & Wales under company number 00574102
> >
> >
Texas Instruments Limited, 800 Pavilion Drive, Northampton, NN4 7YL. Registered in England & Wales under company number 00574102

_______________________________________________
> >> meta-ti mailing list
> >> meta-ti@yoctoproject.org
> >> https://lists.yoctoproject.org/listinfo/meta-ti
> >
> >
> >





^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 0/4] Resubmit TI DSP build issues related to installer
  2012-02-15 15:11     ` Monk, Roger
@ 2012-02-15 15:32       ` Peter Bigot
  0 siblings, 0 replies; 14+ messages in thread
From: Peter Bigot @ 2012-02-15 15:32 UTC (permalink / raw)
  To: Monk, Roger; +Cc: meta-ti@yoctoproject.org

[-- Attachment #1: Type: text/plain, Size: 5446 bytes --]

Thanks.  I just sent the one additional change that I made on the mentioned
branch after the ones I'd previously submitted.

I'm not working BeagleBoard this week, so if there are problems due to
subsequent changes and things need to be rebased I won't be able to get to
it until next week.  I think I saw something about dmai that might conflict.

Peter

On Wed, Feb 15, 2012 at 9:11 AM, Monk, Roger <r-monk@ti.com> wrote:

> On Wed, Feb 01, 2012 at 20:41:26, Peter Bigot wrote:
> > Cc: meta-ti@yoctoproject.org
> > Subject: Re: [meta-ti] [PATCH 0/4] Resubmit TI DSP build issues
> > related to installer
> >
> > On Tue, Jan 31, 2012 at 6:45 PM, Monk, Roger <r-monk@ti.com> wrote:
> > > Hi Peter,
> > >
> > > On Tue, Jan 31, 2012 at 12:40:53, Peter A. Bigot wrote:
> > >> Subject: [meta-ti] [PATCH 0/4] Resubmit TI DSP build issues related
> > >> to installer
> > >>
> > >> From: "Peter A. Bigot" <bigotp@acm.org>
> > >>
> > >> This patch set re-submits the (updated) uncommitted parts of my
> > >> 27Dec2011 patch set to get gstreamer-ti and its dependencies to
> > build
> > >> in a pristine work environment.  It was left back on 29Dec2011 with:
> > >>
> > >> > I'm going to wait for Roger to get back from holidays to comment
> > on
> > >> the installer issues, I'll push the CE and dmai patches later today.
> > >>
> > >
> > > Thanks for re-basing/re-submitting - sorry this has taken a while.
> >  Your patches look good - however, I have some other failing installer
> > scenarios like these for which I have a similar, but unfortunately,
> > not the same patchset, in progress.  I hope to review yours in the
> > coming days if ok.
> >
> > That's fine.  Rather than bug everybody with update emails while the
> > two approaches are reconciled, these fixes are available at
> > https://github.com/pabigot/meta-ti/tree/ti-eula-unpack.  There you'll
> > find the corresponding change as required for omap3-sgx-modules.
> >
> > Peter
> >
>
> Thanks Peter - sorry for the delay.  Looks like I can add my patches
> separately.  I follow-up with those.
>
> You're patchset looks good to go from me - thanks for the time and effort
> you put into this.
>
> ~roger
>
>
> > >> The installer issues are:
> > >>
> > >> ti-eula-unpack spawns the installer, then writes canned input to it
> > >> to do things like set the output directory.  The function that does
> > >> this explicitly tacks a second newline on, in addition to the one
> > >> normally added by Python's print statement.
> > >>
> > >> For ti-cgt6x, the result is that a bare newline is fed to the
> > >> installer at the point where the installation directory is
> > requested,
> > >> instead of the workdir.  Hence, the default directory gets used;
> > >> for this package, that's in /opt/TI.
> > >>
> > >> For ti-xdctools, ti-edma3lld, and ti-dspbios, the bare newline also
> > >> caused the default installation directory to be requested, but for
> > >> this package that's ${HOME}/ti.  Since ti-eula-unpack overrides the
> > >> HOME environment variable to be the work directory, everything
> > >> somehow happened to work out, at least for other folks.  I don't
> > >> think that's a great solution though, so I updated the recipes to
> > >> do it "right", by changing the commands to conform to what the
> > >> current
> > installers expect.
> > >> There may be other installation recipes for other tools that need
> > the
> > >> same fixes.
> > >>
> > >> For the 31Jan2012 attempt, similar changes had to be made to
> > >> ti-codecs- omap3530.
> > >>
> > >> For the 31Jan2012 attempt, the ti-dmai build required more than
> > >> just altering the striplevel: the ${S} value had to be changed
> > >> since
> > there
> > >> was no Makefile at its original value.  There is still an issue
> > >> that prevents use of where make 3.85 produces a "Entering an
> > >> unknown directory" failure in do_install, but that's an upstream
> > >> problem: I just used make 3.81 to complete the build.
> > >>
> > >> Peter A. Bigot (4):
> > >>   ti-eula-unpack: remove extraneous newline in cmd output
> > >>   various: fix installation directories for corrected
> > >> ti-eula-unpack
> > >>   ti-codecs-omap3530: fix for corrected ti-eula-unpack
> > >>   ti-dmai_svn: correct path within svn checkout
> > >>
> > >>  recipes-ti/bios/ti-dspbios.inc                     |    3 ++-
> > >>  .../codec-engine/ti-codecs-omap3530_4.00.00.00.bb  |   20
> > >> ++++++++++--
> > >> --------
> > >>  recipes-ti/devtools/ti-xdctools.inc                |    3 ++-
> > >>  recipes-ti/dmai/ti-dmai_svn.bb                     |    9
> > >> ++++++---
> > >>  recipes-ti/edma3lld/ti-edma3lld.inc                |    3 ++-
> > >>  recipes-ti/includes/ti-eula-unpack.inc             |    2 +-
> > >>  6 files changed, 23 insertions(+), 17 deletions(-)
> > >>
> > >> --
> > >> 1.7.6.4
> > >>
> > >>
> > > Texas Instruments Limited, 800 Pavilion Drive, Northampton, NN4 7YL.
> > > Registered in England & Wales under company number 00574102
> > >
> > >
> Texas Instruments Limited, 800 Pavilion Drive, Northampton, NN4 7YL.
> Registered in England & Wales under company number 00574102
>
> _______________________________________________
> > >> meta-ti mailing list
> > >> meta-ti@yoctoproject.org
> > >> https://lists.yoctoproject.org/listinfo/meta-ti
> > >
> > >
> > >
>
>
>
>

[-- Attachment #2: Type: text/html, Size: 7378 bytes --]

^ permalink raw reply	[flat|nested] 14+ messages in thread

end of thread, other threads:[~2012-02-15 15:33 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-01-31 12:40 [PATCH 0/4] Resubmit TI DSP build issues related to installer Peter A. Bigot
2012-01-31 12:40 ` [PATCH 1/4] ti-eula-unpack: remove extraneous newline in cmd output Peter A. Bigot
2012-02-15 15:11   ` Monk, Roger
2012-01-31 12:40 ` [PATCH 2/4] various: fix installation directories for corrected ti-eula-unpack Peter A. Bigot
2012-02-15 15:11   ` Monk, Roger
2012-01-31 12:40 ` [PATCH 3/4] ti-codecs-omap3530: fix " Peter A. Bigot
2012-02-15 15:11   ` Monk, Roger
2012-01-31 12:40 ` [PATCH 4/4] ti-dmai_svn: correct path within svn checkout Peter A. Bigot
2012-02-15 15:11   ` Monk, Roger
2012-02-01  0:45 ` [PATCH 0/4] Resubmit TI DSP build issues related to installer Monk, Roger
2012-02-01 20:41   ` Peter Bigot
2012-02-15 15:11     ` Monk, Roger
2012-02-15 15:32       ` Peter Bigot
2012-02-11  4:03   ` Denys Dmytriyenko

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.