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: Improved installer #991

Merged
merged 15 commits into from
Jul 8, 2023
Merged

FIX: Improved installer #991

merged 15 commits into from
Jul 8, 2023

Conversation

Aemiii91
Copy link
Member

@Aemiii91 Aemiii91 commented Jul 7, 2023

Changes

  • Installation for MMP no longer relies on the transfer of .tmp_update
    • Unless Onion is already installed and device is on 20230505 or later (this will run runtime.sh directly)
  • Moved the "installer check" into runtime.sh
    • Meaning that from this release and forward, only the miyoo folder needs to be transferred to update Onion
  • Fixed the notorious "black screen installation" bug - it was caused by wrongful LD_LIBRARY_PATH
  • Added core extension cache to build step (removes initial 10s load time for GLO -> Game core)
  • We now have a concept of migration scripts !!
    • These are single-run scripts across Onion updates
    • They are numbered, and the last migration ID that ran will be saved persistently
  • Onion defaults changed:
    • "Color Correction" for mGBA is now OFF
    • "Don't Overwrite SaveRAM" is now ON

@Aemiii91 Aemiii91 added the enhancement New or improved feature label Jul 7, 2023
@Aemiii91 Aemiii91 added this to the v4.2 milestone Jul 7, 2023
@Aemiii91 Aemiii91 self-assigned this Jul 7, 2023
@Aemiii91
Copy link
Member Author

Aemiii91 commented Jul 7, 2023

Installation test cases

  • (Mac user) Delete .tmp_update on SD card if exists, copy only miyoo and miyoo354 dirs to SD card (observe a ~12s delay on install boot, caused by /customer/main trying to start non-existing processes)
  • (Normal install) Delete .tmp_update on SD card if exists, copy .tmp_update, miyoo, and miyoo354 dirs to SD card (no longer a delay present)
  • (Update) Keep .tmp_update on SD card, copy only miyoo dir to SD card (no delay either)

@Aemiii91 Aemiii91 added this pull request to the merge queue Jul 8, 2023
Merged via the queue into main with commit 2e6f538 Jul 8, 2023
@Aemiii91 Aemiii91 deleted the fix-improve-installer branch July 8, 2023 21:39
github-merge-queue bot pushed a commit that referenced this pull request Jul 13, 2023
… fixes - File size check (#969)

## Problem

GLO is currently calculating the chksum of both the current rom and core
- this is very slow for PSX.


## Changes

* Cksum generation removed from GLO (But GLO still makes half of the
cookie, (rom path/core path))
* Cksum generation now added to the host side script but delays
mitigated by checking file size for larger files, less accurate but much
faster.
* Added filesize check to host script (If any of the files are above
25mb we write the file size instead of the cksum) - Cookie label will
become [$typesize] instead of [$typechksum]
* Added filesize check to join script (If any of the files are above
25mb we match the file size instead of the cksum)
* Add proxy scripts to Favourites/Recents/RApp (moved parent scripts to
.tmp_update/script/easynetplay)
* Aligned log output with host or client
* Fixed intermittent issue with wifi re-association.
* Replaced "exit" with "cleanup" to attempt to restore the wifi state if
we don't actually do a full cycle (we fail somewhere critical) - this
will also clear bad network ids from wpa_supp so we don't bloat the file
with stale networks.
* Improved speeds for both hosting and joining
* Fixed logging output and filename
* GLO now hides scripts not available to the MM (such as networking)
* Fixed host side hanging on exit of retroarch
* Script names updated
* Clientside now confirms joining

## To-do:

- [x] Allow the process to be killed at any point
- [ ] ~~Add custom RetroArch config overrides for Easy Netplay~~ Tested
on stock config.
- [x] Add migration script to remove old romscripts (see #991)

## Tested with:

PSX Rearmed -> Digimon world 2 - to validate delay on larger roms
TGBDual GB - Balloon kid - to validate we haven't regressed on smaller
roms/cores
mGBA GBA - Pokemon Leaf (Spectate) to validate we haven't regressed on
smaller roms/cores
Bubble bobble - NES - FCEUMM - Netplay works flawlessly for 2p on this
game

We could really do with a progress bar on the rom download/core download
screen if PSX ever becomes viable for netplay as the roms are huge.

---------

Co-authored-by: XK <[email protected]>
@Aemiii91 Aemiii91 added bug in onion Something isn't working and removed enhancement New or improved feature labels Oct 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug in onion Something isn't working
Projects
Status: 👀 In review
Development

Successfully merging this pull request may close these issues.

2 participants