meta-atmel icon indicating copy to clipboard operation
meta-atmel copied to clipboard

Bring changes to build with clang15 and master branch of oe-core

Open kraj opened this issue 3 years ago • 7 comments

kraj avatar Sep 29 '22 23:09 kraj

Hi,

Concerning EGT, is this move to c++17 mandatory? There is a fallback if the experimental file system is not detected. Maybe this fallback is not working as expected... We are considering making this move but we don't want to do it in a rush.

Regards

ldesroches avatar Oct 06 '22 07:10 ldesroches

Hi,

Concerning EGT, is this move to c++17 mandatory? There is a fallback if the experimental file system is not detected. Maybe this fallback is not working as expected... We are considering making this move but we don't want to do it in a rush.

This was a path to fix build with clang 15+ where libc++ has started to drop older features, if we can fix it in other ways thats ok too.

Regards

kraj avatar Oct 06 '22 14:10 kraj

Can you cherry-pick the layer compat patches ? they are now needed

kraj avatar Oct 11 '22 15:10 kraj

@Dharma-B , @gehariprasath : some patches to queue for the dunfell branch before that we move "master" to kirkstone, probably

noglitch avatar Oct 11 '22 15:10 noglitch

Can you cherry-pick the layer compat patches ? they are now needed

We will apply the same and the other patch for cryptoauthlib after review.

gehariprasath avatar Oct 12 '22 09:10 gehariprasath

@kraj The cryptoauthlib has the manual document which is part of the repository, hence we cannot skip it. Installing git-lfs in the host machine is an easier option.

gehariprasath avatar Oct 13 '22 06:10 gehariprasath

@kraj The cryptoauthlib has the manual document which is part of the repository, hence we cannot skip it. Installing git-lfs in the host machine is an easier option.

OK, I have dropped the concerned patch from pull here.

kraj avatar Oct 13 '22 18:10 kraj

I think this one can be closed as well.

noglitch avatar Jan 05 '23 17:01 noglitch

@noglitch I still have egt patches but I will open a new PR for that

kraj avatar Jan 05 '23 19:01 kraj