Skip to content

fix(erl): match erl_id to the library name the build produces - #939

Merged
uyjulian merged 1 commit into
ps2dev:masterfrom
rixnobis:erl/erl-id-names
Oct 13, 2026
Merged

uyjulian merged 1 commit into
ps2dev:masterfrom
rixnobis:erl/erl-id-names

Conversation

@rixnobis

Copy link
Copy Markdown
Contributor

Defect 4 from #924. Six erl-support.c files declare an erl_id that differs from the .erl the buildsystem emits (EE_LIB, or lib<dir> from Rules.lib.make): tcpip libps2ip141 -> libps2ip, tcpips libps2ip -> libps2ips, keyboard libkeyboard -> libkbd, ps2snd libps2kit -> libps2snd, remote libremote -> librm, sbv libsbv -> libpatches. Per your reply on #924 the buildsystem name wins, so only the erl-support.c side changes. No erl_dependancies entry names any of the old ids. The other 30 already match.

Six erl-support.c files declared an erl_id that differs from the .erl the buildsystem emits (tcpip, tcpips, keyboard, ps2snd, remote, sbv). The buildsystem name is the intended one (ps2dev#924).

@fjtrujy fjtrujy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!
@uyjulian ?

@uyjulian uyjulian left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@uyjulian
uyjulian merged commit 329e2bb into ps2dev:master Oct 13, 2026
5 checks passed
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.

3 participants