search

ps2homebrew / wLaunchELF

ELF loader and File browser for Sony PlayStation 2
Other
502 stars 51 forks source link

change the filename of the packed ELF #21

Closed Jay-Jay-OPL closed 5 years ago

Jay-Jay-OPL commented 6 years ago

I noticed that when we compile wLE, we get two ELF files:

BOOT.ELF (not packed) ULE.ELF (packed)

The issue is that the packed version is ULE.ELF and that is the one we tend to share so users can place that file inside the BOOT folder in their MCs for FMCB and etc. The issue is that we have to rename it to BOOT.ELF so it follows the standard filename that we are mostly accustom to.

So I was thinking, wouldn't it be best to reverse the two filenames, so the one that gets packed should be already renamed to BOOT.ELF? This way it is easier for us to share it, without having the task of having to rename it prior to sharing it? For convenience purposes only?

In the Makefile (LINE 6), this can be done like this:

FIND:

EE_BIN = BOOT.ELF
EE_BIN_PKD = ULE.ELF

REPLACE, WITH:

EE_BIN = ULE.ELF
EE_BIN_PKD = BOOT.ELF

Your thoughts?

sp193 commented 5 years ago

"BOOT.ELF" sounds like something that was used by a modchip. The README file for v3.41i (one of the earliest versions of uLaunchELF, after transitioning from LaunchELF) suggests its development sometimes involved the Matrix modchip. Back then, it was not packed. It seemed to not be never packed, until @AKuHAK changed it at "wip7".

So @AKuHAK, should we change it? Would it affect your build bot?

AKuHAK commented 5 years ago

Hi all, my bot build uses latest versions of all dependencies except ps2toolchain which I should update manually. It uses simple make command. But yes, my bot is hardcoded to boot.elf .

ср, 31 окт. 2018 г., 16:20 Liu Woon Yung notifications@github.com:

"BOOT.ELF" sounds like something that was used by a modchip. The README file for v3.41i (one of the earliest versions of uLaunchELF, after transitioning from LaunchELF) suggests its development sometimes involved the Matrix modchip. Back then, it was not packed. It seemed to not be never packed, until @AKuHAK https://github.com/AKuHAK changed it at "wip7".

So @AKuHAK https://github.com/AKuHAK, should we change it? Would it affect your build bot?

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/AKuHAK/uLaunchELF/issues/21#issuecomment-434705099, or mute the thread https://github.com/notifications/unsubscribe-auth/AAl8SObrc3B-dGzEpPkG_pxAddZaOqzZks5uqbGlgaJpZM4WC3Rq .

sp193 commented 5 years ago

That's great. Thanks! Related commit: 9c18290.