Uploaded image for project: 'MariaDB Server'
  1. MariaDB Server
  2. MDEV-4174

MariaDB 5.5 thread pool code fails to build on OpenBSD / NetBSD / DragonFly

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 5.5.29
    • Fix Version/s: 5.5.30
    • Component/s: None
    • Labels:
      None
    • Environment:
      OpenBSD

      Description

      The thread pool code within MariaDB 5.5 and other newer/older releases fails to build on OpenBSD / NetBSD / DragonFly, etc. These other *BSD OS's also have kqueue support. The attached patch adds the other *BSD OS's to the #if def's within the code to make the code build.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            wlad Vladislav Vaintroub added a comment -

            Brad,
            Could you check if this patch http://bazaar.launchpad.net/~maria-captains/maria/5.5-wlad/revision/3662 satisfies needs on those platforms . It is basically yours, just that complicate preprocessor expression is replaced with system check (the preprocessor contant check grew a bit too large listing all BSD variants .I have just OSX and FreeBSD, so I do not know if this will 100% work on all others.
            Thanks!

            Show
            wlad Vladislav Vaintroub added a comment - Brad, Could you check if this patch http://bazaar.launchpad.net/~maria-captains/maria/5.5-wlad/revision/3662 satisfies needs on those platforms . It is basically yours, just that complicate preprocessor expression is replaced with system check (the preprocessor contant check grew a bit too large listing all BSD variants .I have just OSX and FreeBSD, so I do not know if this will 100% work on all others. Thanks!
            Hide
            brad0 Brad Smith added a comment -

            Duhh. Of course that is the better approach I will check this out with OpenBSD and get back to you.

            Show
            brad0 Brad Smith added a comment - Duhh. Of course that is the better approach I will check this out with OpenBSD and get back to you.
            Hide
            brad0 Brad Smith added a comment -

            This builds fine.

            Also does it make sense to have the #error directive at the top and one further down? One of them should be good enough and the one near the top is more visible when you first open the file in an editor.

            Thanks.

            Show
            brad0 Brad Smith added a comment - This builds fine. Also does it make sense to have the #error directive at the top and one further down? One of them should be good enough and the one near the top is more visible when you first open the file in an editor. Thanks.
            Hide
            wlad Vladislav Vaintroub added a comment -

            Yes, you're right about #error . I now left just one on top, removed second. Thanks.

            Show
            wlad Vladislav Vaintroub added a comment - Yes, you're right about #error . I now left just one on top, removed second. Thanks.

              People

              • Assignee:
                wlad Vladislav Vaintroub
                Reporter:
                brad0 Brad Smith
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 2 hours
                  2h