Skip to content

Update ChucK to 1.5.5.2#26

Merged
DBraun merged 2 commits intoDBraun:mainfrom
Oran2009:chuck-1.5.5.2
Sep 27, 2025
Merged

Update ChucK to 1.5.5.2#26
DBraun merged 2 commits intoDBraun:mainfrom
Oran2009:chuck-1.5.5.2

Conversation

@Oran2009
Copy link
Contributor

Updated ChucK to 1.5.5.2. Two key fixes:

CHUCK_PARAM_USER_CHUGIN_DIRECTORIES renamed to CHUCK_PARAM_IMPORT_PATH_SYSTEM in src/Plugin_ChucK.cpp
Added #include <algorithm> in src/ChucKListenerCHOP.cpp

#include "ChucKListenerCHOP.h"
#include "Plugin_ChucK.h"

#include <algorithm>
Copy link
Owner

Choose a reason for hiding this comment

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

Is this necessary?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It was necessary for me to build successfully on Windows using Python 3.11.

chugin_search.push_back(globalDir + "/chugins" );

chuck->setParam( CHUCK_PARAM_USER_CHUGIN_DIRECTORIES, chugin_search );
chuck->setParam( CHUCK_PARAM_IMPORT_PATH_SYSTEM, chugin_search );
Copy link
Owner

Choose a reason for hiding this comment

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

It seems like it should be CHUCK_PARAM_CHUGIN_LIST_USER_DIR https://github.com/ccrma/chuck/blob/28a3d5fe3182ade782ca774c4161359a74bf126d/VERSIONS#L665

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That was my first thought as well, but seems like that variable was renamed somewhere in between 1.5.3.0 and 1.5.5.2. See below:

https://github.com/ccrma/chuck/blob/29d801d72ddc604a4590566cd6a16e3ee1f7577c/src/core/chuck.cpp#L708
and
https://github.com/ccrma/chuck/blob/e67c82e9081fc7490beaa494dc7a235fc0d1220e/src/core/chuck.cpp#L727

Copy link
Owner

Choose a reason for hiding this comment

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

Interesting. I asked on the #bug-reports Discord. It's probably just a mistake in VERSIONS.

@DBraun
Copy link
Owner

DBraun commented Aug 12, 2025

Thanks! It looks like I should update the macOS GitHub actions. Don't worry about that, but can you address the two comments? It would be helpful to hear if you're testing it with chugins.

@Oran2009
Copy link
Contributor Author

Thanks! It looks like I should update the macOS GitHub actions. Don't worry about that, but can you address the two comments? It would be helpful to hear if you're testing it with chugins.

Replied to both comments. Note that I am building on Windows with Python 3.11, I don't have a macos device to test on there. I haven't tested chugins yet.

@DBraun DBraun merged commit 3ae9608 into DBraun:main Sep 27, 2025
2 of 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.

2 participants

Comments