Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 347346 - JavaScript IDE has Mylyn as a pre-enabled software site
Summary: JavaScript IDE has Mylyn as a pre-enabled software site
Status: CLOSED FIXED
Alias: None
Product: EPP
Classification: Technology
Component: javascript-package (show other bugs)
Version: 1.4.0   Edit
Hardware: PC Linux
: P3 minor (vote)
Target Milestone: 1.4.0   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-26 13:01 EDT by David Williams CLA
Modified: 2011-06-24 07:20 EDT (History)
1 user (show)

See Also:

Attachments
touchpoing configuration change - removing Mylyn repository (2.04 KB, patch)
2011-05-26 13:55 EDT, Markus Knauer CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description David Williams CLA 2011-05-26 13:01:13 EDT 
Its probably always been this way, and I just never noticed, but I see Mylyn in the list of "pre-enabled" software sites for JavaScript IDE (RC2). 

But, Mylyn is not part of the JavaScript IDE. (and, before you ask .. yes, that's by design :) ... I left it out to minimize install ... until end users would ask for it, and none have). 

I know for Java EE IDE (and others) Mylyn is included as a pre-enabled software site, so that any post-release fixes could be (more) easily found ... but, seems not required if Mylyn's not included in the install in the first place. 
 
I don't think it hurts too much ... maybe a tiny performance impact when checking for other updates? ... so listing as minor. Would be fixing if a "one line fix" somewhere ... but, not if it takes some restructuring.
Comment 1 Markus Knauer CLA 2011-05-26 13:55:01 EDT 
Created attachment 196689 [details]
touchpoing configuration change - removing Mylyn repository

Actually it is not a one-liner, it is a 4-line fix.
Patch applied to CVS HEAD, should be in RC3 next week.
Comment 2 David Williams CLA 2011-05-26 15:36:33 EDT 
(In reply to comment #1)
> Created attachment 196689 [details]
> touchpoing configuration change - removing Mylyn repository
> 
> Actually it is not a one-liner, it is a 4-line fix.
> Patch applied to CVS HEAD, should be in RC3 next week.

Thanks Markus, that'd be great.
Comment 3 Markus Knauer CLA 2011-06-24 07:20:38 EDT 
Closing as FIXED.