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

Submodules #45

Closed
wants to merge 2 commits into from
Closed

Submodules #45

wants to merge 2 commits into from

Conversation

joliss
Copy link
Contributor

@joliss joliss commented Apr 26, 2012

I actually just a few minutes ago needed to poke into the Ember code that was running inside my Rails app, so it was easiest to change ember-rails app to rebuild its assets from the sources. (And it's working great.) Here's the patch.

Fixes #4 (see also my comments there for some background) and #22.

@alexrothenberg
Copy link

I'm happy if this replaces my pull request #23 but I

I don't know enough about git submodules to know if this will grab the latest ember and ember-data but it looks like specific SHAs are checked in.

Also I think this will build against the head of both which may be an unpublished version. Is there a way to make it checkout a published tag of ember and ember-rails before building? Ideally we should not have to build ember or ember-data and could just download the published versions but I don't think all the files this gem needs are published.

@joliss
Copy link
Contributor Author

joliss commented May 3, 2012

Yes, specific SHAs are checked in. It will not grab the latest ember/ember-data unless you cd into the submodule and check out the master branch. Updating the repo works like this.

Is there a way to make it checkout a published tag of ember and ember-rails before building?

You can check out whatever tag you want with git (git checkout v0.9.7.1 in the submodule).

@tchak tchak mentioned this pull request May 8, 2012
@joliss
Copy link
Contributor Author

joliss commented May 9, 2012

This PR no longer deletes the vendor files, since that would make it impossible to use gem 'ember-rails', git: 'https://github.com/emberjs/ember-rails'.

@joliss
Copy link
Contributor Author

joliss commented May 9, 2012

I still have this in my working copy, by the way, and it's been really useful for writing up pull requests, like emberjs/ember.js#760 or even emberjs/ember.js#793 (I was able to sprinkle console.log calls into the Ember source while running my app against it, which helped me understand the source I was documenting).

@joliss
Copy link
Contributor Author

joliss commented May 29, 2012

Is there still any interest in this? It would need updating for master, but I'll probably abandon this unless there's some consensus that this should be merged.

@joliss joliss closed this Jun 6, 2012
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.

Add support for downloading ember HEAD into your project for testing
2 participants