Globbing is a bad idea in SRC_URI, it breaks the task checksums and
can't really be fixed. Since we're removing it, drop the test for
its interaction with recipetool.

Signed-off-by: Richard Purdie <richard.pur...@linuxfoundation.org>
---
 .../files/selftest-replaceme-src-globfile           |  1 -
 .../recipetool/selftest-recipetool-appendfile.bb    |  2 --
 meta/lib/oeqa/selftest/cases/recipetool.py          | 13 -------------
 3 files changed, 16 deletions(-)
 delete mode 100644 
meta-selftest/recipes-test/recipetool/files/selftest-replaceme-src-globfile

diff --git 
a/meta-selftest/recipes-test/recipetool/files/selftest-replaceme-src-globfile 
b/meta-selftest/recipes-test/recipetool/files/selftest-replaceme-src-globfile
deleted file mode 100644
index 1e20a2b03eb..00000000000
--- 
a/meta-selftest/recipes-test/recipetool/files/selftest-replaceme-src-globfile
+++ /dev/null
@@ -1 +0,0 @@
-A file matched by a glob in SRC_URI
diff --git 
a/meta-selftest/recipes-test/recipetool/selftest-recipetool-appendfile.bb 
b/meta-selftest/recipes-test/recipetool/selftest-recipetool-appendfile.bb
index 7375c479330..b5f976708f4 100644
--- a/meta-selftest/recipes-test/recipetool/selftest-recipetool-appendfile.bb
+++ b/meta-selftest/recipes-test/recipetool/selftest-recipetool-appendfile.bb
@@ -10,7 +10,6 @@ SRC_URI = "file://installscript.sh \
            file://file1 \
            file://add-file.patch \
            file://subdir \
-           file://selftest-replaceme-src-glob* \
            file://selftest-replaceme-inst-globfile \
            file://selftest-replaceme-inst-todir-globfile \
            file://selftest-replaceme-inst-func"
@@ -27,7 +26,6 @@ do_install() {
        install -m 0644 ${WORKDIR}/selftest-replaceme-todir ${D}${datadir}
        install -m 0644 ${WORKDIR}/file1 
${D}${datadir}/selftest-replaceme-renamed
        install -m 0644 ${WORKDIR}/subdir/fileinsubdir 
${D}${datadir}/selftest-replaceme-subdir
-       install -m 0644 ${WORKDIR}/selftest-replaceme-src-globfile 
${D}${datadir}/selftest-replaceme-src-globfile
        cp ${WORKDIR}/selftest-replaceme-inst-glob* 
${D}${datadir}/selftest-replaceme-inst-globfile
        cp ${WORKDIR}/selftest-replaceme-inst-todir-glob* ${D}${datadir}
        install -d ${D}${sysconfdir}
diff --git a/meta/lib/oeqa/selftest/cases/recipetool.py 
b/meta/lib/oeqa/selftest/cases/recipetool.py
index c2ade2543a3..6bac53cf3da 100644
--- a/meta/lib/oeqa/selftest/cases/recipetool.py
+++ b/meta/lib/oeqa/selftest/cases/recipetool.py
@@ -226,19 +226,6 @@ class RecipetoolTests(RecipetoolBase):
         _, output = 
self._try_recipetool_appendfile('selftest-recipetool-appendfile', 
'/usr/share/selftest-replaceme-subdir', self.testfile, '', expectedlines, 
['testfile'])
         self.assertNotIn('WARNING: ', output)
 
-    def test_recipetool_appendfile_src_glob(self):
-        # A file that's in SRC_URI as a glob
-        expectedlines = ['FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"\n',
-                         '\n',
-                         'SRC_URI += "file://testfile"\n',
-                         '\n',
-                         'do_install_append() {\n',
-                         '    install -d ${D}${datadir}\n',
-                         '    install -m 0644 ${WORKDIR}/testfile 
${D}${datadir}/selftest-replaceme-src-globfile\n',
-                         '}\n']
-        _, output = 
self._try_recipetool_appendfile('selftest-recipetool-appendfile', 
'/usr/share/selftest-replaceme-src-globfile', self.testfile, '', expectedlines, 
['testfile'])
-        self.assertNotIn('WARNING: ', output)
-
     def test_recipetool_appendfile_inst_glob(self):
         # A file that's in do_install as a glob
         expectedlines = ['FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"\n',
-- 
2.25.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#141817): 
https://lists.openembedded.org/g/openembedded-core/message/141817
Mute This Topic: https://lists.openembedded.org/mt/76406981/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to