Skip to content
This repository has been archived by the owner on Apr 17, 2018. It is now read-only.

Incorperate upstream fix for captcha module import #571

Merged
merged 1 commit into from
Jul 5, 2016

Conversation

jglamine
Copy link
Contributor

@jglamine jglamine commented Jul 3, 2016

On windows and OSX, vagrant up fails due to an import error:

==> default:   File "/vagrant/r2/r2/lib/Captcha.py", line 25, in <module>
==> default:     from Captcha.Base import randomIdentifier
==> default: ImportError: No module named Base
==> default: ---- End output of "bash"  "/tmp/chef-script20160703-7426-qg0yug" ----
==> default: Ran "bash"  "/tmp/chef-script20160703-7426-qg0yug" returned 1
==> default: [2016-07-03T17:55:30+00:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)

This was fixed by reddit upstream in reddit-archive/reddit@327a5dc
See also: https://www.reddit.com/r/redditdev/comments/28cf7p/error_with_r2_source_code_no_module_named_base_in/

This change incorporates that fix.

@@ -6,19 +6,21 @@
# software over a computer network and provide for limited attribution for the
# Original Developer. In addition, Exhibit A has been modified to be consistent
# with Exhibit B.
#
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like my editor also removed some trailing whitespace.

@michaelwebb76 michaelwebb76 merged commit c71f349 into bellroy:master Jul 5, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants