gromero commented on PR #13603: URL: https://github.com/apache/tvm/pull/13603#issuecomment-1348554385
@mehrdadh I wonder if it was really the root cause of `st/l4/stm32l4r5z(g-i)tx-pinctrl.dtsi` not being found in your case. Have you tried to narrow down why `gcc` command supposed to take care of these includes (in `cmake/dts.cmake:167`) was failing to find this include file in your env. before refreshing your build environment? I'm not saying I'm against this patch or against such a pragmatic approach for fixing errors like that, it's just that I could not reproduce your issue even using exactly the same `poetry.lock` you're updating -- which suggests it's not the root cause of the error. Also the patch is updating several other packages which I have not idea how they could be remotely related to the issue in question. Hence, since I can't reproduce it and I can't see how the updated packages could be related to the issue, I can't really review it too. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
