Skip to contentSkip to conversation

sstate-diff-machines.sh: Replace MACHINE_ARCH only at the beginning and separated with dash

PATCH In review Martin Jansa 2 participants 2 messages May 30, 2017

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 sstate-diff-machines.sh: Replace MACHINE_ARCH only at the beginning and separated with dash

* I had some components where the MACHINE name was also included in PV of allarch recipe which was tripping the script into saying that they have different signatures (because for one MACHINE was the string in PV replaced with 'MACHINE' and not for other MACHINEs

---
 scripts/sstate-diff-machines.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
scripts/sstate-diff-machines.sh +1 -1
index 056aa0a04c..27c6a33006 100755
@@ -118,7 +118,7 @@ for M in ${machines}; do
     cp -ra ${tmpdir}/stamps/* ${OUTPUT}/${M}
     find ${OUTPUT}/${M} -name \*sigdata\* | sed "s#${OUTPUT}/${M}/##g" | sort > ${OUTPUT}/${M}/list
     M_UNDERSCORE=`echo ${M} | sed 's/-/_/g'`
-    sed "s/${M_UNDERSCORE}/MACHINE/g; s/${M}/MACHINE/g" ${OUTPUT}/${M}/list | sort > ${OUTPUT}/${M}/list.M
+    sed "s/^${M_UNDERSCORE}-/MACHINE/g" ${OUTPUT}/${M}/list | sort > ${OUTPUT}/${M}/list.M
     find ${tmpdir}/stamps/ -name \*sigdata\* | xargs rm -f
   else
     printf "ERROR: no sigdata files were generated for MACHINE $M in ${tmpdir}/stamps\n";
Signature · 1 line
2.13.0
#1 lore raw

✗ patchtest: failure for sstate-diff-machines.sh: Replace MACHINE_ARCH only at the beginning and separated with dash

== Series Details ==

Series: sstate-diff-machines.sh: Replace MACHINE_ARCH only at the beginning and separated with dash
Revision: 1
URL : https://patchwork.openembedded.org/series/6969/
State : failure

== Summary ==

Thank you for submitting this patch series to OpenEmbedded Core. This is an automated response. Several tests have been executed on the proposed series by patchtest resulting in the following failures:

* Patch            sstate-diff-machines.sh: Replace MACHINE_ARCH only at the beginning and separated with dash
 Issue             Commit shortlog is too long [test_shortlog_length] 
  Suggested fix    Edit shortlog so that it is 90 characters or less (currently 91 characters)

If you believe any of these test results are incorrect, please reply to the mailing list ([email protected]) raising your concerns. Otherwise we would appreciate you correcting the issues and submitting a new version of the patchset if applicable. Please ensure you add/increment the version number when sending the new version (i.e. [PATCH] -> [PATCH v2] -> [PATCH v3] -> ...).

---

Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite:     http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe