Add Kernel Version Check

[HTRamsey]

Adds a check for minimum supported kernel version

[AniruddhaKanhere]

The changes look good to me.

Thank you @HTRamsey for taking the time to contribute!

Thanks

[AniruddhaKanhere]

I suppose that this specific version is used due to the fact that before this pdMS_TO_TICKS did not exist which is used a lot in this library.

[HTRamsey]

It's actually from this but looking at the changelog myself I think maybe it should be v8.2 cause that's when task notifications were added. I can try and at least see the oldest version that will compile later.

[AniruddhaKanhere]

Thank you! That would be great.

[tony-josi-aws]

@HTRamsey

Task notifications are used by the IP stack only if ipconfigSELECT_USES_NOTIFY is enabled. Network interfaces might still use it, depending on how the EMAC task is implemented.

[AniruddhaKanhere]

@ActoryOu should we be including the kernel headers here

FreeRTOS-Plus-TCP/test/build-combination/Header_Self_Contain/headerSelfContain.cmake

Line 21 in df5aed9

target_compile_options(freertos_plus_tcp_build_test_${header}

before compile options as the build check is failing there.

[ActoryOu]

@ActoryOu should we be including the kernel headers here

FreeRTOS-Plus-TCP/test/build-combination/Header_Self_Contain/headerSelfContain.cmake

Line 21 in df5aed9

target_compile_options(freertos_plus_tcp_build_test_${header}

before compile options as the build check is failing there.

Sorry for late response.
Yeah, that helps. And @HTRamsey created a better solution by #1120.
Let's do it that way.

[tony-josi-aws]

@HTRamsey

Thanks for this PR. But to me, it seems like the kernel version check is not adding much value here, as without the check as well, the compilation should fail if the user is not using the right kernel version.

Another reason is that, though event groups are added with kernel version v8.1, task notifications (which were added with v8.2) could be used in the network interface or with ipconfigSELECT_USES_NOTIFY. For users who do not use task notification, it doesnt make much sense to have minimum version as v8.2

[HTRamsey]

@tony-josi-aws to your first point, the value would be in making it obvious to users why it is not compiling, instead of getting more questions on the forum. To the second point, the check is currently for v8.1, not v8.2.