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

Stray file Makefile.am in rebar3 release #368

Open
leoliu opened this issue Feb 25, 2019 · 5 comments
Open

Stray file Makefile.am in rebar3 release #368

leoliu opened this issue Feb 25, 2019 · 5 comments

Comments

@leoliu
Copy link
Contributor

leoliu commented Feb 25, 2019

I now use Yaws as a rebar3 dep and after building a release the yaws-2.0.6 directory looks like this:

├── ebin
├── include
└── priv

all these subdirectories have a file Makefile.am which is unusual.

The issue can be reproduced with branch master or rebar3-support.

@vinoski
Copy link
Collaborator

vinoski commented Feb 25, 2019

Running rebar3 release doesn't work for me — I assume because of the lack of a relx section in rebar.config — but running rebar3 compile partly duplicates this problem, as after it finishes I see a couple Makefile.am files under the _build directory. I don't see any way to avoid having rebar3 copy those files, but a post hook like the one below added to rebar.config can remove them:

{post_hooks, [{"(linux|solaris|freebsd|darwin)", compile,
               "find $REBAR_BUILD_DIR -name Makefile.am -exec rm -f {} \\;"}]}.

@leoliu
Copy link
Contributor Author

leoliu commented Feb 25, 2019

I just noticed these files are in the repo i.e. version-controlled. So maybe ignore this issue for now. It's harmless anyway.

@vinoski
Copy link
Collaborator

vinoski commented Feb 25, 2019

Fair enough, but for now I've pushed a commit to run find over the build area after compilation as mentioned earlier in this issue.

@leoliu
Copy link
Contributor Author

leoliu commented Feb 25, 2019

That could work. But...

is it safe to run it inside _build? It will make uncommitted/untracked changes to deps. Ideally it should only be run inside the rel directory or, even better, yaws-VERSION directory in the rel directory.

@vinoski
Copy link
Collaborator

vinoski commented Feb 25, 2019

I convinced myself a bit ago that the find addition wasn't really helpful, so I just pushed another change to remove it.

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

No branches or pull requests

2 participants