Skip to content
  • Stephen Kelly's avatar
    Revert "Allow target_link_libraries with IMPORTED targets." · 48a4cf21
    Stephen Kelly authored
    This reverts commit 9cfe4f1b.
    
    It turns out that correctly adding the content to
    the IMPORTED_LINK_INTERFACE_LIBARIES_<CONFIG> of an upstream target
    from the buildsystem of a downstream project is not simple.
    
    If upstream had added the INTERFACE content, the config-specific
    properties would  be determined by the DEBUG_CONFIGURATIONS of
    upstream.
    
    As downstream, we don't have any information about what
    the DEBUG_CONFIGURATIONS of upstream were, so we can't determine
    which configuration-specific properties to populate. The best we can do
    is add it to all of them or add it to the ones downstream considers to
    be DEBUG_CONFIGURATIONS, neither of which is a good solution.
    
    So, removing the porcelain API for that is the best approach. A human
    can still determine which properties to populate and use
    the set_property API to populate the desired properies.
    
    Another solution to this would be for upstream targets to publish
    what they consider DEBUG_CONFIGURATIONS, but that can be added in
    a future release.
    48a4cf21