adwait-godbole commented on PR #18299:
URL: https://github.com/apache/nuttx/pull/18299#issuecomment-3832530140

   Hi @acassis. Thanks for the quick feedback. From the screenshots I’ve 
attached in the PR description, I’ve already been able to achieve this:
   
   before:
   
   ```bash
   CONFIG_APPS_DIR=\../apps\
   ```
   
   after:
   
   ```bash
   “CONFIG_APPS_DIR=“../apps””
   ```
   
   So removing those wrapping double quotes won’t be much of an issue.
   
   Just wanted to confirm one thing. Do we only want to do this (removing 
wrapping double quotes) for `CONFIG_*` options or for everything that is 
getting dumped on the terminal? I mean do we also want this:
   
   My current PR changes:
   
   ```bash
   NuttX LDFLAGS:
      “-Wl—gc-sections”
   ```
   
   Your Requirement:
   
   ```bash
   NuttX LDFLAGS:
      -Wl—gc-sections
   ```
   
   ? Let me 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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to