Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch build to Gradle build tool #117

Closed
frankschmitt opened this issue Oct 2, 2022 · 5 comments · Fixed by #125
Closed

Switch build to Gradle build tool #117

frankschmitt opened this issue Oct 2, 2022 · 5 comments · Fixed by #125
Assignees
Labels
enhancement New feature or request

Comments

@frankschmitt
Copy link
Contributor

Is your feature request related to a problem? Please describe.
Currently, building / running the project can only be done by executing javac Main.java / java Main manually (or running Main from your IDE, e.g. Eclipse). This is especially confusing for new users, who are used to Maven or Gradle and simply wish to run

gradle build

to build their project without having to worry about the dependencies etc.

Also, gradle allows publishing the package to Maven Central, running the tests etc. in a very straight-forward way.

Describe the solution you'd like
I'd like a project setup that allows the user to simply execute

gradle build

to build the project and

gradle run

to run it.

Describe alternatives you've considered
An alternative to Gradle would be [Maven|https://maven.apache.org/]. Similar features, mostly a matter of preferences (personally, I prefer Gradle).

Additional context
If this sounds sensible to you, I'd be willing to implement it :-)

@VerisimilitudeX
Copy link
Owner

VerisimilitudeX commented Oct 2, 2022

Sure, sounds great! I would prefer using Gradle as well. One question, would a user still be able to run the program using main?

@VerisimilitudeX
Copy link
Owner

Feel free to get started on this!

@naXa777
Copy link

naXa777 commented Oct 2, 2022

would a user still be able to run the program using main?

Yes, sure.

@VerisimilitudeX
Copy link
Owner

would a user still be able to run the program using main?

Yes, sure.

Sounds good, I'll approve your PRs as they come. Afterwards, perhaps we can set up a GitHub Actions pipeline for Gradle.

@VerisimilitudeX
Copy link
Owner

VerisimilitudeX commented Oct 3, 2022

Once you implement this, it would be great if you could create a separate PR with documentation on how to use Gradle.

frankschmitt added a commit to frankschmitt/DNAnalyzer that referenced this issue Oct 3, 2022
Configures a basic Gradle Java app build. To stick to Gradle convention as closely as possible, this commit
- adds several Gradle-specific directories / files to the git ignore
list
- moves the source code from src/ to app/src/main/java/
- adds a (dummy) test class in app/src/test/java

It also updates the README and provides basic information on how to
build + run the DNAnalyzer.
frankschmitt added a commit to frankschmitt/DNAnalyzer that referenced this issue Oct 3, 2022
Configures a basic Gradle Java app build. To stick to Gradle convention as closely as possible, this commit
- adds several Gradle-specific directories / files to the git ignore
list
- moves the source code from src/ to app/src/main/java/
- adds a (dummy) test class in app/src/test/java

It also updates the README and provides basic information on how to
build + run the DNAnalyzer.
@VerisimilitudeX VerisimilitudeX linked a pull request Oct 3, 2022 that will close this issue
frankschmitt added a commit to frankschmitt/DNAnalyzer that referenced this issue Oct 6, 2022
Configures a basic Gradle Java app build. To stick to Gradle convention as closely as possible, this commit
- adds several Gradle-specific directories / files to the git ignore
list
- moves the source code from src/ to app/src/main/java/
- adds a (dummy) test class in app/src/test/java

It also updates the README and provides basic information on how to
build + run the DNAnalyzer.

Configure Gradle build. Implements VerisimilitudeX#117.

Configures a basic Gradle Java app build. To stick to Gradle convention as closely as possible, this commit
- adds several Gradle-specific directories / files to the git ignore
list
- moves the source code from src/ to app/src/main/java/
- adds a (dummy) test class in app/src/test/java

It also updates the README and provides basic information on how to
build + run the DNAnalyzer.

copy files over from main branch that were messed up during merge

move source back to src/ subdir (instead of app/src)
frankschmitt added a commit to frankschmitt/DNAnalyzer that referenced this issue Oct 6, 2022
Configures a basic Gradle Java app build. To stick to Gradle convention as closely as possible, this commit
- adds several Gradle-specific directories / files to the git ignore
list
- moves the source code from src/ to app/src/main/java/
- adds a (dummy) test class in app/src/test/java

It also updates the README and provides basic information on how to
build + run the DNAnalyzer.

Configure Gradle build. Implements VerisimilitudeX#117.

Configures a basic Gradle Java app build. To stick to Gradle convention as closely as possible, this commit
- adds several Gradle-specific directories / files to the git ignore
list
- moves the source code from src/ to app/src/main/java/
- adds a (dummy) test class in app/src/test/java

It also updates the README and provides basic information on how to
build + run the DNAnalyzer.

copy files over from main branch that were messed up during merge

move source back to src/ subdir (instead of app/src)
frankschmitt added a commit to frankschmitt/DNAnalyzer that referenced this issue Oct 6, 2022
Configures a basic Gradle Java app build. To stick to Gradle convention as closely as possible, this commit
- adds several Gradle-specific directories / files to the git ignore
list
- moves the source code from src/ to src/main/java/
- adds a (dummy) test class in app/src/test/java

It also updates the README and provides basic information on how to
build + run the DNAnalyzer.

cleanup README (merge old and new getting started)

use up-to-date java files from upstream
VerisimilitudeX added a commit that referenced this issue Oct 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants