All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Ross Burton" <ross@burtonini.com>
To: Alexander Kanavin <alex.kanavin@gmail.com>
Cc: Randy MacLeod <randy.macleod@windriver.com>,
	Robert Yang <liezhi.yang@windriver.com>,
	 OE-core <openembedded-core@lists.openembedded.org>
Subject: Re: [OE-core] [PATCH 1/1] assimp: Remove it
Date: Wed, 8 Sep 2021 11:08:02 +0100	[thread overview]
Message-ID: <CAAnfSTu2fs=tG7NEW6w5PxLXsczTB5g3cm9_Y5jPgugA+-mc=A@mail.gmail.com> (raw)
In-Reply-To: <CANNYZj_6j9W0KKuR68KjRk+=MTQx_EOcuWnH3Zt1aKZHJZzvHA@mail.gmail.com>

On Tue, 7 Sept 2021 at 13:50, Alexander Kanavin <alex.kanavin@gmail.com> wrote:
> I'm not sure why layerindex says otherewise, but neither vulkan nor anything in meta-qt5 actually depend on assimp.
> qt5/6 has an option, off by default, to use external assimp, vulkan has no dependency at all.
>
> This leaves basically meta-ros, so carrying assimp for the benefit of a single layer is dubious at best.

I thought Vulkan needed assimp so dug into the history.  vulkan-demos
used to need it, but that was replaced with vulkan-samples which does
not.

So, I say we drop assimp from core and move it to meta-oe.

Ross

  reply	other threads:[~2021-09-08 10:08 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-07  9:17 [PATCH 0/1] assimp: Remove it Robert Yang
2021-09-07  9:17 ` [PATCH 1/1] " Robert Yang
2021-09-07 12:04   ` [OE-core] " Randy MacLeod
2021-09-07 12:10     ` Ross Burton
2021-09-07 19:01       ` Khem Raj
2021-09-08  8:55       ` Robert Yang
2021-09-08  9:33         ` Mikko Rapeli
2021-09-08  9:41           ` Robert Yang
2021-09-07 12:49     ` Alexander Kanavin
2021-09-08 10:08       ` Ross Burton [this message]
2021-09-08 10:34         ` Richard Purdie
2021-09-09  5:05         ` Khem Raj
2021-09-09  5:06 ` [OE-core] [PATCH 0/1] " Khem Raj
2021-09-09 10:26   ` Robert Yang
2021-09-09 11:19     ` Alexander Kanavin
2021-09-09 15:43     ` Khem Raj
2021-09-10  8:05       ` Robert Yang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAAnfSTu2fs=tG7NEW6w5PxLXsczTB5g3cm9_Y5jPgugA+-mc=A@mail.gmail.com' \
    --to=ross@burtonini.com \
    --cc=alex.kanavin@gmail.com \
    --cc=liezhi.yang@windriver.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=randy.macleod@windriver.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
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.