Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JBIDE-17808: Mechanism to define and detect "families" of IU #280

Merged
merged 1 commit into from Aug 29, 2014
Merged

JBIDE-17808: Mechanism to define and detect "families" of IU #280

merged 1 commit into from Aug 29, 2014

Conversation

mickaelistria
Copy link
Contributor

  • Plugins can contribute a "family" definition (such as earlyaccess,
    supported,
    blacklisted...) which lists the various ius for the given family
  • A utility class allows to do request to compare current installation
    with families
  • Central editor shows when Early Access content is installed (computed
    from family)
  • An installation page is added to report Early Access IUs

point="org.eclipse.ui.installationPages">
<page
class="org.jboss.tools.central.installation.EarlyAccessInstallationPage"
id="org.jboss.tools.central.installtion.earlyaccessPage"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

installtion is not spelled correctly.

@mickaelistria
Copy link
Contributor Author

I pushed an amended version of the commit that applies the various suggestions you've made.

<extension
point="org.eclipse.ui.installationPages">
<page
class="org.jboss.tools.central.installation.EarlyAccessInstallationPage"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lets call this page CentralInstallationPage since it will be more than earlyaccess

@maxandersen
Copy link
Member

if you do those tiny label renames for the about install box I'm +1 to get this in. the mechanism is sound and we can extend it over time.

* Plugins can contribute a "family" definition (such as earlyaccess,
supported,
blacklisted...) which lists the various ius for the given family
* A utility class allows to do request to compare current installation
with families
* Central editor shows when Early Access content is installed (computed
from family)
* An installation page is added to report Early Access IUs
@mickaelistria
Copy link
Contributor Author

PR updated:

  • Excepton were turned in IOException
  • null stream exits method earlier
  • Labels and class names for installation don't reference early-access any more

@mickaelistria
Copy link
Contributor Author

Note that I don't have the power to merge this change. So @dgolovin or @maxandersen , I let you take care of it when you're ready.

@dgolovin dgolovin merged commit 9aea48d into jbosstools:master Aug 29, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants