-
Notifications
You must be signed in to change notification settings - Fork 26
JBDS-3361 fetch target platform zip, unpack it, and use it within JBDS update site >> master #283
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is no target platform folder in PR or current repository. Am I missing something or some resources are missing in PR?
|
I got The build could not read 1 project -> [Help 1]
[ERROR]
[ERROR] The project com.jboss.devstudio:product:9.0.0-SNAPSHOT (/home/eskimo/Projects/jbdevstudio/4.3.x/fork/jbds-releng/jbdevstudio-product/pom.xml) has 1 error
[ERROR] Child module /home/eskimo/Projects/jbdevstudio/4.3.x/fork/jbds-releng/jbdevstudio-product/targetplatform of /home/eskimo/Projects/jbdevstudio/4.3.x/fork/jbds-releng/jbdevstudio-product/pom.xml does not exist
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging. |
|
Sorry about that; missed that in the commit. Added here: https://github.com/nickboldt/jbdevstudio-product/blob/JBDS-3361_getzip_master/targetplatform/pom.xml |
|
IMO to many changes for small fix.
It is going to be change only in results/pom.xml and all done on maven side. |
|
Why in results? Results is really just for renaming & moving stuff around. My way is what you asked for in the previous PR: instead of mirroring the TP, fetch the zip. Then use that in update site build and installer. If we did the work in the results phase, we'd still be fetching TP for the installer, then doing it AGAIN in results. This is more efficient. |
|
You right about target platform. It is good to have it downloaded as a zip, but in current PR it still going to be downloaded twice:
|
|
devstudio-earlyaccess.properties download during build is killing me ;( |
|
Still cannot build product with this PR |
|
Where is latest aggregated JBT Core 4.3.0.Alpha2 published ? |
|
replaced by #284 |
No description provided.