Re: [OE-core] [PATCH 0/1] assimp: Remove it
On 9/9/21 11:43 PM, Khem Raj wrote: On Thu, Sep 9, 2021 at 3:26 AM Robert Yang wrote: Hi Khem, On 9/9/21 1:06 PM, Khem Raj wrote: also delete meta/lib/oeqa/sdk/cases/assimp.py It's a test case for testing cmake, it should be OK to keep it? Or we need add a new one to replace it. is it downloading assimp code etc. ? if so I would suggest to replace it with something else too as a followup. Yes, it does, I've sent a patch to use expat to replace assimp for the test. // Robert // Robert On Tue, Sep 7, 2021 at 1:46 AM Robert Yang wrote: The following changes since commit 70de1dbb660461bdf0613494f53ec4c78738ae2a: vim: Add packageconfig for sound notification support (2021-09-06 09:49:13 +0100) are available in the Git repository at: git://git.openembedded.org/openembedded-core-contrib rbt/assimp http://cgit.openembedded.org/openembedded-core-contrib/log/?h=rbt/assimp Robert Yang (1): assimp: Remove it meta/conf/distro/include/maintainers.inc |1 - ...hared-lib-from-_IMPORT_CHECK_TARGETS.patch | 32 - ...hub.com-assimp-assimp-issues-2733-up.patch | 1664 - .../use-GNUInstallDirs-where-possible.patch | 257 --- meta/recipes-graphics/vulkan/assimp_5.0.1.bb | 26 - 5 files changed, 1980 deletions(-) delete mode 100644 meta/recipes-graphics/vulkan/assimp/0001-assimp-remove-shared-lib-from-_IMPORT_CHECK_TARGETS.patch delete mode 100644 meta/recipes-graphics/vulkan/assimp/0001-closes-https-github.com-assimp-assimp-issues-2733-up.patch delete mode 100644 meta/recipes-graphics/vulkan/assimp/use-GNUInstallDirs-where-possible.patch delete mode 100644 meta/recipes-graphics/vulkan/assimp_5.0.1.bb -- 2.17.1 -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#155895): https://lists.openembedded.org/g/openembedded-core/message/155895 Mute This Topic: https://lists.openembedded.org/mt/85431029/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [OE-core] [PATCH 0/1] assimp: Remove it
On Thu, Sep 9, 2021 at 3:26 AM Robert Yang wrote: > > Hi Khem, > > On 9/9/21 1:06 PM, Khem Raj wrote: > > also delete meta/lib/oeqa/sdk/cases/assimp.py > > It's a test case for testing cmake, it should be OK to keep it? Or we need > add a > new one to replace it. is it downloading assimp code etc. ? if so I would suggest to replace it with something else too as a followup. > > // Robert > > > > > On Tue, Sep 7, 2021 at 1:46 AM Robert Yang > > wrote: > >> > >> The following changes since commit > >> 70de1dbb660461bdf0613494f53ec4c78738ae2a: > >> > >>vim: Add packageconfig for sound notification support (2021-09-06 > >> 09:49:13 +0100) > >> > >> are available in the Git repository at: > >> > >>git://git.openembedded.org/openembedded-core-contrib rbt/assimp > >>http://cgit.openembedded.org/openembedded-core-contrib/log/?h=rbt/assimp > >> > >> Robert Yang (1): > >>assimp: Remove it > >> > >> meta/conf/distro/include/maintainers.inc |1 - > >> ...hared-lib-from-_IMPORT_CHECK_TARGETS.patch | 32 - > >> ...hub.com-assimp-assimp-issues-2733-up.patch | 1664 - > >> .../use-GNUInstallDirs-where-possible.patch | 257 --- > >> meta/recipes-graphics/vulkan/assimp_5.0.1.bb | 26 - > >> 5 files changed, 1980 deletions(-) > >> delete mode 100644 > >> meta/recipes-graphics/vulkan/assimp/0001-assimp-remove-shared-lib-from-_IMPORT_CHECK_TARGETS.patch > >> delete mode 100644 > >> meta/recipes-graphics/vulkan/assimp/0001-closes-https-github.com-assimp-assimp-issues-2733-up.patch > >> delete mode 100644 > >> meta/recipes-graphics/vulkan/assimp/use-GNUInstallDirs-where-possible.patch > >> delete mode 100644 meta/recipes-graphics/vulkan/assimp_5.0.1.bb > >> > >> -- > >> 2.17.1 > >> > >> > >> > >> -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#155861): https://lists.openembedded.org/g/openembedded-core/message/155861 Mute This Topic: https://lists.openembedded.org/mt/85431029/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [OE-core] [PATCH 0/1] assimp: Remove it
Right, that test is orthogonal and independent of the recipe, so should be kept until there's a replacement. Alex On Thu, 9 Sept 2021 at 12:26, Robert Yang wrote: > Hi Khem, > > On 9/9/21 1:06 PM, Khem Raj wrote: > > also delete meta/lib/oeqa/sdk/cases/assimp.py > > It's a test case for testing cmake, it should be OK to keep it? Or we need > add a > new one to replace it. > > // Robert > > > > > On Tue, Sep 7, 2021 at 1:46 AM Robert Yang > wrote: > >> > >> The following changes since commit > 70de1dbb660461bdf0613494f53ec4c78738ae2a: > >> > >>vim: Add packageconfig for sound notification support (2021-09-06 > 09:49:13 +0100) > >> > >> are available in the Git repository at: > >> > >>git://git.openembedded.org/openembedded-core-contrib rbt/assimp > >> > http://cgit.openembedded.org/openembedded-core-contrib/log/?h=rbt/assimp > >> > >> Robert Yang (1): > >>assimp: Remove it > >> > >> meta/conf/distro/include/maintainers.inc |1 - > >> ...hared-lib-from-_IMPORT_CHECK_TARGETS.patch | 32 - > >> ...hub.com-assimp-assimp-issues-2733-up.patch | 1664 - > >> .../use-GNUInstallDirs-where-possible.patch | 257 --- > >> meta/recipes-graphics/vulkan/assimp_5.0.1.bb | 26 - > >> 5 files changed, 1980 deletions(-) > >> delete mode 100644 > meta/recipes-graphics/vulkan/assimp/0001-assimp-remove-shared-lib-from-_IMPORT_CHECK_TARGETS.patch > >> delete mode 100644 > meta/recipes-graphics/vulkan/assimp/0001-closes-https-github.com-assimp-assimp-issues-2733-up.patch > >> delete mode 100644 > meta/recipes-graphics/vulkan/assimp/use-GNUInstallDirs-where-possible.patch > >> delete mode 100644 meta/recipes-graphics/vulkan/assimp_5.0.1.bb > >> > >> -- > >> 2.17.1 > >> > >> > >> > >> > > > > -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#155854): https://lists.openembedded.org/g/openembedded-core/message/155854 Mute This Topic: https://lists.openembedded.org/mt/85431029/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [OE-core] [PATCH 0/1] assimp: Remove it
Hi Khem, On 9/9/21 1:06 PM, Khem Raj wrote: also delete meta/lib/oeqa/sdk/cases/assimp.py It's a test case for testing cmake, it should be OK to keep it? Or we need add a new one to replace it. // Robert On Tue, Sep 7, 2021 at 1:46 AM Robert Yang wrote: The following changes since commit 70de1dbb660461bdf0613494f53ec4c78738ae2a: vim: Add packageconfig for sound notification support (2021-09-06 09:49:13 +0100) are available in the Git repository at: git://git.openembedded.org/openembedded-core-contrib rbt/assimp http://cgit.openembedded.org/openembedded-core-contrib/log/?h=rbt/assimp Robert Yang (1): assimp: Remove it meta/conf/distro/include/maintainers.inc |1 - ...hared-lib-from-_IMPORT_CHECK_TARGETS.patch | 32 - ...hub.com-assimp-assimp-issues-2733-up.patch | 1664 - .../use-GNUInstallDirs-where-possible.patch | 257 --- meta/recipes-graphics/vulkan/assimp_5.0.1.bb | 26 - 5 files changed, 1980 deletions(-) delete mode 100644 meta/recipes-graphics/vulkan/assimp/0001-assimp-remove-shared-lib-from-_IMPORT_CHECK_TARGETS.patch delete mode 100644 meta/recipes-graphics/vulkan/assimp/0001-closes-https-github.com-assimp-assimp-issues-2733-up.patch delete mode 100644 meta/recipes-graphics/vulkan/assimp/use-GNUInstallDirs-where-possible.patch delete mode 100644 meta/recipes-graphics/vulkan/assimp_5.0.1.bb -- 2.17.1 -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#155851): https://lists.openembedded.org/g/openembedded-core/message/155851 Mute This Topic: https://lists.openembedded.org/mt/85431029/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [OE-core] [PATCH 0/1] assimp: Remove it
also delete meta/lib/oeqa/sdk/cases/assimp.py On Tue, Sep 7, 2021 at 1:46 AM Robert Yang wrote: > > The following changes since commit 70de1dbb660461bdf0613494f53ec4c78738ae2a: > > vim: Add packageconfig for sound notification support (2021-09-06 09:49:13 > +0100) > > are available in the Git repository at: > > git://git.openembedded.org/openembedded-core-contrib rbt/assimp > http://cgit.openembedded.org/openembedded-core-contrib/log/?h=rbt/assimp > > Robert Yang (1): > assimp: Remove it > > meta/conf/distro/include/maintainers.inc |1 - > ...hared-lib-from-_IMPORT_CHECK_TARGETS.patch | 32 - > ...hub.com-assimp-assimp-issues-2733-up.patch | 1664 - > .../use-GNUInstallDirs-where-possible.patch | 257 --- > meta/recipes-graphics/vulkan/assimp_5.0.1.bb | 26 - > 5 files changed, 1980 deletions(-) > delete mode 100644 > meta/recipes-graphics/vulkan/assimp/0001-assimp-remove-shared-lib-from-_IMPORT_CHECK_TARGETS.patch > delete mode 100644 > meta/recipes-graphics/vulkan/assimp/0001-closes-https-github.com-assimp-assimp-issues-2733-up.patch > delete mode 100644 > meta/recipes-graphics/vulkan/assimp/use-GNUInstallDirs-where-possible.patch > delete mode 100644 meta/recipes-graphics/vulkan/assimp_5.0.1.bb > > -- > 2.17.1 > > > > -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#155842): https://lists.openembedded.org/g/openembedded-core/message/155842 Mute This Topic: https://lists.openembedded.org/mt/85431029/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-