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

Feature: added EFC ASN.1 modules #24

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

rmwesley
Copy link

@rmwesley rmwesley commented Dec 4, 2024

Hello! This is my first PR, I hope I followed the proper etiquette.

Changes

In the first 3 out of the 5 commits, I:

  • Added 2 complete EFC bundles with release years set to 2015 and 2023, containing ASN.1 modules for:
    DSRC, EFC, CCC, LAC, AutonomousCharging, InfoExchange + dependencies (X509, RFC5035)
  • Added 2 partial EFC bundles (DSRC, EFC only) with release years set to 2014 and 2018
  • Added 2 partial EFC bundles (DSRC, EFC, CCC only) with release years set to 2009 and 2023

I then also did 2 other commits with some tests.
I added them to the test/ dir. Please tell me if this impacts the CI/CD pipeline. Should I have added my custom tests to res/ instead?

Description

For each of the 3 first commits, I:

  • Added the EFC ASN.1 files to pycrate_asn1dir/
  • Updated specdir.py
  • Manually ran python -m pycrate_asn1c.asnproc to recompile all the specs
  • Committed all the changes.

These bundles include DSRC, EFC, CCC, LAC, AutonomousCharging and
InfoExchange specs with dependencies, such as X509 and RFC5035.
These specs were obtained from the standards.iso.org/iso/ website.

The ISO 14906 DSRC ASN.1 specs sometimes use Windows code pages to
encode characters in comments. I replaced them for their ASCII
counterparts where I could to avoid issues.
Again, the ASN.1 specs were modified to only contain ASCII characters
(even in comments).
CCC imports from DSRC specs, so they are added as dependencies
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.

1 participant