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

Add gb-starter-kit to test suite #849

Open
1 of 2 tasks
ISSOtm opened this issue Apr 20, 2021 · 10 comments
Open
1 of 2 tasks

Add gb-starter-kit to test suite #849

ISSOtm opened this issue Apr 20, 2021 · 10 comments
Labels
builds This affects the build process or release artifacts enhancement Typically new features; lesser priority than bugs meta This isn't related to the tools directly: repo organization, maintainership... tests This affects the test suite

Comments

@ISSOtm
Copy link
Member

ISSOtm commented Apr 20, 2021

@avivace
Copy link
Member

avivace commented Apr 20, 2021

Nice, having them tested in the CI allows those entries to stay consistent and relevant. Thank you

@ISSOtm ISSOtm added enhancement Typically new features; lesser priority than bugs meta This isn't related to the tools directly: repo organization, maintainership... tests This affects the test suite labels Apr 20, 2021
@Rangi42
Copy link
Contributor

Rangi42 commented Apr 26, 2021

gb-starter-kit has two FAILs in src/header.asm for when configuration is needed, which would have to become conditional, like MACRO must_configure / IF !DEF(TEST_RGBDS) / FAIL \1 / ENDC / ENDM and pass rgbasm -D TEST_RGBDS.

@Rangi42 Rangi42 added this to the v0.5.2 milestone May 5, 2021
@ISSOtm ISSOtm closed this as completed in 1ca45ae Jul 25, 2021
ISSOtm added a commit to ISSOtm/rgbds that referenced this issue Jul 25, 2021
ISSOtm added a commit to ISSOtm/rgbds that referenced this issue Jul 25, 2021
@ISSOtm ISSOtm reopened this Jul 25, 2021
@ISSOtm
Copy link
Member Author

ISSOtm commented Jul 25, 2021

For some reason, gb-starter-kit fails to build on macOS and Windows, even with parallel build disabled. This debug output puzzles me:

  Prerequisite `res/crash_font.1bpp.pb8.size' is newer than target `dep/crash_handler.mk'.
 No need to remake target `dep/crash_handler.mk'.

ISSOtm added a commit to ISSOtm/rgbds that referenced this issue Jul 25, 2021
@ISSOtm ISSOtm mentioned this issue Jul 27, 2021
ISSOtm added a commit to ISSOtm/rgbds that referenced this issue Oct 11, 2021
@Rangi42 Rangi42 modified the milestones: v0.5.2, v0.5.3 Oct 27, 2021
ISSOtm added a commit to ISSOtm/rgbds that referenced this issue Oct 31, 2021
ISSOtm added a commit to ISSOtm/rgbds that referenced this issue Oct 31, 2021
ISSOtm added a commit to ISSOtm/rgbds that referenced this issue Oct 31, 2021
ISSOtm added a commit to ISSOtm/rgbds that referenced this issue Oct 31, 2021
ISSOtm added a commit to ISSOtm/rgbds that referenced this issue Oct 31, 2021
@ISSOtm
Copy link
Member Author

ISSOtm commented Nov 1, 2021

After doing a lot of testing, it appears we have a make issue: it sometimes generates dep/crash_handler.mk, then attempts to generate obj/crash_handler.o (which fails due to a missing dep), but then considers it generated anyway.

I've tried using deps both order-only and not, generating both files separately, basically everything I could think of. The problem still occurs, but only on some machines and inconsistently. I'm not nearly qualified enough to understand how to make Make (hah!) handle this correctly, so I'm afraid we'll have to shelve this for a while.

ISSOtm added a commit to ISSOtm/rgbds that referenced this issue Apr 30, 2022
ISSOtm added a commit to ISSOtm/rgbds that referenced this issue Apr 30, 2022
ISSOtm added a commit to ISSOtm/rgbds that referenced this issue Apr 30, 2022
ISSOtm added a commit to ISSOtm/rgbds that referenced this issue Apr 30, 2022
ISSOtm added a commit to ISSOtm/rgbds that referenced this issue Apr 30, 2022
@ISSOtm ISSOtm changed the title Adding gb-boilerplate and gb-starter-kit to CI Add new downstream projects to test suite Oct 4, 2022
@pinobatch
Copy link
Member

The --only-free option of #1161 brings a bit more salience to this issue. Zumi on Discord reminded me of it when I posted a draft article about my experience as a first-time contributor.

SameBoot might be typical of cases where another program for personal computers uses RGBDS as a build dependency. I see it as standing in for the case of building a game in an emulator wrapper for release on modern platforms. I thought of these complications that might affect SameBoot:

  • Can we quickly generalize the test script to build SameBoot without SameBoy?
  • Can the script determine what is a "correct" build of all models' ROMs?
  • Does SameBoot's build process support out-of-PATH RGBDS? And if not, would LIJI be open to it?
  • Can the test script cross-build the logo compressor for Windows?

@Rangi42
Copy link
Contributor

Rangi42 commented Nov 3, 2023

https://github.com/zladx/LADX-Disassembly is actively developed; how about it?

https://github.com/pinobatch/libbet is stable and maintained.

ISSOtm added a commit to ISSOtm/rgbds that referenced this issue Nov 5, 2023
Fixes one item of gbdev#849
Note that the two projects were added *before* the two pret ones, as they
are much simpler and build faster.
This should provide better early feedback e.g. in CI.
@ISSOtm
Copy link
Member Author

ISSOtm commented Nov 5, 2023

Looks like this is not quite it yet, but at least, this seems consistent! (I have an explanation for why the file is empty, but I'm not sure why that action is taken.)

I should fix the warnings raised, at least, to clean up the log a bit.

ISSOtm added a commit to ISSOtm/rgbds that referenced this issue Nov 5, 2023
Fixes one item of gbdev#849
Note that the two projects were added *before* the two pret ones, as they
are much simpler and build faster.
This should provide better early feedback e.g. in CI.
ISSOtm added a commit to ISSOtm/rgbds that referenced this issue Nov 5, 2023
Fixes one item of gbdev#849
Note that the two projects were added *before* the two pret ones, as they
are much simpler and build faster.
This should provide better early feedback e.g. in CI.
@ISSOtm
Copy link
Member Author

ISSOtm commented Nov 5, 2023

Please see ISSOtm/gb-starter-kit#1 (comment) for details about the new issue that popped up.

@Rangi42
Copy link
Contributor

Rangi42 commented Dec 6, 2023

https://github.com/pinobatch/libbet is stable and maintained.

And in the test suite as of #1260. :)

@ISSOtm
Copy link
Member Author

ISSOtm commented Dec 6, 2023

Now that #1264 has been merged, the only project mentioned thus far in this thread that hasn't been added is gb-boilerplate/starter-kit1. As mentioned above, its build is unreliable on Windows due to an issue with Make itself, so that may be a while. For that reason, I am unassigning the milestone.

Let's keep this issue open for more suggestions, though!

Footnotes

  1. The LADX disassembly is in the process of being added, and should land soon-ish.

@ISSOtm ISSOtm removed this from the v0.8.0 milestone Dec 6, 2023
@Rangi42 Rangi42 added the builds This affects the build process or release artifacts label Dec 8, 2023
@Rangi42 Rangi42 changed the title Add new downstream projects to test suite Add gb-started-kit to test suite Aug 6, 2024
@Rangi42 Rangi42 changed the title Add gb-started-kit to test suite Add gb-starter-kit to test suite Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
builds This affects the build process or release artifacts enhancement Typically new features; lesser priority than bugs meta This isn't related to the tools directly: repo organization, maintainership... tests This affects the test suite
Projects
None yet
Development

No branches or pull requests

4 participants