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

[SPLIT] OrangeFox support #218

Merged
merged 4 commits into from
Aug 4, 2023

Conversation

anon1892
Copy link
Member

@anon1892 anon1892 commented Aug 1, 2023

Some modifications to the code, so OrangeFox recovery can be supported (especially useful for Xiaomi devices)

Work started in #188

It works, but I don't like the fix line 267 of tooling.py

README.md Outdated Show resolved Hide resolved
@MagicLike MagicLike mentioned this pull request Aug 1, 2023
@tsterbak
Copy link
Member

tsterbak commented Aug 2, 2023

I'm thinking about how to best proceed with these PRs.
I guess, I prepare a branch for release 0.5.0 and merge them one by one. This way I can make modifications to the code and you could test it with your devices. The only downside would be, that you would need to send a new PR to make changes. How does that sound for you?

@anon1892
Copy link
Member Author

anon1892 commented Aug 2, 2023

Yes, as you wish! Now I'm a pro at making PR, lol

@tsterbak tsterbak added this to the Release Beta-Version 0.5.0 milestone Aug 4, 2023
@tsterbak tsterbak added the enhancement New feature or request label Aug 4, 2023
@tsterbak tsterbak changed the base branch from dev to dev-0.5.0 August 4, 2023 07:07
@tsterbak tsterbak merged commit a4bf0a0 into openandroidinstaller-dev:dev-0.5.0 Aug 4, 2023
@tsterbak
Copy link
Member

tsterbak commented Aug 4, 2023

@anon1892 I have a few questions going forward:

  • Ofox recommends booting the recovery (at least for A/B-partitioned devices). I would also prefer that. What do you think? (source: can-i-run-fastboot-boot-to-boot-an-orangefox-img-file) So the process seems to be dangerous here...
  • I would prefer to expect official OrangeFox names for the recoveries (like OrangeFox-lavender-stable@R11.1_5_A12.zip), why would it be named recovery.img?

@anon1892
Copy link
Member Author

anon1892 commented Aug 4, 2023

For non a/b devices, like lavender #222, I never do that (and not asked in LineageOS installation process)
I also created a function to boot the recovery, see rosemary #225 (I follow LineageOS instructions)

Yes, we can also use the zip file, to check if it's the right OFox version. But after that, we need unzip it and only flash recovery.img file on it. It require more code. But can be a good option, because we will need to distinguish OFox with LOS recovery, which both are named recovery.img

@tsterbak
Copy link
Member

tsterbak commented Aug 4, 2023

For non a/b devices, like lavender #222, I never do that (and not asked in LineageOS installation process)
I also created a function to boot the recovery, see rosemary #225 (I follow LineageOS instructions)

I see :)

Yes, we can also use the zip file, to check if it's the right OFox version. But after that, we need unzip it and only flash recovery.img file on it. It require more code. But can be a good option, because we will need to distinguish OFox with LOS recovery, which both are named recovery.img

Ah sorry, I missed, that it is a zip file...

@anon1892 anon1892 deleted the For_New_PR branch August 12, 2023 09:18
@anon1892 anon1892 mentioned this pull request Aug 12, 2023
3 tasks
tsterbak added a commit that referenced this pull request Aug 16, 2023
Work started and tested in #188 : working

Requirements : 
- #218 for OrangeFox support (especially `fastboot_flash_recovery` and
`fastboot_reboot_recovery`)
- #220 for additional partitions support
- #221 for device specific notes (optional)
tsterbak added a commit that referenced this pull request Sep 27, 2023
Work started and tested in #188 : works fine

Requirements : 
- #218 for OrangeFox support (especially `fastboot_flash_recovery` and
`fastboot_reboot_recovery`
- #221 for device specific notes (optional)
tsterbak added a commit that referenced this pull request Sep 27, 2023
Work started in #188 

Untested but should work (same steps as #222 )

Requirements : 
- #218 for OrangeFox support (especially `fastboot_flash_recovery` and
`fastboot_reboot_recovery`
- #221 for device specific notes (optional)
tsterbak added a commit that referenced this pull request Sep 27, 2023
Work started and tested in #188 : working

Requirements : 
- #218 for OrangeFox support (especially `fastboot_flash_recovery` and
`fastboot_reboot_recovery`
- #221 for device specific notes (optional)
tsterbak added a commit that referenced this pull request Sep 27, 2023
Work started in #188 

Untested but should work

Requirements : 
- #218 for OrangeFox support (optional)
- #221 for device specific notes (optional)
tsterbak added a commit that referenced this pull request Sep 27, 2023
#228)

Related to issue #85. Not tested !

Requirements : 
- #218 for OrangeFox support (especially `fastboot_flash_recovery` and
`fastboot_reboot_recovery`
- #221 for device specific notes (optional)

Actually, the best way of testing may be using the `main` branch of [my
fork](https://github.com/anon1892/oai/tree/main), since it need 3 PR to
work as is.
tsterbak added a commit that referenced this pull request Sep 27, 2023
Related to issue #232.  Not tested !

Requirements : 
- #218 for OrangeFox support (especially `fastboot_flash_recovery` and
`fastboot_reboot_recovery`
- #221 for device specific notes (optional)

@Chronos093
tsterbak added a commit that referenced this pull request Sep 27, 2023
Untested ! Asked in #212. @alkman1982 do you want to help us by testing
it?

Requirements : 
- #218 for OrangeFox support (especially `fastboot_flash_recovery` and
`fastboot_reboot_recovery`)
- #220 for additional partitions support
- #221 for device specific notes (optional)

I used this :
https://forum.xda-developers.com/t/rom-t-unofficial-lineageos-20-0-for-redmi-k20-pro-mi-9t-pro-raphael.4464937/
@SirRGB SirRGB mentioned this pull request Nov 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants