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

update config files from asterix-specs #6

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

Conversation

zoranbosnjak
Copy link

This pull request is split into individual commits, for easy review:

  • xml config files update script
  • updated files (new editions)
  • updated files (existing editions)

The script was prepared and tested some time ago with the related 'asterix' project. Since this project uses the same xml structure, it could as well benefit from automatic config file conversion.

@rez10191
Copy link

rez10191 commented Jun 14, 2022

@zoranbosnjak This is great, it really improves the utility of this project.

Related to the original licensing discussion, it appears that asterixjson2xml.py and update-specs.py are sourced from the asterix project - asterixjson2xml.py, update-specs.py with only minor modifications. Since the asterix project is licensed under the GPL and this is an MIT licensed project, I'm not sure @filipjonckers will want it directly sourced in this project.

Something of note as well: I think this pull request would address issue #5 and address @bernad1's request in issue #4. Perhaps @BdonU can add a CAT033 ast file to asterix-specs and make a push request to resolve the original concern in issue #4 as well.

@JamMarHer
Copy link

Hi, any updates on this?

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.

3 participants