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

Refactor/reuse sapi parser on zgw #49

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

lmolina
Copy link

@lmolina lmolina commented Dec 2, 2024

Change

The script to decode the SAPI logs is being reused by the ZGW project. This PR adds legal information to the script, as well as a reference to the location in the ZGW project.

Related-to: SiliconLabs/zipgateway#30

Checklist

The project ZGW is reusing the script serial_decode_zpc.py. Adding the
SPDX headers to have a clear status on the license and copyright of the
script across projects.
The Z/IP Gateway is reusing the script to parse SAPI logs. The script
was adjusted to be compatible with the ZGW logs. Adding a reference to
the ZGW copy on the original script.

Related-to: SiliconLabs/zipgateway#30
@lmolina
Copy link
Author

lmolina commented Dec 2, 2024

I'm not sure if one can say that Silicon Labs has an agreement with Silicon Labs. Checking the box since this is a contribution as part of my activities within Silicon Labs.

@rzr
Copy link
Collaborator

rzr commented Dec 2, 2024

This should be contributed internally at uic

@@ -2,6 +2,9 @@
# SPDX-License-Identifier: LicenseRef-MSLA
# SPDX-FileCopyrightText: Silicon Laboratories Inc. https://www.silabs.com
'''
The script was replicated and updated to work with Z/IP Gateway
Copy link
Collaborator

Choose a reason for hiding this comment

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

no need to add in source (we can rely in commit message)

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.

2 participants