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

roming tool build errors. #738

Open
7dog123 opened this issue Mar 2, 2025 · 4 comments
Open

roming tool build errors. #738

7dog123 opened this issue Mar 2, 2025 · 4 comments

Comments

@7dog123
Copy link

7dog123 commented Mar 2, 2025

I'm trying to build the ps2dev toolchain on mingw64 because eventually mingw32 will be deprecated sometime in the future as it's already considered as Legacy Environment and I've got these errors.

src/romimg.c: In function 'GetExtInfoStat':
src/romimg.c:95:87: error: cast from pointer to integer of different size [-Werror=pointer-to-int-cast]
   95 |                 struct ExtInfoFieldEntry *ExtInfoEntry = (struct ExtInfoFieldEntry *)((RMIMG_PTRCAST)ImageStat->image + fd->ExtInfoOffset);
      |                                                                                       ^
src/romimg.c:95:58: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast]
   95 |                 struct ExtInfoFieldEntry *ExtInfoEntry = (struct ExtInfoFieldEntry *)((RMIMG_PTRCAST)ImageStat->image + fd->ExtInfoOffset);
      |                                                          ^
src/romimg.c: In function 'CreateBlankROMImg':
src/romimg.c:180:25: error: cast from pointer to integer of different size [-Werror=pointer-to-int-cast]
  180 |         memcpy((void *)((RMIMG_PTRCAST)ResetFile->ExtInfoData + sizeof(struct ExtInfoFieldEntry)), &ROMImg->date, ExtInfoEntry->ExtLength);
      |                         ^
src/romimg.c: In function 'LoadROMImg':
src/romimg.c:310:74: error: cast from pointer to integer of different size [-Werror=pointer-to-int-cast]
  310 |                                         ImageStat.ROMFS_start = (void *)((RMIMG_PTRCAST)ImageStat.image + DataStartOffset);
      |                                                                          ^
src/romimg.c:310:65: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast]
  310 |                                         ImageStat.ROMFS_start = (void *)((RMIMG_PTRCAST)ImageStat.image + DataStartOffset);
      |                                                                 ^
src/romimg.c:349:92: error: cast from pointer to integer of different size [-Werror=pointer-to-int-cast]
  349 |                                                         memcpy(file->ExtInfoData, (void *)((RMIMG_PTRCAST)ImageStat.image + RomDirFileFd.ExtInfoOffset + ExtInfoOffset), RomDir->ExtInfoEntrySize);
      |                                                                                            ^
src/romimg.c:349:83: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast]
  349 |                                                         memcpy(file->ExtInfoData, (void *)((RMIMG_PTRCAST)ImageStat.image + RomDirFileFd.ExtInfoOffset + ExtInfoOffset), RomDir->ExtInfoEntrySize);
      |                                                                                   ^
src/romimg.c:351:89: error: cast from pointer to integer of different size [-Werror=pointer-to-int-cast]
  351 |                                                         memcpy(file->FileData, (void *)((RMIMG_PTRCAST)ImageStat.image + offset), RomDir->size);
      |                                                                                         ^
src/romimg.c:351:80: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast]
  351 |                                                         memcpy(file->FileData, (void *)((RMIMG_PTRCAST)ImageStat.image + offset), RomDir->size);
      |                                    
@israpps
Copy link
Contributor

israpps commented Mar 2, 2025

@AKuHAK what about removing ROMIMG and replacing it with romman?

@AKuHAK
Copy link
Contributor

AKuHAK commented Mar 2, 2025

Romman needs a bit modifications to be pure CMD only (no interaction, no config files).

@israpps
Copy link
Contributor

israpps commented Mar 3, 2025

Is it really needed?

On PS2SDK we'll only use it for making IOPRP images

@AKuHAK
Copy link
Contributor

AKuHAK commented Mar 3, 2025

Is it really needed?

On PS2SDK we'll only use it for making IOPRP images

Yes, and currently romman is overkill for this simple task as it doesn't support CMD only ioprp building (that is necessarry for ci)

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

No branches or pull requests

3 participants