search

nus-cs2113-AY2021S2 / forum

5 stars 0 forks source link

Gradle-wrapper needs to be edited for runtest.sh / runtest.bat to build successfully #57

Open jadenwjh opened 3 years ago

jadenwjh commented 3 years ago

Wondering if any groups are facing this issue where by:

text-ui-test is not building successfully due to the version of gradle stated in gradle-wrapper?

I am referring to the distributionUrl in gradle > wrapper > gradle-wrapper.properties.

6.2-bin is not working for me but once I changed to 6.3 it is fine.

okkhoy commented 3 years ago

Hmm.. sorry I missed this. are you able to build the JAR correctly after updating to 6.3? I don't think the Gradle wrapper version matters so much. You just need to use the correct Java version. One way to test it is to get any of your friend from a different team to run your JAR once.

@jadenwjh

jadenwjh commented 3 years ago

I see. Yup the issue is resolved. Build is successful on 6.3. JAR was also working on multiple computers so we are assuming its fine. Thanks @okkhoy