recipe: scripts: set INCLUDE and LIB on Windows

[igorsafo]

Checklist

• [x] Used a personal fork of the feedstock to propose changes
• [x] Bumped the build number (if the version is unchanged)
• [ ] Reset the build number to 0 (if the version changed)
• [ ] Re-rendered with the latest conda-smithy (Use the phrase code>@<space/conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
• [ ] Ensured the license file is being packaged.

In the current version of the compiler package on Windows when user activates an environment with dpcpp compiler and tries to build an application or a library the following issues happen:

• cmake can't find the libraries ('libircmt.lib') needed for the compiler so the compiler detection fails:

(dpcpp) C:\Users\igorsafo\Workspace\libraries.performance.math.onednn-master\build>cmake -DDNNL_CPU_RUNTIME=DPCPP   -DDNNL_GPU_RUNTIME=DPCPP -GNinja ..
-- The CXX compiler identification is IntelLLVM 2023.2.0 with MSVC-like command-line
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - failed
-- Check for working CXX compiler: C:/Users/igorsafo/AppData/Local/miniconda3/envs/dpcpp/Library/bin/icx.exe
-- Check for working CXX compiler: C:/Users/igorsafo/AppData/Local/miniconda3/envs/dpcpp/Library/bin/icx.exe - broken
CMake Error at C:/Program Files/CMake/share/cmake-3.23/Modules/CMakeTestCXXCompiler.cmake:62 (message):
  The C++ compiler

    "C:/Users/igorsafo/AppData/Local/miniconda3/envs/dpcpp/Library/bin/icx.exe"

  is not able to compile a simple test program.

  It fails with the following output:

    Change Dir: C:/Users/igorsafo/Workspace/libraries.performance.math.onednn-master/build/CMakeFiles/CMakeTmp

    Run Build Command(s):C:/Users/igorsafo/AppData/Local/miniconda3/envs/dpcpp/Library/bin/ninja.exe cmTC_56ae9 && [1/2] Building CXX object CMakeFiles\cmTC_56ae9.dir\testCXXCompiler.cxx.obj
    [2/2] Linking CXX executable cmTC_56ae9.exe
    FAILED: cmTC_56ae9.exe
    cmd.exe /C "cd . && "C:\Program Files\CMake\bin\cmake.exe" -E vs_link_exe --intdir=CMakeFiles\cmTC_56ae9.dir --rc=C:\PROGRA~2\WI3CF2~1\10\bin\100190~1.0\x86\rc.exe --mt=C:\PROGRA~2\WI3CF2~1\10\bin\100190~1.0\x86\mt.exe --manifests  -- C:\Users\igorsafo\AppData\Local\miniconda3\envs\dpcpp\Library\bin\icx.exe /nologo CMakeFiles\cmTC_56ae9.dir\testCXXCompiler.cxx.obj  /link /out:cmTC_56ae9.exe /implib:cmTC_56ae9.lib /pdb:cmTC_56ae9.pdb /version:0.0 /machine:x64  /debug /INCREMENTAL /subsystem:console  kernel32.lib user32.lib gdi32.lib winspool.lib shell32.lib ole32.lib oleaut32.lib uuid.lib comdlg32.lib advapi32.lib && cd ."
    LINK Pass 1: command "C:\Users\igorsafo\AppData\Local\miniconda3\envs\dpcpp\Library\bin\icx.exe /nologo CMakeFiles\cmTC_56ae9.dir\testCXXCompiler.cxx.obj /link /out:cmTC_56ae9.exe /implib:cmTC_56ae9.lib /pdb:cmTC_56ae9.pdb /version:0.0 /machine:x64 /debug /INCREMENTAL /subsystem:console kernel32.lib user32.lib gdi32.lib winspool.lib shell32.lib ole32.lib oleaut32.lib uuid.lib comdlg32.lib advapi32.lib /MANIFEST /MANIFESTFILE:CMakeFiles\cmTC_56ae9.dir/intermediate.manifest CMakeFiles\cmTC_56ae9.dir/manifest.res" failed (exit code 1104) with the following output:
    LINK : fatal error LNK1104: cannot open file 'libircmt.lib'
    icx: error: linker command failed with exit code 1104 (use -v to see invocation)
    ninja: build stopped: subcommand failed.

• Once LIB is extended with %CONDA_PREFIX%\Library\lib the compiler detection passes but the building process fails due to missing headers:

  (dpcpp) C:\Users\igorsafo\Workspace\libraries.performance.math.onednn-master\build>cmake --build .
  [1/1573] Building CXX object src\common\CMakeFiles\dnnl_common.dir\concat.cpp.obj
  FAILED: src/common/CMakeFiles/dnnl_common.dir/concat.cpp.obj
  C:\Users\igorsafo\AppData\Local\miniconda3\envs\dpcpp\Library\bin\icx.exe  /nologo /TP -DCL_TARGET_OPENCL_VERSION=300 -DDNNL_DLL -DDNNL_DLL_EXPORTS -DDNNL_ENABLE_CPU_ISA_HINTS -DDNNL_ENABLE_ITT_TASKS -DDNNL_ENABLE_MAX_CPU_ISA -DDNNL_X64=1 -DNOMINMAX -DTBB_PREVIEW_TASK_ARENA_CONSTRAINTS_EXTENSION=1 -DWIN32_LEAN_AND_MEAN -D_WIN -D__STDC_CONSTANT_MACROS -D__STDC_LIMIT_MACROS -I"C:\Program Files (x86)\intel\oneAPI\tbb\2021.10.0\include" -IC:\Users\igorsafo\Workspace\libraries.performance.math.onednn-master\include -IC:\Users\igorsafo\Workspace\libraries.performance.math.onednn-master\build\include -IC:\Users\igorsafo\Workspace\libraries.performance.math.onednn-master\src /DWIN32 /D_WINDOWS /W3 /GR /EHsc -fno-sycl-id-queries-fit-in-int -fsycl -Qiopenmp  -Wno-ignored-attributes /fp:precise -Wno-unknown-warning-option  -std=c++11   -Wno-pass-failed -Wno-tautological-compare -Wno-pass-failed -Wno-recommended-option  -DITT_API_IPT_SUPPORT /MD /O2 /Ob2 /DNDEBUG -QMD -QMT src\common\CMakeFiles\dnnl_common.dir\concat.cpp.obj -QMF src\common\CMakeFiles\dnnl_common.dir\concat.cpp.obj.d /Fosrc\common\CMakeFiles\dnnl_common.dir\concat.cpp.obj /Fdsrc\common\CMakeFiles\dnnl_common.dir\ -c C:\Users\igorsafo\Workspace\libraries.performance.math.onednn-master\src\common\concat.cpp
  icx: warning: unknown argument ignored in clang-cl '-std=c++11'; did you mean '-Qstd=c++11'? [-Wunknown-argument]
  In file included from C:\Users\igorsafo\Workspace\libraries.performance.math.onednn-master\src\common\concat.cpp:23:
  In file included from C:\Users\igorsafo\Workspace\libraries.performance.math.onednn-master\src\common/engine.hpp:34:
  In file included from C:\Users\igorsafo\Workspace\libraries.performance.math.onednn-master\src\graph/interface/allocator.hpp:30:
  In file included from C:\Users\igorsafo\Workspace\libraries.performance.math.onednn-master\src\graph/utils/allocator.hpp:23:
  In file included from C:\Users\igorsafo\Workspace\libraries.performance.math.onednn-master\src\graph/utils/sycl_check.hpp:24:
  C:\Users\igorsafo\AppData\Local\miniconda3\envs\dpcpp\Library\lib\clang\17\include\CL/sycl.hpp(11,10): fatal error: 'sycl/sycl.hpp' file not found
  #include <sycl/sycl.hpp>

  The same issue happens if user tries to build a simple application:

  
  (dpcpp) C:\Users\igorsafo\Workspace>icx -fsycl simple.cpp
  Intel(R) oneAPI DPC++/C++ Compiler for applications running on Intel(R) 64, Version 2023.2.0 Build 20230627
  Copyright (C) 1985-2023 Intel Corporation. All rights reserved.

In file included from simple.cpp:1: C:\Users\igorsafo\AppData\Local\miniconda3\envs\dpcpp\Library\lib\clang\17\include\CL/sycl.hpp(11,10): fatal error: 'sycl/sycl.hpp' file not found

include <sycl/sycl.hpp>

     ^~~~~~~~~~~~~~~

1 error generated.

Extending `INCLUDE` with `%CONDA_PREFIX%\include` fixes the second issue.

This change makes the activation kind of similar to `setvars.bat` distributed with compiler as part of oneAPI package. It makes the complier ready to be used out of box. If we keep activation as is:
- Each application that uses the compiler should set environment which might change once the compiler version changes.
- The behavior is inconsistent with other compilers, usually once you setup a compiler it sets everything and there is no need to do anything manually.

An alternative solution might be a `setvars.sh` & `setvars.bat` that will be distributed with the compiler package so any user can call it to setup the environment.

This should fix the issues in https://github.com/conda-forge/onednn-feedstock/pull/53 on Windows

+@isuruf

[conda-forge-webservices[bot]]

Hi! This is the friendly automated conda-forge-linting service.

I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.

Here's what I've got...

For recipe:

• When defining a source/url please add a sha256, sha1 or md5 checksum (sha256 preferably).
• When defining a source/url please add a sha256, sha1 or md5 checksum (sha256 preferably).
• When defining a source/url please add a sha256, sha1 or md5 checksum (sha256 preferably).
• When defining a source/url please add a sha256, sha1 or md5 checksum (sha256 preferably).
• When defining a source/url please add a sha256, sha1 or md5 checksum (sha256 preferably).
• When defining a source/url please add a sha256, sha1 or md5 checksum (sha256 preferably).
• When defining a source/url please add a sha256, sha1 or md5 checksum (sha256 preferably).
• When defining a source/url please add a sha256, sha1 or md5 checksum (sha256 preferably).
• pin_compatible should be used instead of pin_subpackage for dpcpp_impl_win-64 because it is not a known output of this recipe: ['intel-cmplr-lic-rt', 'intel-cmplr-lib-rt', 'intel-fortran-rt', 'intel-opencl-rt', 'dpcpp-cpp-rt', 'dpcpp_impl_linux-64', 'dpcpp_linux-64', 'oneccl-devel', 'intel-compiler-repack'].

[igorsafo]

@conda-forge-admin, please rerender

[oleksandr-pavlyk]

@igorsafo These changes in activation script just work around a problem that should really be fixed in intel::dpcpp_win-64.

We have to do this in dpctl's build script: https://github.com/IntelPython/dpctl/blob/master/conda-recipe/bld.bat#L3-L4

The fix for this is that %PREFIX%\include% folder in intel::dpcpp_impl_win-64 should really have been placed in %PREFIX%\Library\include, ref: https://github.com/conda-forge/intel-compiler-repack-feedstock/pull/4#discussion_r960084632

Something along this lines would resolve the need to set LIB variable, e.g. %PREFIX%\compiler\lib be installed into %PREFIX%\Library\lib, but I am less sure about this.

Such layout reshuffling could be done during repackaging by conda-forge, for example.

[igorsafo]

@igorsafo These changes in activation script just work around a problem that should really be fixed in intel::dpcpp_win-64.

We have to do this in dpctl's build script: https://github.com/IntelPython/dpctl/blob/master/conda-recipe/bld.bat#L3-L4

The fix for this is that %PREFIX%\include% folder in intel::dpcpp_impl_win-64 should really have been placed in %PREFIX%\Library\include, ref: #4 (comment)

Something along this lines would resolve the need to set LIB variable, e.g. %PREFIX%\compiler\lib be installed into %PREFIX%\Library\lib, but I am less sure about this.

Such layout reshuffling could be done during repackaging by conda-forge, for example.

Thanks for the explanation. Will you be able to fix the issue on intel::dpcpp_win-64 side?