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/12/13 04:41:24 UTC

[GitHub] [nuttx] xiaoxiang781216 opened a new pull request, #7861: Continue fix the macOS ci issue

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

   ## Summary
   
   - Revert "Fix ccache: error: Could not find compiler "x86_64-elf-gcc" in PATH"
   - cibuild.sh: Fix "Untracked files: tools/ci/bloaty-src/" 
   - cibuild.sh: Install ccache in the last step 
   
   ## Impact
   
   ## Testing
   Pass CI
   


-- 
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] [nuttx] xiaoxiang781216 commented on pull request #7861: Continue fix the macOS ci issue

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

   > I don't understand the bloaty change? 
   
   The key change is clone bloaty into tools folder instead nuttx/tools/ci folder.
   
   >Why did we need to keep the source repo around instead of just deleting it.
   
   It's fine to remove source repo. I remove the deletion just keep consistent with nuttx-tools because we don't remove it after finish build:
   https://github.com/apache/nuttx/pull/7861/files#diff-5739bde660f59b7d7d70b27e67bbf8062932aa21821da1883c7733b738575c88R240-R253


-- 
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] [nuttx] xiaoxiang781216 commented on pull request #7861: Continue fix the macOS ci issue

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

   it's strange why this pr pass ci, but other following patch fail.


-- 
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] [nuttx] xiaoxiang781216 commented on pull request #7861: Continue fix the macOS ci issue

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

   @btashton restore the deletion here: https://github.com/apache/nuttx/pull/7864


-- 
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] [nuttx] btashton commented on pull request #7861: Continue fix the macOS ci issue

Posted by GitBox <gi...@apache.org>.
btashton commented on PR #7861:
URL: https://github.com/apache/nuttx/pull/7861#issuecomment-1347823074

   I don't understand the bloaty change? Why did we need to keep the source repo around instead of just deleting 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] [nuttx] masayuki2009 merged pull request #7861: Continue fix the macOS ci issue

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


-- 
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