| Summary: | [Tooling] Use stable URLs for runtime download | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [RT] RAP | Reporter: | Ralf Sternberg <rsternberg> | ||||
| Component: | Releng | Assignee: | Project Inbox <rap-inbox> | ||||
| Status: | CLOSED FIXED | QA Contact: | |||||
| Severity: | enhancement | ||||||
| Priority: | P1 | CC: | beyhan.veliev | ||||
| Version: | 1.4 | Flags: | rsternberg:
review?
(holger.staudacher) |
||||
| Target Milestone: | 1.4 RC2 | ||||||
| Hardware: | PC | ||||||
| OS: | Linux | ||||||
| Whiteboard: | |||||||
| Bug Depends on: | 346521 | ||||||
| Bug Blocks: | 346811 | ||||||
| Attachments: |
|
||||||
|
Description
Ralf Sternberg
Created attachment 196193 [details]
updates the repository URLs to the stable ones
(In reply to comment #1) > Created attachment 196193 [details] > updates the repository URLs to the stable ones Beyhan, I think the following constants are no longer needed and should be removed, shouldn't they? private final static String TARGET_VERSION_LATEST_BUILD = "1.4"; //$NON-NLS-1$ private final static String TARGET_VERSION_LATEST_RELEASE = "1.3"; //$NON-NLS-1$ (In reply to comment #2) > (In reply to comment #1) > Beyhan, I think the following constants are no longer needed and should be > removed, shouldn't they? > > private final static String TARGET_VERSION_LATEST_BUILD = "1.4"; //$NON-NLS-1$ > private final static String TARGET_VERSION_LATEST_RELEASE = "1.3"; > //$NON-NLS-1$ Not jet. I was waiting for our composite repositories to fix this. I thought there is a bug for this but I dind't find and I opened a new bug 346811. I will look at it this week. Beyhan, I reviewed your patch and modified some things. First of all the composite URLs are working. Second, I don't see that we should go with the version in the target name. If I install a target from our dialog e.g. the latest stable build I can receive an 1.4RC1 build. When I come back two month later and update my target I will get an 1.5M1 build. In this case the 1.4 will be still in the target name. To avoid this we decided to use this as target platform names: RAP Runtime - latest stable build RAP Runtime - latest release This also marks bug 346811 as unnecessary and I will close this bug soon. I committed the patch to CVS HEAD and the 1.4 maintenance branch. Thanks for the work. |