Skip to content

why does the CMakeList check to ensure you provide a custom HTTPClient callback when building as a static library?聽#36

Open

Description

@Zephilinox

I assume there was some issue that caused this check to be added? I'm curious. I would have thought it would work just fine 馃

Activity

  1. ArtursKadikis commented on Jan 25, 2022

    @ArtursKadikis
    Member

    Which check specifically do you mean?

  2. Zephilinox commented on Jan 26, 2022

    @Zephilinox
    Author

    sorry, I should have linked to it directly :) it's this one https://github.com/Countly/countly-sdk-cpp/blob/master/CMakeLists.txt#L16

  3. ArtursKadikis commented on Jan 26, 2022

    @ArtursKadikis
    Member

    Hello. I'm not actually sure myself, the original developers are not part of the team anymore. After looking into git blame, it looks like they were added in this PR:
    #16

    And one of the issues it would fix is this:
    #15

  4. Zephilinox commented on Jan 26, 2022

    @Zephilinox
    Author

    hi :)

    Hello. I'm not actually sure myself, the original developers are not part of the team anymore.

    aw okay, no worries :)

    #16 fixes windows by excluding it from that check, so it wasn't the original reason for the check being added (for non-windows platforms) 馃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions