Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [epp-dev] Using p2.inf to optionally import Linux Tools features in CPP package

On 05/16/2012 02:32 PM, Markus Knauer wrote:
Looks good to me... Doug, any objections? Feel free to trigger a new
(juno.epp-repository-build) build in order to test this change as soon
as possible.

What's the bug number related to this change?


Good point.  Just opened one:

https://bugs.eclipse.org/bugs/show_bug.cgi?id=379734

Doug has approved so I'll push it and start a build.

-- Jeff J.

Thanks,
Markus

On Wed, May 16, 2012 at 8:27 PM, Jeff Johnston <jjohnstn@xxxxxxxxxx
<mailto:jjohnstn@xxxxxxxxxx>> wrote:

    If there are no objections or technical issues, I would like to
    apply the attached patch to the CPP package feature so that the
    Linux Tools features are optionally imported as opposed to included.
      The features were included because some of them have filters so
    they cannot be imported by some of the supported CPP platform/OS
    combinations.  This wasn't a problem when Linux Tools had its own
    EPP package and restricted supported platforms to Linux x86 and
    x86_64.  The import statement does not support use of an "optional"
    flag so an include statement was used.

    The change would have the benefit that those features imported this
    way should be able to be updated in the future.  Without this patch,
    those included features can only be updated by an update to the
    parent CPP package feature.

    IMO, this meshes better with the other features that are required by
    the CPP package (such as CDT features) that can be updated.

    -- Jeff J.

    _______________________________________________
    epp-dev mailing list
    epp-dev@xxxxxxxxxxx <mailto:epp-dev@xxxxxxxxxxx>
    https://dev.eclipse.org/mailman/listinfo/epp-dev




_______________________________________________
epp-dev mailing list
epp-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/epp-dev



Back to the top