search

willowtreeapps / conductor-mobile

Conductor Mobile is a port of the Conductor Web Framework for iOS and Android
Apache License 2.0
6 stars 5 forks source link

[BUG] version 0.19.0 failed to build for Jitpack #128

Closed aaron-goff closed 5 years ago

aaron-goff commented 5 years ago

The problem

When publishing on jitpack, version 0.19.0 failed. https://jitpack.io/#willowtreeapps/conductor-mobile

Build starting...
Start: Fri Aug 30 18:55:10 UTC 2019 e273148adafd
Git:
0.19.0-0-g11ae006
commit 11ae006b23a422cd6bcccaefb7cc815951676e36
Merge: 493a08d bc21e26
Author: Mark Han 
Date:   Tue Aug 27 12:02:42 2019 -0400

    Merge branch 'master' into release/0.19.0

Running install command:
./gradlew assemble publishToMavenLocal
Picked up JAVA_TOOL_OPTIONS: -Dfile.encoding=UTF-8 -Dhttps.protocols=TLSv1.2
Error: Could not find or load main class org.gradle.wrapper.GradleWrapperMain
Build tool exit code: 0
Looking for artifacts...
Picked up JAVA_TOOL_OPTIONS: -Dfile.encoding=UTF-8 -Dhttps.protocols=TLSv1.2
Picked up JAVA_TOOL_OPTIONS: -Dfile.encoding=UTF-8 -Dhttps.protocols=TLSv1.2
Looking for pom.xml in build directory and ~/.m2
2019-08-30T18:55:13.819712695Z
Exit code: 0

ERROR: No build artifacts found
aaron-goff commented 5 years ago

@mrk-han and myself have been looking into this and we're kinda lost... Our best guess is that removing the pom without correctly implementing the gradle to publish to jitpack is causing issues (looking at the log output for 0.18.0, it builds with the pom even though the project has been migrated to gradle), but we're not 100% sure on how to a) fix it and b) test the fix...

Removing myself and adding help wanted cause 🤷‍♂ I'm lost

mrk-han commented 5 years ago

Just checking out and mentioning differences in the documentation vs our setup.

./gradlew assemble vs ./gradlew build image

Docs suggest build but we are using assemble, they almost do the same thing.