Remove "redundant" test condition in pcl_targets.cmake.

[SergioRAgostinho]

@jspricke Actually, do you see any reason for this test you referenced:

pcl/cmake/pcl_targets.cmake

Line 113 in a602b19

if(NOT ${EXT_DEP_FOUND} OR (NOT (${EXT_DEP_FOUND} STREQUAL "TRUE")))

Why isn't the first part of the condition sufficient? Looking at the Git history, this was added as a single commit and does not provide any motivation. Maybe we just drop it? Or better after the release?

Originally posted by @taketwo in #2563 (comment)

[jspricke]

my best bet would be https://cmake.org/cmake/help/latest/policy/CMP0012.html bot no idea otherwise.

[stale]

Marking this as stale due to 30 days of inactivity. It will be closed in 7 days if no further activity occurs.