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

Cannot require 'faker' after update to 1.8.1 #975

Closed
botandrose opened this issue Jul 10, 2017 · 7 comments
Closed

Cannot require 'faker' after update to 1.8.1 #975

botandrose opened this issue Jul 10, 2017 · 7 comments

Comments

@botandrose
Copy link

$ irb
r2.2.0 :001 > require 'faker'
NameError: uninitialized constant Faker::Date
	from /home/micah/.rvm/gems/ruby-2.2.0@complexity/gems/faker-1.8.1/lib/faker/time.rb:2:in `<module:Faker>'
@turboladen
Copy link

Having the same issue here with ruby 2.4.1 on Ubuntu 14.04.4 LTS.

@rcaught
Copy link

rcaught commented Jul 11, 2017

#971

@stympy
Copy link
Contributor

stympy commented Jul 11, 2017

I have tried to reproduce this on Ubuntu 16.04 and 14.04 with ruby installed via apt, including both the Ubuntu default version (2.3.1 on 16.04) and the brightbox packages (2.4.1 on both Ubuntu versions). In every case I was able to do this:

$ gem install faker
$ irb
irb(main):001:0> require 'faker'
=> true

To aid me in getting this resolved, please let me know how you have installed Ruby and how you have installed faker (either via the gem command or by bundler).

@kmats
Copy link

kmats commented Jul 11, 2017

i think,
baf7f82 does not guarantee the requiring order.
so sometimes 'faker/time' is required before 'faker/date' and raise "NameError: uninitialized constant Faker::Date"

@stympy stympy closed this as completed in 4ea2d4c Jul 11, 2017
@stympy
Copy link
Contributor

stympy commented Jul 11, 2017

Yeah, I was thinking the same thing, I was just hoping to be able to reproduce before trying a fix. Oh well. :)

@fotos
Copy link

fotos commented Jul 11, 2017

Maybe the 1.8.1 gem should be yanked?

@stympy
Copy link
Contributor

stympy commented Jul 11, 2017

Gem v1.8.1 has been yanked

landongrindheim pushed a commit to landongrindheim/faker that referenced this issue Oct 7, 2017
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

No branches or pull requests

6 participants