Bug 49652 - Support SPI: configuration management
Support SPI: configuration management
Status: RESOLVED FIXED
Product: projects
Classification: Unclassified
Component: Generic Projects UI
4.x
All All
: P3 (vote)
: 6.x
Assigned To: apireviews
issues@projects
http://wiki.netbeans.org/wiki/view/Ru...
plan60
: API, API_REVIEW_FAST
Depends on: 102029
Blocks: 41837 49636
  Show dependency treegraph
 
Reported: 2004-09-28 18:09 UTC by Jesse Glick
Modified: 2007-04-20 22:00 UTC (History)
6 users (show)

See Also:
Issue Type: ENHANCEMENT
:


Attachments
Proposed patch (30.59 KB, patch)
2006-06-15 21:40 UTC, Jesse Glick
Details | Diff
Currently proposed patch (173.46 KB, patch)
2006-08-31 21:08 UTC, Jesse Glick
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jesse Glick 2004-09-28 18:09:45 UTC
Would permit project types to easily show a
standard GUI for using configurations (run
profiles and the like).
Comment 1 Jesse Glick 2006-06-14 17:27:39 UTC
See issue #49636.
Comment 2 Jesse Glick 2006-06-15 20:58:17 UTC
I would like to submit the new API for review. See the linked-to proposal for
all details.
Comment 3 Jesse Glick 2006-06-15 21:40:07 UTC
Created attachment 31097 [details]
Proposed patch
Comment 4 Milos Kleint 2006-06-16 09:51:42 UTC
MK1: Is the active configuration meant to be persisted across IDE sessions?
MK2: Is it meant to be shareable or on per-user basis?
MK3: getActiveConfiguration() is meant to always return a profile. What happens
to the case (in j2se project) with the default values for Run project in project
customizer? are these supposed to be turned into a profile? Will the Run project
UI then act as editor for this default profile?

more general notes:
MK4: it seems to be fairly generic and I don't see a problem in implementing the
API in non-standard project types (eg. in maven), however I'd like to point out
that the major usecase seems to be "run project" profiles and with the given API
it's probably the only possible usecase. -> there's only one instance of
provider in lookup and only one profile can be the active one at a time. I think
this fact should be communicated to the project type providers to keep the UI
consistent across the IDE.
(by stating that in the documentation or by renaming the interfaces)
Comment 5 Jesse Glick 2006-06-16 15:00:55 UTC
Re. MK1 & MK2: yes, the choice of configuration is persisted per-user, in
nbproject/private/config.properties. The definition of the configurations can
mix sharable and per-user settings. See the URL for details on format.

Re. MK3: a j2seproject always has a <default> null profile. See the URL for details.

