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

Bug: CMSIS packs toolbar is shown in all perspectives #487

Closed
marcel-kanter opened this issue Feb 22, 2021 · 11 comments
Closed

Bug: CMSIS packs toolbar is shown in all perspectives #487

marcel-kanter opened this issue Feb 22, 2021 · 11 comments
Assignees
Labels
Milestone

Comments

@marcel-kanter
Copy link

The Embedded Tools install the CMSIS packs package. If you install the embedded tools into an eclipse with other perspectives, the "Make CMSIS packs perspective" tool bar is visible in ALL perspectives AND the main all-perspective-closed view of eclipse.

This is a bit anoying to have a toolbar floating around that should simply show an perspective, especially when it does not work from the all closed view. Please remove this toolbar.

Additionally the name in the toolbar configuration is not proper and shows the internal name.

@marcel-kanter marcel-kanter changed the title Bug: CMSIS packs menu entry is shown in all perspectives Bug: CMSIS packs toolbar is shown in all perspectives Feb 22, 2021
@ilg-ul
Copy link
Contributor

ilg-ul commented Feb 22, 2021

Can you provide some images and some more details?

@marcel-kanter
Copy link
Author

grafik

  • Verify "2 items selected" right below the selecting box

  • Click "Next", Click "Next", Choose "Accept the terms of license aggreement" and click "Finish"

  • Wait for the packets to get downloaded and installed (process bar is in the bottom right by default)

  • Click "Restart Now" when it asks for the restart

  • Wait for eclipse to re-open

  • See the "Make CMSIS Packs perspective visible toolbar" in Resource Perspective

grafik

  • Click on The Button and the perspective gets opended besides the fact that it was not selected and is officially not installed as package

  • Verify this by clicking on "Help" -> "About Eclipse Platform", click on "Installation Details"

grafik

  • Click on close of the opened dialogs and close the editor or what this is in the cmsis packs perspective

grafik

  • Click on "Window" -> "Perspective" -> "Close All Perspectives"

The Toolbar is still there

grafik
grafik

  • Click on the Icon "Make CMSIS Packs Perspective visible" and nothing happens.

(I dont know how to make a screen shot of something that does not show up)

To summarize:

The jar files

Eclipse Embedded CDT	Embedded C/C++ CMSIS Packs Core Plug-in	3.0.0.202012080907	org.eclipse.embedcdt.packs.core
Eclipse Embedded CDT	Embedded C/C++ CMSIS Packs UI Plug-in	2.0.0.202012080907	org.eclipse.embedcdt.packs.ui

are installed even when CMSIS packs are not selected for install. You can find the versions in "Help" -> "About Eclipse Platform" -> "Installation Details" -> "Plug-Ins". Or by extraction of the jar files in the "plugins" subfolder where you have extracted the zip file with the platform.

This tab of the this dialog basically shows all versions of all jar files in the plugins/dropins folder.

The toolbar isnt working when all perspectives are closed

This may be inteded or a bug.

The toolbar is added all perspectives"

To verify this:

  • Click on "Window" -> "Perspective" -> "Open Perspective" -> "Other"
  • Choose "Team synchronizing"

The toolbar is there
grafik

  • Repeat the steps for "Resource (default)" perspective or any other perspective already created.

grafik

The localization for the Toolbar seems to be missing or the description is missing

  • Click on "Window" -> "Perspective" -> "Customize Perspective"

grafik

@ilg-ul
Copy link
Contributor

ilg-ul commented Feb 25, 2021

Thank you for the detailed bug report, it should be enough to reproduce the problem.

At first sight there might be multiple small issues, like unwanted dependencies, or missing conditions.

I'll investigate.

@ilg-ul
Copy link
Contributor

ilg-ul commented Mar 1, 2021

I more or less reproduced the problem.

However, in two tests, one on Windows and one on macOS, the initial view of the CMSIS Packs perspective was correct, it showed an empty window, instructing to press the Refresh button. I guess you already had an AppData\Roaming\CMSIS-Packs folder, possibly with inconsistent content. I suggest you rename/remove the CMSIS Packs folder, and reinstall the Eclipse + plug-ins from scratch.

Putting this aside, I summarise your report as several separate issues:

  • the Windows -> Perspectives -> Close All Perspectives leaves the Packs button enabled in the toolbar, and this button is not functional, it does not reopen the perspective; to reopen it is necessary to do Windows -> Perspectives -> Open Perspective
  • the Packs perspective should not be visible if the Embedded C/C++ CMSIS Packs (Experimental) feature is not installed
  • the perspective description is missing and replaced by showPerspective

Can you confirm this?

@ilg-ul
Copy link
Contributor

ilg-ul commented Mar 1, 2021

The missing label was fixed as the separate #488.

New logic was added to the toolbar button to make it visible only for C/C++ projects (having the org.eclipse.cdt.core.cnature nature).

Unfortunately the current code in the managed build (Arm & RISC-V) UI plug-ins have a dependency on the packs UI plug-in, which also display the CMSIS Packs perspective, and hiding this perspective completely requires moving the perspective code to a separate plug-in.

I created #489 to track this issue, to be addressed in a future release.

Time is too short now before 2021-03, these changes will be part of 6.1.3, to be included in the 2021-06 release.

@ilg-ul
Copy link
Contributor

ilg-ul commented Mar 1, 2021

A preview of the next 6.1.3 is available for testing from:

Please install it and let me know if this works for you.

@marcel-kanter
Copy link
Author

I more or less reproduced the problem.

However, in two tests, one on Windows and one on macOS, the initial view of the CMSIS Packs perspective was correct, it showed an empty window, instructing to press the Refresh button. I guess you already had an AppData\Roaming\CMSIS-Packs folder, possibly with inconsistent content. I suggest you rename/remove the CMSIS Packs folder, and reinstall the Eclipse + plug-ins from scratch.

Putting this aside, I summarise your report as several separate issues:

* the Windows -> Perspectives -> Close All Perspectives leaves the Packs button enabled in the toolbar, and this button is not functional, it does not reopen the perspective; to reopen it is necessary to do  Windows -> Perspectives -> Open Perspective

* the Packs perspective should not be visible if the **Embedded C/C++ CMSIS Packs (Experimental)** feature is not installed

* the perspective description is missing and replaced by _showPerspective_

Can you confirm this?

Yes. As you described. BTW I do not have any packs installed. So the packs view maybe empty correctly.

(I'm a bit busy these days to excuse my late replies)

@ilg-ul
Copy link
Contributor

ilg-ul commented Mar 1, 2021

No problem, the develop/p2 site will remain available, so please test it when possible.

@marcel-kanter
Copy link
Author

So far so good. It seems to work now. Name is shown correctly and it is not appearing on all perspectives when no CMSIS packs packet is installed. The CMSIS Packs perspective still can be opened (as per #489)

The logic with the project (action sets) seems to work - even when switching between general purpose project type and c/c++ project. But the customize dialog shows a bit strange Tool-Tip: (Cosmetics)

grafik

@ilg-ul
Copy link
Contributor

ilg-ul commented Mar 20, 2021

Ok, thank you for confirming.

@ilg-ul
Copy link
Contributor

ilg-ul commented Mar 20, 2021

Fixed on 2021-03-01, in 6.1.3.

@ilg-ul ilg-ul closed this as completed Mar 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants