Skip to content

Comments

Reference hex.pm package in instructions#14

Open
fhunleth wants to merge 1 commit intovaliot:masterfrom
fhunleth:readme-update
Open

Reference hex.pm package in instructions#14
fhunleth wants to merge 1 commit intovaliot:masterfrom
fhunleth:readme-update

Conversation

@fhunleth
Copy link

@fhunleth fhunleth commented Sep 3, 2022

I assume that since this has been published to hex.pm that you want people to use the hex package instead of the latest master branch commit of the GitHub repository.

@acrogenesis acrogenesis requested a review from alde103 September 3, 2022 22:52
@alde103
Copy link
Member

alde103 commented Sep 6, 2022

We definitely prefer to use hex.pm but this project requires elixir_cmake, which seems to have been abandoned for a while, we are using a fork of that project and as I remember hex doesn't accept GitHub dependencies, anyway I think it's worth doing a PR to elixir_cmake.

@fhunleth
Copy link
Author

fhunleth commented Sep 7, 2022

Ahh. That makes a lot of sense. Thanks for the explanation.

Yes, I agree that it would be nice to send a PR to elixir-cmake and see what happens. I hope that they're responsive now.

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.

2 participants