Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 367727 - Incorrect error message when eclipse-build version does not match the source tarbals
Summary: Incorrect error message when eclipse-build version does not match the source ...
Status: RESOLVED FIXED
Alias: None
Product: Linux Tools
Classification: Tools
Component: eclipse-build (show other bugs)
Version: 1.0   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 1.0   Edit
Assignee: Krzysztof Daniel CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-03 04:19 EST by Krzysztof Daniel CLA
Modified: 2019-05-17 05:24 EDT (History)
2 users (show)

See Also:


Attachments
Fix proposition (1.26 KB, patch)
2012-01-04 05:36 EST, Krzysztof Daniel CLA
krzysztof.daniel: review-
Details | Diff
Fix proposition (3.36 KB, patch)
2012-01-04 05:52 EST, Krzysztof Daniel CLA
akurtakov: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Krzysztof Daniel CLA 2012-01-03 04:19:18 EST
when build.properties label does not match the source tarbal name, 
the error always is like that:

BUILD FAILED
/home/kdaniel/redhatwork/org.eclipse.linuxtools.eclipse-build/eclipse-build/build.xml:151: SDK source tarball is not present.  Please either download a file such as eclipse-3.6.1-src.tar.bz2 from http://download.eclipse.org/technology/linuxtools/eclipse-build/ or generate one by running ./buildSDKSource.sh.


Better error message and possibly some recovery instructions should be displayed there.
Comment 1 Alexander Kurtakov CLA 2012-01-04 04:58:28 EST
It would be really good if the tarball name in the message is not hardcoded but to use the name that ant tries to unpack
Comment 2 Krzysztof Daniel CLA 2012-01-04 05:36:36 EST
Created attachment 208993 [details]
Fix proposition

Fix proposition
Comment 3 Krzysztof Daniel CLA 2012-01-04 05:52:44 EST
Created attachment 208994 [details]
Fix proposition
Comment 4 Alexander Kurtakov CLA 2012-01-04 10:32:49 EST
Thanks for the patch.
Applied.
Comment 5 Alexander Kurtakov CLA 2012-01-04 10:35:04 EST
Closing.