Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
[m2e-dev] sonatype / m2eclipse-tycho change request

Igor:

this is a sonatype / m2eclipse-tycho change request;

1) here
https://github.com/sonatype/m2eclipse-tycho/blob/master/org.sonatype.tycho.m2e/src/org/sonatype/tycho/m2e/internal/OsgiBundleProjectConfigurator.java

you have a little snippet:

private void generateBundleManifest( ProjectConfigurationRequest request, IProgressMonitor monitor )
        throws CoreException
    {
        List<MojoExecution> executions = getMojoExecutions( request, monitor );
     &nb sp;  if ( executions.size() != 1 )
        {
            throw new IllegalArgumentException();
        }
    }

which is of grave concern to me :-)

2) I have a legitimate case, with felix embedder, where I invoke manifest 2 times:
a) first to make a host manifest, which is used at run time to generate org.osgi.framework.system.packages
b) second to make normal manifest which is used for dependency

3) this "executions.size() != 1" results in bunch of projects blowing up eclipse
with mysterious exceptions on pretty much every operation, clean/update etc;
the work around is bunch needless of maven profile hackery;

4) do you think you could make your approach a bit more relaxed?
say, if there is more then 1 execution, first look for default, say "id=default-manaifest" or "id=default-bundle"
at last, look for "id=eclipse-manifest", and finally, do not blow up, just log a normal error
and do not generate any manifest at all?

5) there are few more places with "throw new IllegalArgumentException(); "; could relax there too? :-)

Thank you,

Andrei


Back to the top