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

undefined method `uniq' for #<MRuby::Gem::List:0x00000001fee358> #11

Closed
archseer opened this issue Jul 17, 2013 · 6 comments
Closed

undefined method `uniq' for #<MRuby::Gem::List:0x00000001fee358> #11

archseer opened this issue Jul 17, 2013 · 6 comments

Comments

@archseer
Copy link
Contributor

From mruby/mruby#1386

MRuby::Build.new do |conf|
  toolchain :gcc

  # include the default GEMs
  conf.gembox 'default'

  conf.gem :github => 'mattn/mruby-require'
end

Output:

(in /home/speed/ruby/moon/vendor/mruby)
GIT   https://github.com/mattn/mruby-require.git -> build/mrbgems/mruby-require
Cloning into 'build/mrbgems/mruby-require'...
remote: Counting objects: 231, done.
remote: Compressing objects: 100% (132/132), done.
remote: Total 231 (delta 65), reused 227 (delta 63)
Receiving objects: 100% (231/231), 29.55 KiB | 0 bytes/s, done.
Resolving deltas: 100% (65/65), done.
Checking connectivity... done
rake aborted!
undefined method `uniq' for #<MRuby::Gem::List:0x00000001fee358>
Rakefile:26:in `load'
@archseer
Copy link
Contributor Author

This is what broke it mruby/mruby@4e16491, the MRuby::Gem::List no longer has uniq and reject!

@charlescui
Copy link

+1

@mattn
Copy link
Owner

mattn commented Jul 29, 2013

Currently, I don't have much time to fix this issue. Could you please send me a pull-req?

@archseer
Copy link
Contributor Author

@mattn the way I fixed it in my branch was to just remove the offending code, because I'd need to add the methods back to MRuby::Gem::List.. archseer@13d7b40

I didn't need the require gem feature anyway, just needed to require external files.

@archseer
Copy link
Contributor Author

I also tried changing the way rb files get required (no compilation to mrb, so we can get proper debug backtraces), but I hit a snag, which is probably why you guys did it that way in the first place.

mruby/mruby#1418

@mattn mattn closed this as completed in b7f7ed2 Aug 1, 2013
@mattn
Copy link
Owner

mattn commented Aug 1, 2013

@archseer please separate issue about archseer@e382d50 Thanks

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

3 participants