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

symlink mib hrl output in apps `include' directories #967

Merged
merged 2 commits into from
Dec 8, 2015

Conversation

talentdeficit
Copy link
Contributor

this restores compatibility with rebar2 and erlang.mk

the fix is somewhat of a hackjob but without refactoring rebar_base_compiler pretty substantially there's no good way to pass the mib compiler the extra information it needs

this restores compatibility with rebar2 and erlang.mk
@ferd
Copy link
Collaborator

ferd commented Dec 7, 2015

+1

@goertzenator
Copy link

This PR puts the MIB include file into 2 locations, the application include and priv/mibs/include. Is there good reason to place it into priv/mibs/include? Perhaps @toland from PR #948 could comment on the prevalence priv/mibs/include. If there really are projects in the wild that use priv/mibs/include, then +1 to this.

When I learned rebar3, unraveling what was going on in the _build dir was frustrating. I write this comment thinking of the future me's who will see the 2 include instances and wonder what's going on.

@talentdeficit
Copy link
Contributor Author

i am ok with just putting the header file in include but i know nothing about mibs or how they are used so i'm inclined to just do whatever is suggested

ideally someone would write a decent mibs provider plugin and we could drop mibs support from rebar3

@toland
Copy link
Contributor

toland commented Dec 7, 2015

FWIW, I was just trying to preserve what I thought was the original author's intent. I have always put the hrl files in include, but I thought maybe the the original author knew something I don't :)

My vote would be to drop the hrl files in the project include directory and be done with it.

@talentdeficit
Copy link
Contributor Author

i've altered this pr to remove the header file in priv/mibs/include and to just move it directly to the applications include directory

@ferd
Copy link
Collaborator

ferd commented Dec 8, 2015

Still +1.

@goertzenator
Copy link

Looks good! +1.

@toland
Copy link
Contributor

toland commented Dec 8, 2015

+1

ferd added a commit that referenced this pull request Dec 8, 2015
symlink mib hrl output in apps `include' directories
@ferd ferd merged commit 21ae314 into erlang:master Dec 8, 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.

None yet

4 participants