You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by "Donny9 (via GitHub)" <gi...@apache.org> on 2023/08/21 12:54:09 UTC
[GitHub] [nuttx] Donny9 opened a new pull request, #10336: libc/gpsutils: merged into one target to fix issue of parallel compile
Donny9 opened a new pull request, #10336:
URL: https://github.com/apache/nuttx/pull/10336
## Summary
libc/gpsutils: merged into one target to fix the issue of parallel compilation
Because multiple dependencies behind the context are compiled in parallel, if they have dependencies on each other, it will cause compilation errors.
This PR is realted to https://github.com/apache/nuttx-apps/pull/1867
## Impact
## Testing
./tools/configure.sh lilygo_tbeam_lora_gps/gps && make -j8
--
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] Donny9 commented on pull request #10336: libc/gpsutils: merged into one target to fix issue of parallel compile
Posted by "Donny9 (via GitHub)" <gi...@apache.org>.
Donny9 commented on PR #10336:
URL: https://github.com/apache/nuttx/pull/10336#issuecomment-1686351683
> @Donny9 @xiaoxiang781216 I think I missed the gpsutils been moved to nuttx/libs/libc/. Question: why do want to move gpsutils to kernel instead of keeping it on userspace? Is it related to GPS becoming part of the sensor subsystem?
@acassis yes, it is replated to GPS driver(https://github.com/apache/nuttx/pull/9954). We put the parsing library under libc, so that both kernel and userspace can call it, and the header file path remains unchanged.
--
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 merged pull request #10336: libc/gpsutils: merged into one target to fix issue of parallel compile
Posted by "xiaoxiang781216 (via GitHub)" <gi...@apache.org>.
xiaoxiang781216 merged PR #10336:
URL: https://github.com/apache/nuttx/pull/10336
--
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] acassis commented on pull request #10336: libc/gpsutils: merged into one target to fix issue of parallel compile
Posted by "acassis (via GitHub)" <gi...@apache.org>.
acassis commented on PR #10336:
URL: https://github.com/apache/nuttx/pull/10336#issuecomment-1686313746
@Donny9 @xiaoxiang781216 I think I missed the gpsutils been moved to nuttx/libs/libc/. Question: why do want to move gpsutils to kernel instead of keeping it on userspace? Is it related to GPS becoming part of the sensor subsystem?
--
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