Deprecate and eventually remove NON_INTERACTIVE_BUILD ?

[traversaro]

The NON_INTERACTIVE_BUILD variable has confusing semantics (the main changes it does is that it set the EP_UPDATE_DISCONNECTED, see

ycm-cmake-modules/modules/YCMEPHelper.cmake

Line 212 in 46af9ad

set_property(DIRECTORY PROPERTY EP_UPDATE_DISCONNECTED 1)

) but this is not documented properly nor widely understood by YCM users. The main reason why it was used was to avoid the checks on git mail and name being set, that were anyhow removed in #472, but then that created more errors down the road (see #225 and #474).

So I think that we should first deprecate it, and then remove it.

[traversaro]

fyi @valegagge @CarlottaSartore

[valegagge]

giving my two cents, since we are going to remove the forcing of name and email git set, It seems to me a good idea to deprecate and then remove NO_INTERACTIVE_BUILD.

This should fix the issue pointed out here: https://github.com/ami-iit/element_joint-velocity-control/issues/10#issuecomment-2599253213