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

check for mib include file #909

Closed
wants to merge 1 commit into from

Conversation

goertzenator
Copy link

When a mib is compiled, check that the include file is created and placed in the right location.
Test added for both normal and umbrella projects.

Both these tests currently fail, and IIRC for different reasons.

This is the replacement PR for the unmergable #873

When a mib is compiled, check that the include file is created and placed in the right location.
Test added for both normal and umbrella project.
@talentdeficit
Copy link
Contributor

i will look into this tomorrow or this weekend

@talentdeficit
Copy link
Contributor

@goertzenator could you take a look at this issue and let us know if it's relevant to this issue? it's a possible dupe but i don't understand mibs well enough to know if the header should also be present in the include dir

@talentdeficit
Copy link
Contributor

i've attempted to solve this issue with #967 by linking the hrl file in priv/mibs/include into the app dir's include file

the mib compiler should probably be broken out into it's own provider/plugin eventually, but hopefully this addresses the issue for now

@talentdeficit
Copy link
Contributor

@goertzenator @tsloughter this is good to close now, right?

@goertzenator
Copy link
Author

Yes, please close.

@ferd ferd closed this Dec 10, 2015
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