Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Are X11 extension checks in configure.in necessary? #65

Closed
SDLBugzilla opened this issue Feb 10, 2021 · 0 comments
Closed

Are X11 extension checks in configure.in necessary? #65

SDLBugzilla opened this issue Feb 10, 2021 · 0 comments

Comments

@SDLBugzilla
Copy link
Collaborator

This bug report was migrated from our old Bugzilla tracker.

Reported in version: HG 1.2
Reported for operating system, platform: Linux, x86

Comments on the original bug report:

On 2006-01-30 12:10:17 +0000, Ryan C. Gordon wrote:

We have a bunch of AC_TRY_COMPILE checks in the configure script for things like --enable-video-x11-vm ... these are now dynamically loaded, so we fail gracefully at runtime if the libraries aren't installed, and we include the extension code and headers in SDL, so we don't need to check if the build system has these extensions, either. Should we just remove these blocks from configure.in? Should we just remove the compile checks, leaving this as a switch to explicitly disable the support at build time?

--ryan.

On 2006-01-31 09:11:50 +0000, Sam Lantinga wrote:

(In reply to comment # 0)

We have a bunch of AC_TRY_COMPILE checks in the configure script for things
like --enable-video-x11-vm ... these are now dynamically loaded, so we fail
gracefully at runtime if the libraries aren't installed, and we include the
extension code and headers in SDL, so we don't need to check if the build
system has these extensions, either. Should we just remove these blocks from
configure.in? Should we just remove the compile checks, leaving this as a
switch to explicitly disable the support at build time?

Sure, that sounds fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant