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 2021/09/12 13:23:52 UTC

[GitHub] [incubator-nuttx] acassis opened a new pull request #4523: boards: Remove -nostartfiles -nodefaultlibs from LDFLAGS

acassis opened a new pull request #4523:
URL: https://github.com/apache/incubator-nuttx/pull/4523


   ## Summary
   boards: Remove -nostartfiles -nodefaultlibs from LDFLAGS
   ## Impact
   Fix all boards that are failing, see: https://github.com/apache/incubator-nuttx/issues/4510
   ## Testing
   Compiling
   


-- 
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] yamt commented on pull request #4523: boards: Remove -nostartfiles -nodefaultlibs from LDFLAGS

Posted by GitBox <gi...@apache.org>.
yamt commented on pull request #4523:
URL: https://github.com/apache/incubator-nuttx/pull/4523#issuecomment-917732414


   > I noticed that removing "-nostartfiles -nodefaultlibs" increased about 20 bytes in the final binaries, but I didn't test all the board.
   
   which board is 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 merged pull request #4523: boards: Remove -nostartfiles -nodefaultlibs from LDFLAGS

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 merged pull request #4523:
URL: https://github.com/apache/incubator-nuttx/pull/4523


   


-- 
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] hartmannathan commented on pull request #4523: boards: Remove -nostartfiles -nodefaultlibs from LDFLAGS

Posted by GitBox <gi...@apache.org>.
hartmannathan commented on pull request #4523:
URL: https://github.com/apache/incubator-nuttx/pull/4523#issuecomment-918303601


   Looks like I didn't get to it in time. I'll try to test the changes soon and I'll let you know...


-- 
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] acassis commented on pull request #4523: boards: Remove -nostartfiles -nodefaultlibs from LDFLAGS

Posted by GitBox <gi...@apache.org>.
acassis commented on pull request #4523:
URL: https://github.com/apache/incubator-nuttx/pull/4523#issuecomment-917635978


   I noticed that removing "-nostartfiles -nodefaultlibs" increased about 20 bytes in the final binaries, but I didn't test all the board.
   @davids5 are you with "bloatware" tool in place to detect any issue? I recall you said was planning to integrated it on our CI, we really need 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] hartmannathan commented on pull request #4523: boards: Remove -nostartfiles -nodefaultlibs from LDFLAGS

Posted by GitBox <gi...@apache.org>.
hartmannathan commented on pull request #4523:
URL: https://github.com/apache/incubator-nuttx/pull/4523#issuecomment-918306367


   Are we sure that -nostartfiles -nodefaultlibs are unnecessary?
   
   The whole issue started from an incompatible change in (IIRC) binutils. Would it be better to detect the version of ld using a script in tools/ and output the correct flags that it will recognize?


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