Small fix in defjump #2

Merged
merged 1 commit into from Jun 29, 2012

4 participants

@vhallac

The macro expansion for the root parameter was broken. It caused rinari to fail
jumping.

Assuming I've used the new macro correctly on my rinari mods, the macro didn't detect the root of a project correctly. With this change, it looks OK.

@vhallac vhallac Fix the defjump macro root expansion
The macro expansion for the root parameter was broken. It caused rinari to fail
jumping.
948a607
@purcell
Collaborator

Eric - if you want to give me write access to the repo, I'm happy to commit this on your behalf and update the rinari submodule ref too.
-Steve

@milkypostman

Do you have this working on the latest rinari head?

@vhallac

Only on my fork of it. I haven't used RoR/rinari for a few months at least, so my fork is no longer in sync with the latest head.

@purcell
Collaborator

Hey @eschulte, can you click "Merge" on this please? Rinari HEAD won't work without it.

-Steve

@eschulte eschulte merged commit 9d31706 into eschulte:master Jun 29, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment