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

Configure the ASN.1 compiler differently for different ASN.1 files #8

Closed
mhssler opened this issue Sep 7, 2017 · 9 comments · Fixed by #18
Closed

Configure the ASN.1 compiler differently for different ASN.1 files #8

mhssler opened this issue Sep 7, 2017 · 9 comments · Fixed by #18

Comments

@mhssler
Copy link

mhssler commented Sep 7, 2017

I need the possibility to compile ASN.1 files with different settings within the same application.

Are you interested in a pull request for this? I have attached an updated README.md to sketch the user interface.

@knusbaum
Copy link
Owner

knusbaum commented Sep 8, 2017

@mhssler Thank you! This looks like a useful feature, and the interface is straight-forward.
Please open a PR if you are interested in contributing it to the repo.

@mhssler
Copy link
Author

mhssler commented Sep 11, 2017

I will give it a try.

@mhssler
Copy link
Author

mhssler commented Sep 13, 2017

When I started with this, I noticed that this provider only uses the top rebar.config file. Is this intentional or a missing feature?

@knusbaum
Copy link
Owner

@mhssler Not sure what you mean 'top rebar.config'. I've only ever used one rebar.config in a project.

@mhssler
Copy link
Author

mhssler commented Nov 3, 2017

$ tree
.
├── lib
│   └── app
│       ├── rebar.config  <-- Application rebar.config
│       └── src
│           ├── app.app.src
│           └── app_app.erl
└── rebar.config   <--- Top rebar.config

And no, this provider does not use the "application rebar.config" when there is a "top rebar.config". I have a fix for that which I am trying to test.

@knusbaum
Copy link
Owner

Ahh, I see now. I would hope that rebar provides the tools to do that easily.

@ILGSupreme
Copy link

ILGSupreme commented Oct 18, 2018

Any updates on this issue? @knusbaum

@knusbaum
Copy link
Owner

@ILGSupreme I have not worked on this issue, as my time is heavily occupied at the moment.
I'd welcome any PRs for this, though, if you feel you want to mess with it.

@ILGSupreme
Copy link

I made a fork with a "smart" solution to it. I might put some time into the code to see if there is a good way of doing it.

sebastiw added a commit to sebastiw/provider_asn1 that referenced this issue Nov 19, 2022
CLOSES: knusbaum#8

Option to override configuration per file or match pattern.
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 a pull request may close this issue.

3 participants