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

Problems with Generator support in Eclipse CMSIS pack manager plugin #110

Closed
edriouk opened this issue Oct 27, 2023 · 3 comments
Closed
Assignees

Comments

@edriouk
Copy link
Contributor

edriouk commented Oct 27, 2023

NXP has reported problems in the Eclipse CMSIS pack manager plugin used in the MCUXPresso IDE and also IAR. After importing the pack into the MCUXpresso IDE in CMSIS plugin perspective, there is possible to “enable” the generator component and there is the green arrow button that can be pressed:

The generator batch file launches, but it seems that parameters are passed in incorrect way.
It seems that the parameters are “interleaved” with empty strings (every odd parameter is “”).

Also the files referenced in the GPDSC are not added into the project. Maybe there are some issue with the path as we are not sure, what should be the reference root for the files mentioned in the .gpdsc

@edriouk
Copy link
Contributor Author

edriouk commented Oct 27, 2023

There are several problems with the issue:

  1. Wrong processing of generator arguments: an empty argument always added regardless if "switch" attribute exists.
  2. Working directory is not prepended to the gpdsc file name.
  3. There is an NPE caused by 2. : parent directory is null and needs special processing
  4. gpdsc file is not fully processed : only components get added, not project files
    (because they outside of the project directory).

edriouk pushed a commit that referenced this issue Oct 27, 2023
Fix to #110 Problems with Generator support in Eclipse CMSIS pack manager plugin
@edriouk
Copy link
Contributor Author

edriouk commented Oct 27, 2023

CmsisPackPlugIn2.9.0_patch.zip
The issue is fixed and needs to be verified.

@edriouk edriouk pinned this issue Oct 27, 2023
@edriouk edriouk self-assigned this Oct 27, 2023
@edriouk
Copy link
Contributor Author

edriouk commented May 21, 2024

Fixed in 2.10.0

@edriouk edriouk closed this as completed May 21, 2024
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

No branches or pull requests

1 participant