Skip to content

Remove bundled p5.sound from complete library zip - #9073

Merged
ksen0 merged 9 commits into
processing:mainfrom
AmanUllah687:remove-bundled-p5sound
Sep 11, 2026
Merged

ksen0 merged 9 commits into
processing:mainfrom
AmanUllah687:remove-bundled-p5sound

Conversation

@AmanUllah687

Copy link
Copy Markdown
Contributor

Resolves #9037

p5.sound is now maintained as a separate library
(https://www.jsdelivr.com/package/npm/p5.sound) and the bundled
lib/addons/p5.sound.js was a stale v1.0.1 build (2021) incompatible
with the 2.x core, causing "registerMethod is not a function" errors.

The website download page now links to the current p5.sound CDN
build instead (processing/p5.js-website#1560, merged).

p5.sound is now a separately maintained library (https://www.jsdelivr.com/package/npm/p5.sound) and the bundled lib/addons/p5.sound.js was a stale v1.0.1 build (2021) incompatible with the 2.x core, causing 'registerMethod is not a function' errors.

The website download page now links to the current p5.sound CDN build instead (processing/p5.js-website#1560).

Resolves processing#9037
@p5-bot

p5-bot Bot commented Aug 11, 2026

Copy link
Copy Markdown

Comment on lines -1 to -3
<!DOCTYPE html>
<html lang="">

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.

Why is this part removed?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good catch, thank you, that was accidentally dropped during editing, not an intentional removal. Just pushed a fix restoring the DOCTYPE and html tags. The only actual change to this file is the comment explaining where to get p5.sound.js.

@ksen0
ksen0 merged commit c76e600 into processing:main Sep 11, 2026
4 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.

[p5.js 2.0+ Bug Report]: Wrong version of p5-sound included in complete library zip

3 participants