search

RDunkley / meta-dotnet-core

BitBake recipes to use the pre-built binaries provided by Microsoft for .Net Core and their remote debugger.
MIT License
34 stars 28 forks source link

Thud branch mixes old and new yocto syntax #36

Closed vovanp closed 1 year ago

vovanp commented 2 years ago

After latest commit in the thud branch, commit efcf20454fbf85055a0bf37aeb52aa7a748dc536 some files have new yocto syntax and therefor could not be compiled with the Thud branch.
log: ERROR: ParseError at /meta-dotnet-core/recipes-devtools/vsdbg/vsdbg_17.0.10712.2.inc:12: unparsed line: 'RDEPENDS:${PN} += " procps"'

RDunkley commented 2 years ago

I fixed this on the master branch, but forgot to push to Thud. In the meantime you should be able to use master.

On Tue, Mar 8, 2022, 1:35 AM vovanp @.***> wrote:

After latest commit in the thud branch, commit efcf204 https://github.com/RDunkley/meta-dotnet-core/commit/efcf20454fbf85055a0bf37aeb52aa7a748dc536 some files have new yocto syntax and therefor could not be compiled with the Thud branch. log: ERROR: ParseError at /meta-dotnet-core/recipes-devtools/vsdbg/vsdbg_17.0.10712.2.inc:12: unparsed line: 'RDEPENDS:${PN} += " procps"'

— Reply to this email directly, view it on GitHub https://github.com/RDunkley/meta-dotnet-core/issues/36, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB5SOIOS2JAFL2SN6DXIO5LU64GNHANCNFSM5QFU55QA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you are subscribed to this thread.Message ID: @.***>

vovanp commented 2 years ago

But how can I use Thud branch directly. It will be not possible to use master branch directly, because in the next yocto release the syntax will change.

ghost commented 2 years ago

I am targeting dunfell and was facing the exact same error. Using master fixed it, or at least made parsing successful. However, it would be great to have it on the release branch.

ghost commented 2 years ago

I am targeting dunfell and was facing the exact same error. Using master fixed it, or at least made parsing successful. However, it would be great to have it on the release branch.

Using the poky with version 3.1.14 fixed my issue.

RDunkley commented 2 years ago

I ended up keeping Master with the old syntax. Honister is the first release with the new syntax so I decided to keep master as the old and create a new branch for Honister.

So master should work right now for you until I can update thud. Moving forward I plan on keeping Master the old syntax until a few more yocto versions come out (since most users won't be on Honister yet).

On Tue, Mar 8, 2022, 6:47 AM Vladimir Pustovalov @.***> wrote:

But how can I use Thud branch directly. It will be not possible to use master branch directly, because in the next yocto release the syntax will change.

— Reply to this email directly, view it on GitHub https://github.com/RDunkley/meta-dotnet-core/issues/36#issuecomment-1061794120, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB5SOIOSZDLYIKQHK72CFI3U65K5VANCNFSM5QFU55QA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you commented.Message ID: @.***>

RDunkley commented 1 year ago

It was way past time to move forward on post-Gatesgarth syntax. The main branch has been updated to Honister and above. If you need the old syntax, then use the pre-honister branch.