-
Notifications
You must be signed in to change notification settings - Fork 557
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
Can't compile EXTERNAL type #294
Comments
You could also try these forks https://github.com/mouse07410/asn1c/tree/master |
I am struggling with both of those forks. Mouse07410 master gives:
And the brchiu branch compiles but when run on the snippet above gives:
If you manage to rebase an open a PR I would be very grateful. Thanks! |
Here it is #295 The EXTERNAL is tested in asn1c/tests/tests-asn1c-compiler/159-EXTERNAL-OK.asn1 Lines 1 to 4 in 6dba416
asn1c/tests/tests-c-compiler/check-src/check-159.c Lines 122 to 126 in 6dba416
And the Travis builds |
@velichkov could I ask you to rebase and submit your PR against https://github.com/mouse07410/asn1c/tree/vlm_master ? I tried to apply your PR #295 as-is, but got several conflicts that I can't resolve on my own. :-( I thought support for
P.S. https://github.com/mouse07410/asn1c/tree/vlm_master is the current default of my fork, and should be used for all the work. https://github.com/mouse07410/asn1c/tree/master is old, doesn't have the latest updates and fixes merged, and does not have OER support. |
Here it is mouse07410#36
The way the tests are organized, makes supporting multiple forks very difficult. |
Thank you!!
Is there anything we can do about it? Say, one big radical change that would make life easier down the road? |
I'm trying to compile the following:
And I get
What am I doing wrong?
The text was updated successfully, but these errors were encountered: