Hi Alex,

On Mon, 4 Sept 2023 at 11:04, Alexander Kanavin <[email protected]> wrote:
> On Mon, 4 Sept 2023 at 10:49, Benjamin Bara <[email protected]> wrote:
> > +do_configure:prepend () {
> > +    if ! [ "${@bb.utils.filter('TUNE_FEATURES', 'neon', d)}" ]; then
> > +        # unset the neon-specific variables if no neon available
> > +        sed -i 's/HAVE_NEON=".*"/HAVE_NEON=""/g' ${S}/configure.ac
> > +        sed -i 's/TESS_NEON=".*"/TESS_NEON=""/g' ${S}/configure.ac
>
> Ad hoc patching with sed should be avoided if possible because it's
> difficult to understand and is prone to silent breakage (e.g. no
> longer doing what it was meant to). Can you make a proper patch please
> and submit upstream as well? Perhaps add an explicit option?

Sure, I will post a v2 of this patch as soon as I found out how to contribute
to ghostscript :D

Thanks,
Benjamin
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#187172): 
https://lists.openembedded.org/g/openembedded-core/message/187172
Mute This Topic: https://lists.openembedded.org/mt/101144716/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to