Skip to content

Conversation

@MKLythus
Copy link

@MKLythus MKLythus commented Sep 30, 2024

This pull request...

  • Fixes a bug
  • Introduces a new feature
  • Improves an existing feature
  • Boosts code quality or performance

Description

Updated JDA-Utilities in accordance with the fix provided in #1700

Purpose

Fixes the issue where JDA-Utilities dependency is missing, making the jar unable to be built in maven. This PR also allows other contributors/people to build the project on their own.

Relevant Issue(s)

This PR closes issue #1700

@gasms1424
Copy link

gasms1424 commented Nov 19, 2024

I made a somewhat "crazy" fix for the missing/not found dependency problem after racking my brains trying to compile the latest version of jmusic, if you want to use it follow the steps:

• Get JdaUtilities.jar (3.0.5 for jmusic 0.4.3)
• Extract JAR using supported program (ex: winrar/7zip)
• Copy the com folder and place it in the following directory: ${project.basedir}/src/main/resources
• Copy the JAR for same directory
The directory with the dependency should look like this:

└── ${project.basedir}/
    └── src/
        └── main/
            └── resources/
                ├── com/
                │   └── jagrosh/
                │       └── jdautilities/
                │           ├── command
                │           ├── commons
                │           ├── doc
                │           ├── examples
                │           ├── menu
                │           └── oauth2
                ├── natives
                ├── jdautilities.jar
                ├── logback.xml
                └── reference.conf

• Add add this plugin in pom.xml:

        <plugin>
            <groupId>org.apache.maven.plugins</groupId>
            <artifactId>maven-compiler-plugin</artifactId>
            <version>3.8.1</version>
            <configuration>
                <source>1.8</source>
                <target>1.8</target>
            </configuration>
        </plugin>

• Use mvn package to compile

Warning: this is a temporary fix that I discovered after much research...

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