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

Create build workflow #15

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Create build workflow #15

wants to merge 3 commits into from

Conversation

dakotath
Copy link
Contributor

@dakotath dakotath commented Jan 1, 2025

Moving the build workflow over here, something is going on with devkitPro package servers, because it's all returning 403 now on this workflows.

@DacoTaco
Copy link
Member

DacoTaco commented Jan 1, 2025

youre not using the docker container, use the docker container

@dakotath
Copy link
Contributor Author

dakotath commented Jan 1, 2025

Alright, I've switched to the docker container, but it's still 403ing when I do dkp-pacman -Syu

@DacoTaco
Copy link
Member

DacoTaco commented Jan 1, 2025

because you don't need to do the pacman -Syu, thats not up to the workflow to manage :)
does it build without it?
(i suspect the docker images need updating, but thats aside this PR)

@dakotath
Copy link
Contributor Author

dakotath commented Jan 1, 2025

It builds until it reaches the crypto examples (failes on SHA Engine Example), I do suspect the images would need to be updated too, since all examples build fine on all my other systems and the old workflow.

@DacoTaco
Copy link
Member

DacoTaco commented Jan 1, 2025

ye, they'll need updating. i suggest removing the pacman -Syu and ill see to get the dockers updated for it :)

@dakotath
Copy link
Contributor Author

dakotath commented Jan 1, 2025

Done that already.

@Crayon2000
Copy link
Contributor

@dakotath, is there a reason why you changed from actions/checkout@v4 to actions/[email protected]?

@dakotath
Copy link
Contributor Author

dakotath commented Jan 1, 2025

Honestly I forgot I even changed that

Copy link
Member

@WinterMute WinterMute left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's devkitPPC, not DevKitPPC.

@Fancy2209
Copy link

by the way, github actions runners all have 4 cores now, so you can use -j4

@Fancy2209
Copy link

Fancy2209 commented Jan 6, 2025

It's devkitPPC, not DevKitPPC.

can't this be fixed by editing the commit message when doing a squash merge?

@WinterMute
Copy link
Member

It's devkitPPC, not DevKitPPC.

can't this be fixed by editing the commit message when doing a squash merge

The comment is for information in a vain attempt to get people to name things properly and stop the random capitalisation. This commit message wouldn't appear on squash and the contributor would remain unaware of the proper naming assuming this gets merged. Personally I don't really see the benefit of a workflow here and I really don't see the benefit of providing artifacts for an examples repository intended to illustrate use of the tools and also serve as a test for installed tools/libs.


- name: Build
run: |
make -j2
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

-j2 should be replaced with $(nproc) as I did here switchbrew/libnx@d6e38f2

Runners currently have up to 4 cores but I've still seen 2 core runners recently and of course they may up the cores in the future.

run: |
make -j2

- name: Upload Build Artifacts
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

uploading artifacts from an examples repository just seems like a waste of resources to me.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not necessary your own storage that's getting used during the process, so it doesn't really seem like a big deal to me. And also, there are things in certain build environments that break, and I see useful to testing things in multiple build environments (including actions). For example: sometimes I'll update libogc on my Arch laptop and then everything breaks to the point of the exception handler not even working. (This type of result has never been solidly reproducible so don't ask me how to reproduce it). Besides that, I'll get to the rest of the file when I'm done the day.

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

Successfully merging this pull request may close these issues.

5 participants