-
-
Notifications
You must be signed in to change notification settings - Fork 57
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
Romefile RepositoryMap does not work with github references #7
Comments
To clarify, I understand that your scenario is as follows: In your Cartfile.resolved:
But your framework name is actually not Awesome-cat-names but something else. Correct? |
Please try pre-release https://github.com/blender/Rome/releases/tag/v0.4.0.4 |
@erichoracek any feedback? |
This build appears to have fixed the issue—thanks again for the responsiveness! Everything now seems to be working beautifully ✨ |
Great! I will make one more change to the Romefile format to align it with the INI file format and then make a stable realease. This will require changing the old Romefile. Sorry for that. |
@erichoracek v0.4.0.5 with GitHub reference and Romefile ini is out. Please try, beware that the format of the Romefile has changed.
Please see Romefile section in README.md |
It would appear that the Romefile repository map doesn't work with github references—only with git references. It seems to have to do with the
deriveFrameworkName
function at https://github.com/blender/Rome/blob/master/src/Lib.hs#L192, which doesn't appear to be using theromeMap
to do a repo to framework lookup. This results in theRomefile
mappings being unused when they are mapping from a github repo name to a framework name.The text was updated successfully, but these errors were encountered: