Re: [heycam/webidl] Contributing to webidl just got a lot harder (#171)

> The empty spans, I think, come from an outdated syntax highlighter dependency? My guess is Bikeshed doesn't have whatever metadata is in place to tell Python the correct syntax highlighter version. We've run into that several times now where different people all have the same supposedly-up-to-date versions of Bikeshed, but some get empty spans and some don't.

I have no idea what these empty spans are and haven't heard of them before. What's up with it?

> Biblio reordering should be fine

Hmm, didn't realize they were reordering. I'll switch to sorting these so they don't reorder.

@bzbarsky Would love some more detail on what parts of the install instructions didn't work for you - they're correct to the best of my knowledge, and have been updated by people running into problems over time.

> That's ignoring the "let's globally install untrusted code as root" issue, of course, which doesn't make me happy either.

Yeah. :/  Several people have complained about wanting me to use one of the virtualization options out there, but I have no familiarity with them whatsoever, and nobody wants to put in sufficient effort to document it for me. Current instructions just install things "the normal way" that you see in the wild, which is unsafe but eh.

> But even given that, it doesn't actually end up putting anything named "bikeshed" in my path. 

That's very strange, as the `pip install` should have done so. No one else has ever reported such a problem; they all get it in their path immediately.

> And the actual bikeshed.py doesn't use the python it says to install html5lib for, so can't find it.

...what?

> I've managed to run it locally by explicitly invoking the right python version on bikeshed.py and reading the makefile to figure out what arguments to pass to it. Which is fine as far as I''m concerned, but is a way higher contribution bar than we used to have.

As currently written, at least, using the makefile at all doesn't do anything - you can just run `bikeshed` (with no arguments) in the folder and it'll figure out what to do.  Some people seem to rely like using that workflow, tho. Shrug.



-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/heycam/webidl/issues/171#issuecomment-245965887