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

Fix incorrect device naming #1

Merged
merged 2 commits into from Jul 12, 2022
Merged

Fix incorrect device naming #1

merged 2 commits into from Jul 12, 2022

Conversation

IgorEisberg
Copy link
Contributor

ro.product..device values "haydnpro" and "haydn_in" don't exist and never existed, those are only applicable for ro.product..name.
There are only "haydn" and "haydnin" for ro.product.*.device.
This mistake prevents flashing ROMs that check for the ro.product.device.

sekaiacg and others added 2 commits July 10, 2022 12:52
Signed-off-by: sekaiacg <sekaiacg@gmail.com>
ro.product.*.device values "haydnpro" and "haydn_in" don't exist and never existed, those are only applicable for ro.product.*.name.
There are only "haydn" and "haydnin" for ro.product.*.device.
This mistake prevents flashing ROMs that check for the ro.product.device.
@sekaiacg sekaiacg self-requested a review July 12, 2022 04:26
@sekaiacg sekaiacg merged commit d58312f into sekaiacg:android-12.1 Jul 12, 2022
@ingbrzy
Copy link

ingbrzy commented Jul 13, 2022

@sekaiacg can you build a new haydn TWRP with this fix included, please? we want to test it with our xiaomi.eu ROM.. Thank you

@sekaiacg
Copy link
Owner

@sekaiacg can you build a new haydn TWRP with this fix included, please? we want to test it with our xiaomi.eu ROM.. Thank you

https://dl.akr-developers.com/?dir=skkk/TWRP/A12/v3.8.3_A12

@sekaiacg sekaiacg requested review from sekaiacg and removed request for sekaiacg July 13, 2022 05:19
@ingbrzy
Copy link

ingbrzy commented Jul 13, 2022

Confirmed, haydn ROM now can be installed successfully..
Thanks @IgorEisberg for fix and @sekaiacg for TWRP update!
Good job!

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.

None yet

3 participants