cijoe icon indicating copy to clipboard operation
cijoe copied to clipboard

Defining project_path stuck in recursion

Open mptre opened this issue 13 years ago • 19 comments

I experienced some difficulties while trying to start the cijoe server on my local machine (OS X). The set call on line 103 got stuck in recursion. This could be solved by passing the third parameter named ignore_setter.

mptre avatar Oct 01 '11 22:10 mptre

Please merge this into master. I also had the same problem, this commit resolved it.

awd avatar Oct 06 '11 14:10 awd

I had this same problem as well

eyberg avatar Oct 17 '11 23:10 eyberg

+1, same problem here

marceldegraaf avatar Oct 24 '11 12:10 marceldegraaf

FYI: this patch no longer seems to work with Sinatra 1.1.3. Just tried it and got this:

/Users/mdg/.rvm/gems/ruby-1.9.2-p136/gems/sinatra-1.1.3/lib/sinatra/base.rb:865:in `set': wrong number of arguments (3 for 2) (ArgumentError)
    from /Users/mdg/.rvm/gems/ruby-1.9.2-p136/gems/cijoe-0.9.4/lib/cijoe/server.rb:107:in `project_path='
    from /Users/mdg/.rvm/gems/ruby-1.9.2-p136/gems/sinatra-1.1.3/lib/sinatra/base.rb:875:in `set'
    from /Users/mdg/.rvm/gems/ruby-1.9.2-p136/gems/cijoe-0.9.4/lib/cijoe/server.rb:90:in `start'
    from /Users/mdg/.rvm/gems/ruby-1.9.2-p136/gems/cijoe-0.9.4/bin/cijoe:50:in `<top (required)>'
    from /Users/mdg/.rvm/gems/ruby-1.9.2-p136/bin/cijoe:19:in `load'
    from /Users/mdg/.rvm/gems/ruby-1.9.2-p136/bin/cijoe:19:in `<main>'

marceldegraaf avatar Oct 24 '11 13:10 marceldegraaf

+1. This commit resolved the issue for me with sinatra 1.3.1.

yaotti avatar Nov 01 '11 14:11 yaotti

:+1:

tiagogodinho avatar Nov 01 '11 21:11 tiagogodinho

I'm still experiencing the same problem using Sinatra 1.3.1. This commit is yet working as a solution.

mptre avatar Nov 02 '11 23:11 mptre

+1, same issue

linuxsable avatar Dec 06 '11 19:12 linuxsable

+1

potomak avatar Dec 09 '11 08:12 potomak

+1

ncreuschling avatar Jan 15 '12 19:01 ncreuschling

+1

maljub01 avatar Feb 07 '12 17:02 maljub01

+1

CIJoe was unusable without making this change manually. Once I did it worked. Would be nice to see this in master.

kenmazaika avatar Mar 04 '12 00:03 kenmazaika

+1...

relrod avatar Apr 04 '12 07:04 relrod

+1

stas avatar Apr 04 '12 13:04 stas

+1

Is this project still maintained? 7 months of people having to manually patch a fundamental fix seems to long.

defrex avatar Apr 23 '12 18:04 defrex

Maybe not, if you look carefully at project's URL you'll see it says: defunkt/cijoe On Apr 23, 2012 8:23 PM, "defrex" < [email protected]> wrote:

+1

Is this project still maintained? 7 months of people having to manually patch a fundamental fix seems to long.


Reply to this email directly or view it on GitHub: https://github.com/defunkt/cijoe/pull/72#issuecomment-5287874

potomak avatar Apr 23 '12 18:04 potomak

You're saying the founder of GitHub has something better to do then patch a CI server that they don't use anymore? To bad, Jenkins is bloated and ugly.

defrex avatar Apr 23 '12 18:04 defrex

+1

DNNX avatar Aug 28 '12 13:08 DNNX

+1

prakhar1989 avatar Jan 12 '15 16:01 prakhar1989