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

Some bug in the ToS #854

Closed
greduan opened this Issue Jun 27, 2014 · 18 comments

Comments

Projects
None yet
10 participants
@greduan
Contributor

greduan commented Jun 27, 2014

Hey @balupton, it seems there's something going on in the ToS section of DocPad's code.

#851
#853
#857

The bug is that when the user runs docpad run and is asked to accept the ToS, when the user says Y DocPad just stops executing.

At this point if the user runs docpad run again the same thing happens.

Any idea what could be happening? Seems like some logic went awry at some point in the code...

@balupton

This comment has been minimized.

Member

balupton commented Jun 27, 2014

Could be the helper service is down

@greduan

This comment has been minimized.

Contributor

greduan commented Jun 27, 2014

Could you check that?

@norpol

This comment has been minimized.

norpol commented Jun 28, 2014

Start docpad with the additional flag --offline and it will work.

@greduan

This comment has been minimized.

Contributor

greduan commented Jul 1, 2014

@balupton Should I add a FAQ entry to the docs for this?

@josephfinlayson

This comment has been minimized.

josephfinlayson commented Jul 5, 2014

I've experienced this too on Elementary OS. This is incredibly annoying.

@greduan

This comment has been minimized.

Contributor

greduan commented Jul 5, 2014

Added entry to FAQ for this issue: docpad/documentation@6e521ff

@bguiz

This comment has been minimized.

bguiz commented Jul 7, 2014

Could be the helper service is down

@balupton If this is really why it is not working, then I would expect it to at least output an error message prior to exiting. Consider this?

@greduan

This comment has been minimized.

Contributor

greduan commented Jul 11, 2014

@balupton It seems that going back to v6.68.1 fixed the issue for a user: #862

@balupton

This comment has been minimized.

Member

balupton commented Aug 20, 2014

@fizzvr

This comment has been minimized.

fizzvr commented Sep 1, 2014

--offline 👍

@austinbutler

This comment has been minimized.

austinbutler commented Jan 16, 2015

Using --offline only presented me with 1 skeleton, so it wasn't a great workaround. Instead I used this, which allowed me to install npm packages globablly without root: https://github.com/glenpike/npm-g_nosudo

@7wonders

This comment has been minimized.

7wonders commented Feb 16, 2015

Also getting this problem but then --offline presents a new error:

Fantastic. We like informed people!
error: An error occured: 
ReferenceError: userConfigString is not defined
  at DocPad.updateUserConfig (/project/node_modules/docpad/out/lib/docpad.js:1555:40)
  at /project/node_modules/docpad/out/lib/interfaces/console.js:271:22
  at DocPad.track (/project/node_modules/docpad/out/lib/docpad.js:2167:11)
  at /project/node_modules/docpad/out/lib/interfaces/console.js:265:25
  at /project/node_modules/docpad/node_modules/promptly/index.js:80:9
  at Interface.onLine (/project/node_modules/docpad/node_modules/promptly/node_modules/read/lib/read.js:111:5)
  at Interface.emit (events.js:107:17)
  at Interface._onLine (readline.js:214:10)
  at Interface._line (readline.js:553:8)
  at Interface._ttyWrite (readline.js:830:14)
  at ReadStream.onkeypress (readline.js:109:10)
  at ReadStream.emit (events.js:110:17)
  at readline.js:1175:14
  at Array.forEach (native)
  at emitKeys (readline.js:993:10)
  at ReadStream.onData (readline.js:910:14)
  at ReadStream.emit (events.js:107:17)
  at readableAddChunk (_stream_readable.js:163:16)
  at ReadStream.Readable.push (_stream_readable.js:126:10)
  at TTY.onread (net.js:529:20)
@shearer12345

This comment has been minimized.

shearer12345 commented Feb 18, 2015

I get the same error as @7wonders when using the the --offline option (node version v0.12.0, built from source)

@balupton

This comment has been minimized.

Member

balupton commented Feb 18, 2015

Should be fixed in the upcoming version. Hang tight.

@shearer12345

This comment has been minimized.

shearer12345 commented Feb 18, 2015

Great. Thanks. :-)

-- Shearer

On 18 February 2015 at 15:04, Benjamin Lupton notifications@github.com
wrote:

Should be fixed in the upcoming version.


Reply to this email directly or view it on GitHub
#854 (comment).

@norpol

This comment has been minimized.

norpol commented Feb 19, 2015

In my opinion, it's hilarious that you need an --offline function for using this tool...

@balupton

This comment has been minimized.

Member

balupton commented Feb 19, 2015

@norpol fetching skeletons, fetching dep versions, feedr plugin, cachr plugin, etc all use online abilities. I find it very cool that we have a flag in DocPad that allows DocPad to gracefully refer to its caches when offline rather than falling over (as it should do if online and a request fails). Hilarious wouldn't be the word I use for such maturity of a project.

@norpol

This comment has been minimized.

norpol commented Feb 20, 2015

I don't (and didn't) want to bother you, but what bothers me is when I see something like that:

info: Welcome to DocPad v6.73.2 (global installation: /home/tom/.npm-packages/lib/node_modules/docpad)
notice: If everyone who saw this message donated $1/week, maintaining DocPad would become sustainable: http://docpad.org/donate
info: Contribute: http://docpad.org/docs/contribute
info: Plugins: 
info: Environment: development
Before we continue, have you read and agree to DocPad's Terms of Service (http://bevry.me/tos) and Privacy Policy (http://bevry.me/privacy)? [Y/n]

But yeah, that's a personal opinion. All other tools on my machine I'm using are not asking me to agree their tos and privacy policy. They've just got simply a licence. :c

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment