[GitHub] [incubator-nuttx] xiaoxiang781216 edited a comment on issue #343: Revert "README.txt: add MacOS section for flock installation"

[GitBox <gi...@apache.org>]

xiaoxiang781216 edited a comment on issue #343: Revert "README.txt: add MacOS section for flock installation"
URL: https://github.com/apache/incubator-nuttx/pull/343#issuecomment-589494845
 
 
   > @xiaoxiang781216 which arch does it break?
   > using shlock on macOS is not the main point of [apache/incubator-nuttx-apps#81](https://github.com/apache/incubator-nuttx-apps/pull/81) .
   > we can't assume bash, can we? i don't think it's worth to write .bat file for this.
   
   Here:
   https://github.com/FishsemiCode/nuttx/blob/song-u1/arch/ceva/src/song/Toolchain.defs
   And you can see we don't define AR at all.
   By desgin, the interface between the common Makefile and toolchain is ARCHIVE/COMIPLE... and the comment said that the user can overwrite these macros as need. AR/CC isn't public interface.
   If we take your approach: we should use AR/CC.. directly and then ARCHIVE/COMPILE should be removed to avoid the confusion.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services