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

Ultralib Update #233

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Ultralib Update #233

wants to merge 3 commits into from

Conversation

hensldm
Copy link
Contributor

@hensldm hensldm commented Dec 25, 2024

Update ultralib to current main. The main thing is ultralib now has the correct prototypes for print functions, so explicit casts are no longer needed as well as moving some internal PR includes to easier to access headers. Also comes with some newer documentation, and comes with the ability to build using modern gcc if desired.

subrepo:
  subdir:   "lib/ultralib"
  merged:   "74f0eee"
upstream:
  origin:   "[email protected]:decompals/ultralib.git"
  branch:   "main"
  commit:   "2717d45"
git-subrepo:
  version:  "0.4.9"
  origin:   "https://github.com/ingydotnet/git-subrepo"
  commit:   "cce3d93"
@AngheloAlf
Copy link
Contributor

Seems like the ultralib update introduced new warnings

@hensldm
Copy link
Contributor Author

hensldm commented Dec 26, 2024

Ahh, its mainly the new kmcprintf which shouldn't even be building tbh. I've created a PR in ultralib to fix that. The other ones have to do with different line numbers due to added comments in asm files.

@hensldm
Copy link
Contributor Author

hensldm commented Dec 26, 2024

Drafting until ultralib gets updated to not build kmcprintf for non debug versions

@hensldm hensldm marked this pull request as draft December 26, 2024 16:14
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.

2 participants