search

greynewell / musegpt

Local LLMs in your DAW!
https://musegpt.org/
GNU Affero General Public License v3.0
28 stars 1 forks source link

fix: Remove requirement for WebView 2 #9 #19

Closed greynewell closed 3 days ago

greynewell commented 3 days ago

Summary by Sourcery

Remove the WebView 2 requirement from the CMake configuration to fix compatibility issues.

Bug Fixes:

sourcery-ai[bot] commented 3 days ago

Reviewer's Guide by Sourcery

This pull request removes the requirement for WebView 2 in the project's CMake configuration. It modifies the CMakeLists.txt file to remove WebView 2-related settings, potentially simplifying the build process and reducing dependencies.

File-Level Changes

Change Details Files
Removed WebView 2 requirement and related compile definitions
  • Removed 'NEEDS_WEBVIEW2 TRUE' from juce_add_plugin configuration
  • Removed 'JUCE_USE_WIN_WEBVIEW2_WITH_STATIC_LINKING=1' from target_compile_definitions
 CMakeLists.txt
Tips - Trigger a new Sourcery review by commenting `@sourcery-ai review` on the pull request. - Continue your discussion with Sourcery by replying directly to review comments. - You can change your review settings at any time by accessing your [dashboard](https://app.sourcery.ai): - Enable or disable the Sourcery-generated pull request summary or reviewer's guide; - Change the review language; - You can always [contact us](mailto:support@sourcery.ai) if you have any questions or feedback.