Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 325066 - reconsider customizeAccessRules
Summary: reconsider customizeAccessRules
Status: RESOLVED FIXED
Alias: None
Product: WTP Releng
Classification: WebTools
Component: releng (show other bugs)
Version: 3.10   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 3.10.0   Edit
Assignee: webtools.releng CLA
QA Contact: David Williams CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-12 23:37 EDT by David Williams CLA
Modified: 2018-06-29 15:30 EDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description David Williams CLA 2010-09-12 23:37:10 EDT
In WTP builds, we have long had a custom task to take into account how a project had set their access rules. See 
http://wiki.eclipse.org/WTP:_Consumer_Control_of_Access_Rules

The purpose of this was so that WTP committers, as consumers of other parts of WTP, could "permit" themselves to use other parts of WTP without warning, so there would be no warnings about "discouraged access" for other WTP projects, with the whole goal of focusing attention on "discouraged access" from non-WTP projects (which are much more serious). 

I think its time to reconsider. I don't think anyone is particularly making use of this nice feature ... and it does add a little overhead to the build, Not much, but maybe 20 minutes of a 2 hour build, so worth reconsidering, if it is not being useful. 

As a first step, I'll simply remove the "defaults" of "org.eclipse.wst.*" and "org.eclipse.jst.*". 

Then later (a few days? weeks?) I'll add a variable so it can be turned on or off for any particular build or component, and set the default to off. 

And see if anyone notices . :) 

Perhaps more "friend" relationships should be defined? Across WTP projects?
Comment 1 David Williams CLA 2011-03-05 21:20:14 EST
I've refactored stuff, and added a variable, and now default is "off". 

Should save a few minutes. 

Currently the variable 
customizeAccessRules 
would have to be defined in build.cfg (or, not sure, build.properties might work?) 
We can make or test this to be more refined later, if anyone wants it turned back on, so it can be controlled component by component.