utzig commented on pull request #2323: URL: https://github.com/apache/mynewt-core/pull/2323#issuecomment-649154779
> @utzig No I do not think that is the case. Well, this was part of my issue with how things were arranged. The file lwipopts.h was placed in net/ip/include. However, you may not want this particular version of lwipopts.h. That particular version is now in /net/ip/lwip_mn/include/lwipopts.h. Well, then you still have to remove the `net/ip` dependency inside `net/ip/lwip_base`, no? > > One issue I was having was that when I was trying to include a different package with its own lwipopts.h, the one from net/ip/include was being included as opposed to the one I wanted. > > One way this does work is to include net/ip/lwip_mn to the stm32f7discovery target but that seems a bit odd. I don't think you should be changing BSPs. I believe you can add an extra dependency to `net/ip/lwip_mn` here: https://github.com/apache/mynewt-core/blob/master/hw/drivers/lwip/stm32_eth/pkg.yml#L30 ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
