Skip to content

Attempt at GitHub Actions CI - #5

Closed
Margen67 wants to merge 1 commit into
CTurt:masterfrom
Margen67:actions
Closed

Attempt at GitHub Actions CI#5
Margen67 wants to merge 1 commit into
CTurt:masterfrom
Margen67:actions

Conversation

@Margen67

@Margen67Margen67 commented Jun 29, 2020

Copy link
Copy Markdown

This workflow should provide automatic builds with GitHub Actions, but fails: https://github.com/Margen67/FreeDVDBoot/runs/819186393#step:4:8

@thblckjkr

Copy link
Copy Markdown

It looks like the image ubuntu-latest doesn't have the genisoimage tool preinstalled.

To solve it...

You could install it with a step on the build process with apt-get update -y && apt-get install genisoimage -y. But that takes some time, and it could be wasting build time for future builds (It will depend on how many builds this repo will have).

Also, you could look for the .deb package from this site, download it with wget, and then apt install PACKAGE.deb or dpkg -i PACKAGE.deb; apt install -f. That will be probably more time efficient since it is not updating the entire apt repo.

The simplest way (i think) would be to just download the binary and run it like directly, but i haven't seen any binary hosted, there is an example of someone that does it witha simple script, but the binary is hosted on dropbox and it could give a little bit of trouble.

So, I guess you could try the first method and see if it works, then discuss it with CTurt to see if he likes it.

@JayFoxRox

JayFoxRox commented Jun 30, 2020

Copy link
Copy Markdown

It looks like the image ubuntu-latest doesn't have the genisoimage tool preinstalled.

Are we looking at the same log?(Edit: No, we weren't)

Yes, but only for 3.11E and 3.11J.


For 3.10EU it also has this:

payload.c:67: warning: `noreturn' function does return
/usr/local/ps2dev/ee/lib/gcc-lib/ee/3.2.3/../../../../ee/bin/ld: unrecognized option '-z'
/usr/local/ps2dev/ee/lib/gcc-lib/ee/3.2.3/../../../../ee/bin/ld: use the --help option for usage information

It appears that the linker that ships with ps2sdk, for some reason, doesn't like the -z option. I could imagine that it's just a symlink to the platform linker, which isn't what ps2sdk expects? Maybe ps2sdk also needs an ancient compiler.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@Margen67@thblckjkr@JayFoxRox