nh-server / switch-guide

ISC License
210 stars 80 forks source link

Remove unnecessary lines from hekate_ipl.ini and improve config readability #206

Closed Sanrax closed 1 year ago

Sanrax commented 1 year ago

The spacing of hekate_ipl.ini provided by the guide makes it confusing to read at first glance, like using {} instead of just a blank line to separate sections.

The naming scheme is inconsistent. We have Atmosphere FSS0 SYS and Atmosphere FSS0 emuMMC. Sys is also an emmc chip, so wouldn't it be more appropriate to have it say Atmosphere FSS0 sysMMC to match the naming of emu?

This is an example of my proposed config included in this PR to fix the mentioned issues, and merge into the main branch: (both emu and sys .ini's have been updated)

[config]
updater2p=1

[Atmosphere FSS0 emuMMC]
fss0=atmosphere/package3
kip1=atmosphere/kips/*
emummcforce=1
icon=bootloader/res/emu_boot.bmp

[Atmosphere FSS0 sysMMC]
fss0=atmosphere/package3
kip1=atmosphere/kips/*
emummc_force_disable=1
icon=bootloader/res/sys_cfw_boot.bmp

[Stock sysMMC]
fss0=atmosphere/package3
emummc_force_disable=1
stock=1
icon=bootloader/res/stock_boot.bmp
oreo639 commented 1 year ago

Lgtm although make sure to update references to it here: https://github.com/nh-server/switch-guide/blob/ffb4c17a55557b113b7cfd2bb5b6e6d02b393ee3/docs/user_guide/sysnand/launching_cfw.md?plain=1#L14 https://github.com/nh-server/switch-guide/blob/ffb4c17a55557b113b7cfd2bb5b6e6d02b393ee3/docs/user_guide/emummc/launching_cfw.md?plain=1#L17 https://github.com/nh-server/switch-guide/blob/ffb4c17a55557b113b7cfd2bb5b6e6d02b393ee3/docs/extras/transfer_sd.md?plain=1#L49

Sanrax commented 1 year ago

Lgtm although make sure to update references to it here:

https://github.com/nh-server/switch-guide/blob/ffb4c17a55557b113b7cfd2bb5b6e6d02b393ee3/docs/user_guide/sysnand/launching_cfw.md?plain=1#L14

https://github.com/nh-server/switch-guide/blob/ffb4c17a55557b113b7cfd2bb5b6e6d02b393ee3/docs/user_guide/emummc/launching_cfw.md?plain=1#L17

https://github.com/nh-server/switch-guide/blob/ffb4c17a55557b113b7cfd2bb5b6e6d02b393ee3/docs/extras/transfer_sd.md?plain=1#L49

Alright, will do.

Sanrax commented 1 year ago

Alright @oreo639, changes finished. That should take care of all launch button mentions in the guide.

oreo639 commented 1 year ago

Thanks.