Re. MK4: yes, only one configuration can be active per project (at least as far
as this SPI is concerned). This does not necessarily restrict configurations to
being used only for run settings, though it does discourage orthogonal mixing of
configuration types since only kind of configuration can be selected using the
global UI. Java ME projects already use configurations to control both build and
run settings at once (not to mention the editor's appearance, etc.). See the URL
for details and discussion. Could you explain more how you would expect the
documentation to be changed to make this clear, or what is wrong with the
current interface names in this regard? Also, what (if anything) would you
consider using the SPI for in Maven projects?
Comment 6 Milos Kleint 2006-06-16 17:16:22 UTC
Ok, sorry I have not to read the doc you referenced. I've learned to ignore
anything above the comments section (a bad habit of mine)
here is the URL: http://projects.netbeans.org/nonav/buildsys/configurations.html

Re documentation: the design document should be referenced from the javadoc for
ConfirationProvider or from the changes document.
The setActiveConfiguration() method should point out the expected level of
persistance.
The interfaces rename idea can be disregarded. I thought the configs are to be
used exclusively for run profiling.

re MK4 maven - maven (at least the 2.x series) has the orthogonal profiles
implemented. These profiles can bound to OS, JDK, System properties etc. or
explicitly triggered. They can influence any part of the build lifecycle, adding
build steps, adding dependencies etc. That's something that cannot be modelled
by this UI/API. My thinking was along the lines of limiting the scope of
functionality to the scope provided in the default j2se/web/.. projects to get
the same UI feel.
Comment 7 Milos Kleint 2006-06-16 17:16:32 UTC
Ok, sorry I have not to read the doc you referenced. I've learned to ignore
anything above the comments section (a bad habit of mine)
here is the URL: http://projects.netbeans.org/nonav/buildsys/configurations.html

Re documentation: the design document should be referenced from the javadoc for
ConfirationProvider or from the changes document.
The setActiveConfiguration() method should point out the expected level of
persistance.
The interfaces rename idea can be disregarded. I thought the configs are to be
used exclusively for run profiling.

re MK4 maven - maven (at least the 2.x series) has the orthogonal profiles
implemented. These profiles can bound to OS, JDK, System properties etc. or
explicitly triggered. They can influence any part of the build lifecycle, adding
build steps, adding dependencies etc. That's something that cannot be modelled
by this UI/API. My thinking was along the lines of limiting the scope of
functionality to the scope provided in the default j2se/web/.. projects to get
the same UI feel.
Comment 8 Jesse Glick 2006-06-16 20:45:33 UTC
Re. Javadoc: good points, thanks, will do.

Re. Maven: first, as far as this SPI is concerned you need only deal with those
configuration types which are user-selectable. Maven will anyway apply the right
settings for the current OS, JDK, etc. So for the user-selectable configuration
types, we have three choices:

OrthoCfgTypes-1. Directly support orthogonal config types in the SPI. I am
hesitant to do this because the GUI would get more complicated and we don't
really need it for other project types that I know of. I could experiment with
it if it would really be valuable for someone.

OrthoCfgTypes-2. For a given Maven project, pick at most one type of config to
make selectable using the global GUI. But permit others to be selected e.g. from
the project's context menu or properties dialog.

OrthoCfgTypes-2a. You could even make the PCP impl for the Maven project just
show the configs of the type which was last changed by the full GUI. So as soon
as you e.g. pick your compiler config (debug? release? instrumented?) you would
see all compiler configs for the project listed in the combo box. If you later
on changed your run config you would see all run configs listed. Would satisfy
the basic problem, but UI might be just too weird.

OrthoCfgTypes-3. If there are not so many configs, and only a few _combinations_
are often used, you could show the cross product or some subset the user picks
in the Properties dialog.
Comment 9 ivan 2006-06-22 23:24:06 UTC
As I mentioned on a recent nbdev post what Eclipse calls Configurations are
a very different beast. (This is independent of whether their choice
of the word Configuration was a good idea :-)

Traditionally configurations (as in configuration management) had to do with
building activities and inasmuch as Projects produce artifacts and are
oriented towards synthesis associating Configurations with projects as
this proposal does makes sense.

However, Eclipse didn't do this. Why?
What Eclipse configurations capture is attributes having to do with running
things, not building them. The Motivation for this proposal also talk about
cmdline arguments and running.
As such strong association with projects doesn't make
sense and Eclipse Configurations are orthogonal.
In CND we call them RunProfiles.
In Eclipse they capture cmdline arguments, environment variables, choice of
engines etc. They are named and refer to (rather than owned by) an
executable artifact (project or individual class, or external executable).

As is usually the case with orthognality IMO the Eclipse usage of RunProfiles
is more flexible.


Comment 10 Thomas Preisler 2006-06-28 02:46:10 UTC
It looks fine. Besides name changes, it is basically the same API as the one we
are currently using. We got it from the Mobility team.

I have two issues:

Please don't use generics yet. We have a copy of the sources in CND so it works
with NB5.0 and NB5.5 and it is still a requirement that we compile CND with 4.1.

Why show the configurations in the combo box in alphabetical order? It should be
up to the implementer to sort them (or not), and the IDE should just show them
in the listed order. I'm not sure we (CND) want to show them in alphabetical order.
Comment 11 Jaroslav Tulach 2006-06-28 15:34:44 UTC
Y01 May I ask about threading policy of the interface? Can it fire and update 
its values in any threads? I have not read javadoc, but I found no mention 
about these problems anywhere in the docs or this issue.
Comment 12 Jesse Glick 2006-06-28 15:36:48 UTC
Re. generics - the code under review is intended for NB 6.0 which will require
JDK 5. If you plan to maintain a private copy of the API in the CND module, you
can erase generic type signatures if you want to. Anyway your code compiled
against your private API will not interoperate against the new public API unless
you do some weird tricks with java.lang.reflect.Proxy.

Re. alphabetization - this is already the case. j2seproject sorts its configs.
projectui displays them as they come.
Comment 13 Jesse Glick 2006-06-30 19:26:42 UTC
Y01 threading policy - probably needs definition in Javadoc and perhaps some
impl work. I am on vacation next week but I will look at it after that.
Comment 14 Jesse Glick 2006-08-21 22:46:23 UTC
Y01 solved in branch. Javadoc mandates use of ProjectManager.mutex, impl does it.
Comment 15 greggwon 2006-08-22 03:44:34 UTC
I'd prefer that runtime configurations be completely separate from projects.  I
have runtime configurations that are specific to different types of applications
and have very generic properties.  I'd like to be able to augment that generic
configuration with per project configuration such as 'run in directory' and 'jvm
arguments' as we have available now.  I'd also like to have the ability to use
properties substitution into the configuration values.
Comment 16 Jesse Glick 2006-08-22 19:42:31 UTC
greggwon: what you are talking about would probably be better handled via
freeform projects and some Ant scripting of your choice. Out of scope for this
RFE, I think.
Comment 17 ivan 2006-08-22 20:00:31 UTC
No Jesse, freeform projets and creative ant scripting won't do.
I'd like to insist that separation of concerns of building (synthesis)
and running (anlysis, since lots of running is under analytical tools like
debuggers and performance analyzers) be given serious considereation.

Right now if I want to run my app with one set of input and then
with another set of input, it's extremely awkward. Under Eclipse and
Sun Workshop one can create a named "runprofile" independent of 
build-related stuff and easily switch between them.

Using build configurations to manage alternative run scenarios won't do.
In the native environment build configurations span a 3-5 dimensional space,
spanning OS, architecture, compiler/ABI and debug level. It's
already hard to manage and adding a run-scenario dimension is IMO 
entirely impractical.

I appreciate that this discussion might be outside the scope of
this immediate rfe, but it needs to happen somewhere.
Comment 18 Jesse Glick 2006-08-22 20:36:47 UTC
ivan: you can already implement separate build and run configurations in any
project type, with no special support. This issue is about a *shared and global*
UI for managing per-project configurations. There is not a strong use case for
build configurations for Java SE projects, ME projects have configs that apply
to build and run, EE projects probably would be similar to either SE or ME, and
last I checked CND projects are using the same single-dimensional configuration
system that ME projects use. If and when several major project types decide they
need >1 independent dimension of configurations, then it would make sense to
extend this SPI in the obvious ways to support a shared global UI displaying
multiple configuration dimensions. But right now it looks unnecessary.
Comment 19 Jesse Glick 2006-08-29 00:42:32 UTC
I added a new SPI method,

boolean configurationsAffectAction(String command);
Comment 22 Jesse Glick 2006-08-31 21:07:54 UTC
I would like to conclude this review by Friday Sep 08 if there are not
objections to it by then. Will attach running patch for reference.
Comment 23 Jesse Glick 2006-08-31 21:08:48 UTC
Created attachment 33484 [details]
Currently proposed patch
Comment 24 Jesse Glick 2006-09-08 19:34:18 UTC
See issue #49636 for commit log.


By use of this website, you agree to the NetBeans Policies and Terms of Use. © 2012, Oracle Corporation and/or its affiliates. Sponsored by Oracle logo