All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Adrian Bunk" <bunk@stusta.de>
To: Claude Bing <cbing@cybernetics.com>
Cc: openembedded-devel@lists.openembedded.org
Subject: Re: [oe] [meta-oe][PATCH] php: fix opcache link error in 7.4
Date: Fri, 8 May 2020 21:41:29 +0300	[thread overview]
Message-ID: <20200508184129.GA7248@localhost> (raw)
In-Reply-To: <090c0075-c785-e3fd-1da4-c56518a64c0c@cybernetics.com>

On Fri, May 08, 2020 at 10:42:22AM -0400, Claude Bing wrote:
> On 5/8/20 6:56 AM, Adrian Bunk wrote:
> > On Thu, May 07, 2020 at 02:28:06PM -0400, Claude Bing wrote:
> >> Explicitly specifying -lrt is required for opcache to be linked against
> >> the proper dependencies.  Additionally, PHP disables libdl when it
> >> detects a cross-compilation environment for some reason.  In order to
> >> load any type of extension, re-enabling libdl is required.
> >> ...
> >> +# link against librt (libc) if opcache is specified in order to avoid the
> >> +# following error:
> >> +# Failed loading /usr/lib/php7/extensions/no-debug-non-zts-20190902/opcache.so: /usr/lib/php7/extensions/no-debug-non-zts-20190902/opcache.so: undefined symbol: shm_unlink
> >> +LDFLAGS += "${@ " -lrt " if bb.utils.contains('PACKAGECONFIG', 'opcache', 'true', 'false', d) else "" }"
> > 
> > Is this actually a bug in 0001-opcache-config.m4-enable-opcache.patch
> > and should instead be fixed there?
> 
> Yes, that does indeed fix the problem.  Would you like me to reply to
> this thread with a new patch, or create a different thread?
>...

Your original patch is already in master, please submit a new patch in a 
different thread that removes this workaround from php_7.4.4.bb and 
fixes 0001-opcache-config.m4-enable-opcache.patch.

Thanks
Adrian

  reply	other threads:[~2020-05-08 18:41 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-05-07 18:28 [meta-oe][PATCH] php: fix opcache link error in 7.4 Claude Bing
2020-05-08 10:56 ` [oe] " Adrian Bunk
2020-05-08 14:42   ` Claude Bing
2020-05-08 18:41     ` Adrian Bunk [this message]
2020-05-08 21:27       ` Claude Bing
2020-05-09 18:54         ` Steve Sakoman
2020-05-11 14:32           ` Claude Bing

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=20200508184129.GA7248@localhost \
    --to=bunk@stusta.de \
    --cc=cbing@cybernetics.com \
    --cc=openembedded-devel@lists.openembedded.org \
    /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.