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

packchk : remove WARNING M379 ? #216

Closed
jeromecoutant opened this issue Mar 16, 2022 · 7 comments
Closed

packchk : remove WARNING M379 ? #216

jeromecoutant opened this issue Mar 16, 2022 · 7 comments
Labels
packchk Issue is related to packchk

Comments

@jeromecoutant
Copy link
Contributor

With our current pdsc file describing B-U585I-IOT02A board, we got a M379 warning:

*** WARNING M379: xxx/STMicroelectronics.board_resources_B-U585I-IOT02A.pdsc (Line 52)
No example(s) found for Board '[STMicroelectronics] B-U585I-IOT02A'.

We don't plan to provide an example list for each board pdsc.

So proposition is to remove this warning in packchk and in
https://open-cmsis-pack.github.io/Open-CMSIS-Pack-Spec/main/html/packChk.html

@fred-r

@LMESTM
Copy link
Contributor

LMESTM commented Mar 16, 2022

@jeromecoutant
To be precise, we wouldn't want to deliver the example IDE projects within this packs, maybe a list of supported examples, but not included in the pack, might be useful (to be discussed).

@JonatanAntoni JonatanAntoni added the packchk Issue is related to packchk label Apr 6, 2022
@JonatanAntoni
Copy link
Collaborator

Hi @jeromecoutant, @LMESTM, @fred-r,

I think this warning can be discussed. Board specifications are typically provided as part of Board Support Packages (BSPs). And a BSP is expected to ship at least one example per board to give the user some sort of starting point.

You can supress this warning if required but I'd leave it in the tool.

Cheers,
Jonatan

@LMESTM
Copy link
Contributor

LMESTM commented Apr 6, 2022

@JonatanAntoni how would we supress the warning ?
We may want to decouple the projects deliveries from the board description part. Providing a single example might actually be a disappointment to the user who wants to get access to the full list of examples that can run on this board. It could be better to provide pointers to available examples maybe rather than one actual project ? (Also for which IDE anyway should the project be delivered ? and which IDE version ?)

@jkrech
Copy link
Member

jkrech commented Apr 6, 2022

If you run packchk -h it should tell:

 -x Mxxx              : Suppress Messages

If yo ask me the format in the future should be csolution.yml and not for a specific IDE.

@LMESTM
Copy link
Contributor

LMESTM commented Apr 6, 2022

@JonatanAntoni thanks for the command.
I still think this should be left optional to the provider of the BSP pack to deliver a project or not.
I guess that we can live with the -x option. @jeromecoutant ?

grasci-arm pushed a commit to ARM-software/devtools that referenced this issue Apr 26, 2022
…k#216)

* [projmgr] Update board and device schema requirements

* Address review comments
@thorstendb-ARM
Copy link
Collaborator

Does this option solve the ticket? Or do we need further discussion.

@jkrech
Copy link
Member

jkrech commented Sep 7, 2022

I suggest to close this specific issue. Conceptually we may want to think about a separate execution mode, where packchk provides feedback about the 'quality' / 'completeness' / 'use of features' of a Pack to cleanly separate between validation and enhancements.

@jkrech jkrech closed this as completed Sep 7, 2022
@thorstendb-ARM thorstendb-ARM removed their assignment Sep 7, 2022
@jkrech jkrech removed their assignment Sep 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packchk Issue is related to packchk
Projects
None yet
Development

No branches or pull requests

5 participants