org.apache.maven.DefaultMaven |
Line |
should all the logging be left to the CLI? |
179 |
Really should fail if it was not? What if it is aggregating - eg "ear"? |
506 |
should we now include the pom.xml in the current directory? String includes = System.getProperty( "maven.reactor.includes", "**/" + POMv4 ); String excludes = System.getProperty( "maven.reactor.excludes", POMv4 ); |
1092 |
org.apache.maven.MavenArtifactFilterManager |
Line |
configure this from bootstrap or scan lib |
63 |
org.apache.maven.cli.AbstractConsoleDownloadMonitor |
Line |
can't use getLogger() because this isn't currently instantiated as a component |
60 |
org.apache.maven.cli.CLIManager |
Line |
introducing a space here...not sure what else to do but collapse whitespace |
277 |
org.apache.maven.cli.MavenCli |
Line |
we need to do some more work here. Some plugins use sys out or log errors at info level. Ideally, we could use Warn across the board |
291 |
Additionally, we can't change the mojo level because the component key includes the version and it isn't known ahead of time. This seems worth changing. |
294 |
[BP]: do we set one per mojo? where to do it? |
515 |
[BP]: doing this here as it is CLI specific, though it doesn't feel like the right place (likewise logger). |
585 |
release |
605 |
something in plexus to show all active hooks? |
606 |
org.apache.maven.execution.MavenSession |
Line |
make this the central one, get rid of build settings... |
51 |
org.apache.maven.extension.DefaultExtensionManager |
Line |
this could surely be simpler/different on trunk with the new classworlds |
187 |
org.apache.maven.lifecycle.DefaultLifecycleExecutor |
Line |
This is dangerous, particularly when it's just a collection of loose-leaf projects being built within the same reactor (using an inclusion pattern to gather them up)... |
134 |
probably don't want to do all this up front |
177 |
MNG-4081. What about extensions within the current reactor?? |
186 |
shouldn't hit this, investigate using the same resolution logic as otheres for plugins in the reactor |
466 |
Test this! It should be fixed, but I don't want to destabilize until I have the issue I'm working on fixed. |
679 |
check ID is correct for reports if the POM configured no reports, give all from plugin |
940 |
this is all kind of backwards from the POMM. Let's align it all under 2.1. We should create a new lifecycle executor for modelVersion >5.0.0 |
1177 |
if moved to the plugin manager we already have the descriptor from above and so do can lookup the container directly |
1435 |
if moved to the plugin manager we already have the descriptor from above and so do can lookup the container directly |
1470 |
MNG-4081...need to flush this plugin once we look at it, to avoid using an external version of a plugin when a newer version will be created in the current reactor... |
1538 |
MNG-4081...need to flush this plugin once we look at it, to avoid using an external version of a plugin when a newer version will be created in the current reactor... |
1580 |
The methods and fields below are only needed for products like Hudson, that provide their own LifecycleExecutor and component configuration that extend default implementation, and which may become out-of-date as component requirements are updated within Maven itself. |
1993 |
org.apache.maven.plugin.DefaultPluginManager |
Line |
since this is only used in the lifecycle executor, maybe it should be moved there? There is no other use for the mapping manager in here |
157 |
this should be possibly outside All version-resolution logic has been moved to DefaultPluginVersionManager. |
175 |
this might result in an artifact "RELEASE" being resolved continuously FIXME: need to find out how a plugin gets marked as 'installed' and no ChildContainer exists. The check for that below fixes the 'Can't find plexus container for plugin: xxx' error. |
193 |
Should we error out, or simply warn and skip?? |
420 |
the mojoDescriptor should actually capture this information so we don't get this far |
655 |
plexus changes to make this more like the component descriptor so this can be used instead PlexusConfiguration mergedConfiguration = mergeConfiguration( pomConfiguration, mojoDescriptor.getConfiguration() ); |
709 |
I defy anyone to find these messages in the '-X' output! Do we need a new log level? ideally, this would be elevated above the true debug output, but below the default INFO level... [BP] (2004-07-18): need to understand the context more but would prefer this could be either WARN or removed - shouldn't need DEBUG to diagnose a problem most of the time. |
1033 |
this should be built in to the configurator, as we presently double process the expressions |
1050 |
Is this the right thing to do? |
1198 |
recessive seems to be dominant here? |
1278 |
shouldn't be necessary |
1305 |
could the configuration be passed to lookup and the configurator known to plexus via the descriptor so that this meethod could entirely be handled by a plexus lookup? |
1342 |
such a call in MavenMetadataSource too - packaging not really the intention of type |
1480 |
we don't need to resolve over and over again, as long as we are sure that the parameters are the same check this with yourkit as a hot spot. Don't recreate if already created - for effeciency, and because clover plugin adds to it |
1484 |
the packaging could be different, but the exception doesn't contain that info most likely it would be produced by the project we just found in the reactor since all the other info matches. Assume it's ok. |
1549 |
org.apache.maven.plugin.MavenPluginCollector |
Line |
throw an (not runtime) exception if there is a prefix overlap - means doing so elsewhere we also need to deal with multiple versions somehow - currently, first wins |
65 |
org.apache.maven.plugin.PluginParameterExpressionEvaluator |
Line |
don't catch exception |
219 |
don't catch exception |
245 |
don't catch exception |
273 |
without #, this could just be an evaluate call... |
319 |
org.apache.maven.plugin.PluginParameterExpressionEvaluatorTest |
Line |
used to be SCOPE_COMPILE, check |
296 |
org.apache.maven.plugin.version.DefaultPluginVersionManager |
Line |
[jc] Revisit to remove this piece of state. PLUGIN REGISTRY MAY BE UPDATED ON DISK OUT-OF-PROCESS! |
77 |
Revisit to remove this piece of state. PLUGIN REGISTRY MAY BE UPDATED ON DISK OUT-OF-PROCESS! |
84 |
check the GUI-friendliness of this approach to collecting input. If we can't port this prompt into a GUI, IDE-integration will not work well. |
392 |