Skip to content

Build script adjust #13

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.

Sign up for GitHub

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

Merged
JaceCear merged 3 commits into from
Mar 12, 2026
+42 −12
Merged

Build script adjust #13

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
d6e9241
Update CI/CD script
JaceCear Mar 12, 2026
3782599
Re-add build name
JaceCear Mar 12, 2026
7b6bfe4
Fix build RC
JaceCear Mar 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
46 changes: 35 additions & 11 deletions .github/workflows/build.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,13 +42,6 @@ jobs:
- name: Checkout
if: ${{ github.event_name == 'push' }}
uses: actions/checkout@master

- name: Checkout gh-pages
if: ${{ github.event_name == 'push' }}
uses: actions/checkout@master
with:
path: gh-pages
ref: gh-pages

- name: Checkout agbcc
uses: actions/checkout@master
Expand All @@ -58,8 +51,11 @@ jobs:

- name: Install tools
run: |
sudo apt update && sudo apt install gcc-arm-none-eabi binutils-arm-none-eabi
sudo apt update && sudo apt install gcc-arm-none-eabi binutils-arm-none-eabi xorg-dev
Copy link
Contributor

@freshollie freshollie Mar 12, 2026

Choose a reason for hiding this comment

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

FYI you only need xorg I think for running in headless? I suppose that could be useful for a TAS at some point but it's probably not needed on the GBA build anyway

Copy link
Collaborator Author

@JaceCear JaceCear Mar 12, 2026

Choose a reason for hiding this comment

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

I want to try keeping the file consistent between the 3 of them as much as possible.

We could add TAS tests to the others once they're ready! ^^

python3 -m pip install gitpython
# build-essential, git, and libpng-dev are already installed
# gcc-arm-none-eabi is only needed for the modern build
# as an alternative to dkP

- name: Install agbcc
run: |
Expand All @@ -78,7 +74,6 @@ jobs:
python3 scripts/progress.py text

- name: Generate reports
if: ${{ github.event_name == 'push' }}
run: |
mkdir -p gh-pages/reports
mkdir -p gh-pages/maps
Expand All @@ -88,8 +83,37 @@ jobs:
python3 scripts/progress.py shield-json -m > gh-pages/reports/progress-sa1-shield-matching.json
echo "REPORTS_COMMIT_MSG=$( git log --format=%s ${GITHUB_SHA} )" >> $GITHUB_ENV
cp sa1.map gh-pages/maps/${GITHUB_SHA}.map

- name: Update reports

- name: Upload progress
if: ${{ github.event_name == 'push' || github.event_name == 'pull_request' }}
uses: actions/upload-artifact@v7
with:
path: |
gh-pages

publish-progress:
name: Publish progress
runs-on: ubuntu-24.04
needs: [build-gba]
# Only able to run from repo pull requests (by maintainers, or on the main branch via push)
if: ${{ github.event_name == 'push' || github.event_name == 'pull_request' }}
permissions:
contents: write
steps:
- name: Checkout gh-pages
uses: actions/checkout@master
with:
path: gh-pages
ref: gh-pages
- name: Download progress
uses: actions/download-artifact@v7
with:
path: |
gh-pages
- run: |
cd gh-pages
git status
- name: Publish reports
if: ${{ github.event_name == 'push' }}
uses: EndBug/add-and-commit@v7
with:
Expand Down
2 changes: 1 addition & 1 deletion tools/_shared/arena_alloc/Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ CC = gcc

SOURCE = arena_alloc.c

# Clear the default suffixes
# Clear the default suffixes (needed to fix build race condition)
.SUFFIXES:

.PHONY: all clean
Expand Down
3 changes: 3 additions & 0 deletions tools/_shared/c_header_parser/Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,9 @@ SRCS = c_header_parser.c

CFLAGS = -I $(ARENA_ALLOC_DIR) -Wall -Wextra -Werror $(NO_ERROR) -std=gnu11 -O2

# Clear the default suffixes (needed to fix build race condition)
.SUFFIXES:

.PHONY: all clean

all: libc_header_parser.a
Expand Down
3 changes: 3 additions & 0 deletions tools/_shared/csv_conv/Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,9 @@ SRCS = csv_conv.c
CFLAGS = -I $(CSV_CONV_DIR) -Wall -Wextra -Werror $(NO_ERROR) -std=gnu11 -O2
LDFLAGS =

# Clear the default suffixes (needed to fix build race condition)
.SUFFIXES:

.PHONY: all clean

all: libcsv_conv.a
Expand Down
Loading