Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update CMakeLists.txt #270

Closed
wants to merge 1 commit into from

Conversation

floroeske
Copy link

Change Gem::AtomLyIntegration_CommonFeatures.Editor.Static to Gem::CommonFeaturesAtom.Editor.Static to match latest o3de-extras o3de/o3de-extras@bbb0da7.

Change Gem::AtomLyIntegration_CommonFeatures.Editor.Static to Gem::CommonFeaturesAtom.Editor.Static to match latest o3de-extras o3de/o3de-extras@bbb0da7.

Signed-off-by: floroeske <floroeske@protonmail.ch>
Copy link
Contributor

@jhanca-robotecai jhanca-robotecai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your contribution, but we prefer to keep the demo in sync with official o3de releases. As indicated in the readme file, this demo is targeting main branches of both o3de and o3de-extras. This PR should be closed until the next release in February 2024.

Edit: The decision about keeping all demos in sync with main branches is justified by the fact that, currently, we do not have the resources to keep them up to date with o3de development. Apologies for the confusing branch naming, we will keep it in mind and fix it after the upcoming point release.

@floroeske
Copy link
Author

No problem. Thanks for clearing that up.

It wasn't clear to me which branches of o3de, o3de-extras and ROSConDemo are supposed to work together. The ROSConDemo readme on the main branch states, use the development branch (https://github.com/o3de/ROSConDemo/blob/main/README.md?plain=1#L58). I guess that created a bit of confusion. So I just used development everywhere.

I'll try again with o3de (main), o3de-extras(main) and ROSConDemo(development).

@floroeske floroeske closed this Dec 7, 2023
@jhanca-robotecai
Copy link
Contributor

Thank you for pointing that out, it is a mistake on our side. Please use the development branch of this demo, the main branch is one year old and, most probably, will not work correctly. I will fix this after o3de point release (next week).

@jhanca-robotecai
Copy link
Contributor

One more comment: it should be possible to run this demo from the docker files. Those were checked recently.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants