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 2020/10/14 03:58:28 UTC
[GitHub] [incubator-nuttx] RB-tel opened a new pull request #1987: [BACKPORT-1940]cygwin build - path issue fix for Renesas Arch
RB-tel opened a new pull request #1987:
URL: https://github.com/apache/incubator-nuttx/pull/1987
## Summary
BACKPORT to Release 10.0
## Impact
NA
## Testing
NA
----------------------------------------------------------------
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
[GitHub] [incubator-nuttx] btashton closed pull request #1987: [BACKPORT-1940]cygwin build - path issue fix for Renesas Arch
Posted by GitBox <gi...@apache.org>.
btashton closed pull request #1987:
URL: https://github.com/apache/incubator-nuttx/pull/1987
----------------------------------------------------------------
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
[GitHub] [incubator-nuttx] Ouss4 edited a comment on pull request #1987: [BACKPORT-1940]cygwin build - path issue fix for Renesas Arch
Posted by GitBox <gi...@apache.org>.
Ouss4 edited a comment on pull request #1987:
URL: https://github.com/apache/incubator-nuttx/pull/1987#issuecomment-710261544
Oops, I just noticed that there are more than one commit. Was this branch based on master?
Please reset your branch to releases/10.0 and just cherry pick the single commit into the branch.
----------------------------------------------------------------
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
[GitHub] [incubator-nuttx] btashton commented on pull request #1987: [BACKPORT-1940]cygwin build - path issue fix for Renesas Arch
Posted by GitBox <gi...@apache.org>.
btashton commented on pull request #1987:
URL: https://github.com/apache/incubator-nuttx/pull/1987#issuecomment-710274533
Btw the way I have found works best for backporting PRs is to add `.patch` to the PR url and use curl to pipe that to `git apply`. Keeps extra things from sneaking in and preserves the individual commits if applicable.
----------------------------------------------------------------
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
[GitHub] [incubator-nuttx] Ouss4 commented on pull request #1987: [BACKPORT-1940]cygwin build - path issue fix for Renesas Arch
Posted by GitBox <gi...@apache.org>.
Ouss4 commented on pull request #1987:
URL: https://github.com/apache/incubator-nuttx/pull/1987#issuecomment-710261544
Oops, I just noticed that there are more than one commit. Was this branch based on master?
Please just cherry pick the single commit into the branch.
----------------------------------------------------------------
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
[GitHub] [incubator-nuttx] Ouss4 commented on pull request #1987: [BACKPORT-1940]cygwin build - path issue fix for Renesas Arch
Posted by GitBox <gi...@apache.org>.
Ouss4 commented on pull request #1987:
URL: https://github.com/apache/incubator-nuttx/pull/1987#issuecomment-710259562
The base of this PR was master, I changed it to the release branch. Waiting for the checks to merge...
----------------------------------------------------------------
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
[GitHub] [incubator-nuttx] btashton edited a comment on pull request #1987: [BACKPORT-1940]cygwin build - path issue fix for Renesas Arch
Posted by GitBox <gi...@apache.org>.
btashton edited a comment on pull request #1987:
URL: https://github.com/apache/incubator-nuttx/pull/1987#issuecomment-710274533
Btw the way I have found works best for backporting PRs is to add `.patch` to the PR url and use curl to pipe that to `git am`. Keeps extra things from sneaking in and preserves the individual commits if applicable.
```
curl -L https://github.com/apache/incubator-nuttx/pull/1940.patch | git am
```
----------------------------------------------------------------
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