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

fix for uninitialized constant Spree::FileUtilz #21

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix for uninitialized constant Spree::FileUtilz #21

wants to merge 1 commit into from

Conversation

thebyrd
Copy link

@thebyrd thebyrd commented Jun 25, 2012

The newest version of ruby requires all namespaces in the path to be declared. This is currently breaking the install on Ruby 1.9.3

…declared. This is currently breaking the install on Ruby 1.9.3
@joshmcarthur
Copy link
Owner

Hi @davidbyrd11 - this change is already in place on the 1_1_x branch, which is the only one I plan on keeping up to date for 1.9.3. Because I didn't tag the release properly when this gem was first released, I'm keeping master back on the oldest supported version of Spree to prevent people using the gem directly from the master branch of the github repo.

@roushkaman
Copy link

hi! It's not work

@prasmalla
Copy link

hello josh,
i'm trying to install this gem on spree 1.3.2 and Spree::Core::FileUtilz was removed from spree few months back. will this gem be getting any updates soon? thanks for all your work.

@joshmcarthur
Copy link
Owner

@prasmalla have you tried the 1_1_x branch? I believe that has a fix for the Spree::Core::FileUtilz class being removed.

@prasmalla
Copy link

i haven't tried the 1_1_x branch but looking at its source code - / lib / tasks / install.rake and its still using Spree::Core::FileUtilz

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.

4 participants