You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2022/05/10 15:16:09 UTC

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request, #6230: lm3s6965-ek/qemu-kostest: Fix warning: selects NETDEVICES which has unmet direct dependencies (NET)

xiaoxiang781216 opened a new pull request, #6230:
URL: https://github.com/apache/incubator-nuttx/pull/6230

   ## Summary
   
   ## Impact
   
   ## Testing
   
   


-- 
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: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6230: Fix ci warning

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on PR #6230:
URL: https://github.com/apache/incubator-nuttx/pull/6230#issuecomment-1123175866

   > @xiaoxiang781216
   > 
   > I found the root cause of lm3s6965-ek:qemu-kostest. I will create a new PR later, so please remove the commit for it.
   
   I can add the dependence back if you want, I just think that it isn't necessary to enable ethernet in kotest.


-- 
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: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #6230: Fix ci warning

Posted by GitBox <gi...@apache.org>.
masayuki2009 commented on PR #6230:
URL: https://github.com/apache/incubator-nuttx/pull/6230#issuecomment-1123206007

   >@masayuki2009 please review again.
   
   I think the qemu-ostest/defconfig is still incorrect.
   
   I created a new PR instead of the commit.
   https://github.com/apache/incubator-nuttx/pull/6233
   


-- 
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: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #6230: Fix ci warning

Posted by GitBox <gi...@apache.org>.
masayuki2009 commented on PR #6230:
URL: https://github.com/apache/incubator-nuttx/pull/6230#issuecomment-1123009056

   https://github.com/apache/incubator-nuttx/pull/5597
   


-- 
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: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #6230: Fix ci warning

Posted by GitBox <gi...@apache.org>.
masayuki2009 commented on PR #6230:
URL: https://github.com/apache/incubator-nuttx/pull/6230#issuecomment-1123161631

   @xiaoxiang781216 
   
   I found the root cause of lm3s6965-ek:qemu-kostest.
   I will create a new PR later, so please remove the commit for it.
   


-- 
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: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6230: Fix ci warning

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on PR #6230:
URL: https://github.com/apache/incubator-nuttx/pull/6230#issuecomment-1123264838

   > > @masayuki2009 please review again.
   > 
   > I think the qemu-ostest/defconfig is still incorrect.
   > 
   > I created a new PR instead of the commit. #6233
   
   The change look better, I have dropped the related change from this PR.


-- 
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: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #6230: Fix ci warning

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on PR #6230:
URL: https://github.com/apache/incubator-nuttx/pull/6230#issuecomment-1123178240

   @masayuki2009 please review again.


-- 
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: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-nuttx] masayuki2009 merged pull request #6230: Fix ci warning

Posted by GitBox <gi...@apache.org>.
masayuki2009 merged PR #6230:
URL: https://github.com/apache/incubator-nuttx/pull/6230


-- 
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: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org