Dan Nicholson wrote: > On Tue, Feb 24, 2009 at 8:51 PM, Benjamin Close > <[email protected]> wrote: >> Signed-off-by: Benjamin Close <[email protected]> >> --- >> src/egl/main/Makefile | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/src/egl/main/Makefile b/src/egl/main/Makefile >> index 89b4a20..a543f67 100644 >> --- a/src/egl/main/Makefile >> +++ b/src/egl/main/Makefile >> @@ -63,7 +63,7 @@ library: $(TOP)/$(LIB_DIR)/libEGL.so >> $(TOP)/$(LIB_DIR)/libEGL.so: $(OBJECTS) >> $(MKLIB) -o EGL -linker '$(CC)' -ldflags '$(LDFLAGS)' \ >> -major 1 -minor 0 \ >> - -install $(TOP)/$(LIB_DIR) -ldl $(OBJECTS) $(LIBS) >> + -install $(TOP)/$(LIB_DIR) -dlopen $(OBJECTS) $(LIBS) > > Does this actually work? The -dlopen option is being passed to mklib, > which doesn't seem to do anything useful with it. I suspect this has > broken the build on linux.
I did a quick build test (on Linux) and it worked. -dlopen is a valid mklib option but a no-op for most platforms. Its meaning is different than -ldl though. > We already handle dlopen in libc on BSD in configure by setting > DLOPEN_LIBS and adding them to GL_LIB_DEPS and DRI_LIB_DEPS. We should > probably just do the same for egl. That sounds right. I'll look at that. -Brian _______________________________________________ xorg mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/xorg
