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

Add back SbMemoryGetStackBounds for EG AOSP #3434

Merged
merged 1 commit into from
Jun 5, 2024

Conversation

yuying-y
Copy link
Contributor

@yuying-y yuying-y commented Jun 3, 2024

This API is needed for running EG AOSP for SB < 15

b/341636739

Change-Id: Ied03b5574d48d9380fd1e7b067cb8c5f02fb6a37

@yuying-y yuying-y requested a review from y4vor June 3, 2024 23:56
Copy link
Contributor

@y4vor y4vor left a comment

Choose a reason for hiding this comment

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

This API was deprecated and remove in C24, SB15.

This API is needed for running EG AOSP for SB < 15

b/341636739

Change-Id: Ied03b5574d48d9380fd1e7b067cb8c5f02fb6a37
@yuying-y
Copy link
Contributor Author

yuying-y commented Jun 4, 2024

This API was deprecated and remove in C24, SB15.

Yes, but it should work with SB14, C24. See details in b/341636739. It should be kept in the BUILD.gn similar to https://source.corp.google.com/h/lbshell-internal/cobalt_src/+/COBALT:starboard/linux/shared/BUILD.gn;l=136

@yuying-y yuying-y enabled auto-merge (squash) June 4, 2024 23:05
@yuying-y yuying-y requested a review from y4vor June 4, 2024 23:06
@y4vor
Copy link
Contributor

y4vor commented Jun 5, 2024

This API was deprecated and remove in C24, SB15.

Yes, but it should work with SB14, C24. See details in b/341636739. It should be kept in the BUILD.gn similar to https://source.corp.google.com/h/lbshell-internal/cobalt_src/+/COBALT:starboard/linux/shared/BUILD.gn;l=136

C24 doesn't use SbMemoryGetStackBounds.

@yuying-y yuying-y merged commit fbf1d05 into youtube:main Jun 5, 2024
309 of 310 checks passed
@yuying-y yuying-y added the cp-25.lts.1+ Cherry Pick to the 25.lts.1+ branch label Jun 5, 2024
cobalt-github-releaser-bot pushed a commit that referenced this pull request Jun 5, 2024
This API is needed for running EG AOSP for SB < 15

b/341636739

Change-Id: Ied03b5574d48d9380fd1e7b067cb8c5f02fb6a37
(cherry picked from commit fbf1d05)
yuying-y added a commit that referenced this pull request Jun 6, 2024
This API is needed for running EG AOSP for SB < 15

b/341636739

Change-Id: Ied03b5574d48d9380fd1e7b067cb8c5f02fb6a37
(cherry picked from commit fbf1d05)
yuying-y added a commit that referenced this pull request Jun 11, 2024
This API is needed for running EG AOSP for SB < 15

b/341636739

Change-Id: Ied03b5574d48d9380fd1e7b067cb8c5f02fb6a37
(cherry picked from commit fbf1d05)
yuying-y added a commit that referenced this pull request Jun 12, 2024
)

Refer to the original PR: #3434

This API is needed for running EG AOSP for SB < 15

b/341636739

Change-Id: Ied03b5574d48d9380fd1e7b067cb8c5f02fb6a37

Co-authored-by: yuying-y <[email protected]>
@kaidokert kaidokert added the cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch label Aug 15, 2024
cobalt-github-releaser-bot pushed a commit that referenced this pull request Aug 15, 2024
This API is needed for running EG AOSP for SB < 15

b/341636739

Change-Id: Ied03b5574d48d9380fd1e7b067cb8c5f02fb6a37
(cherry picked from commit fbf1d05)
kaidokert pushed a commit that referenced this pull request Aug 15, 2024
)

Refer to the original PR: #3434

This API is needed for running EG AOSP for SB < 15

b/341636739

Change-Id: Ied03b5574d48d9380fd1e7b067cb8c5f02fb6a37

Co-authored-by: yuying-y <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch cp-25.lts.1+ Cherry Pick to the 25.lts.1+ branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants