-
Notifications
You must be signed in to change notification settings - Fork 293
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
GitHub Pod Parsing #167
Comments
👍. I was asking about it earlier in https://github.com/github/markup/blob/master/lib/github/markups.rb#L41 |
As @Skarsnik mentioned on irc, why not use the |
👍 on using |
So? |
OK, now what? |
Now do a PR against https://github.com/github/markup/blob/master/lib/github/markups.rb#L41 to differentiate and render pod6 |
Unassigning @coke because it seems like he agreed to work on file renaming but not the github markup thing. Assign yourself again if that's not the case. |
GitHub replied about setting up Perl 6 Pod renderer: github/markup#907 (comment) |
Per the github/markup ticket, they could use someone from our team to make a PR:
|
Yeah, but they'd need to install Perl 6 for any of that to be of use. They talk about Ruby glue, but how do we get the stuff to glue with? And as far as my understanding goes: it'd be (a) a fully functional Perl 6 compiler that would compile arbitrary code from strangers; and (b) they don't pre-build the pages, right? So how would they cope with (a) and having potentially thousands of pages rendered with slow and resource hungry compiler? Do we have any option of limiting the pod parser to just pod and not arbitrary code? |
Wait... what... really? Turing-complete markup? Such a big temptation,
such a rich source of CVEs over the past decade...
Anyway. I think both problems can be circumvented by pregenerating HTML
pages from Pod during upload preparation.
To avoid polluting the source trees with the generated HTML, it could be
pushed to a different repository. Or to a docs-only branch (on GitHub,
it's called "gh-pages" and has URLs).
|
I don't see how that advice help at all with the topic of this Issue. Who'd be pregenerating anything? GitHub? |
@zoffixznet off the top of my head I guess that a restricted setting could be made that only does enough to parse and render the pod, but it would still be somewhat vulnerable to anything in BEGIN, INIT or CHECK blocks. |
And I would say this one is actually solved, right? |
Not at all. For example, here's a random page that is still not rendered: As an example, here is a page that is rendered: https://github.com/perl6/nqp/blob/master/README.pod (but it's POD5) |
I do have an issue with the |
So does .pod6 get finally rendered correctly?
2018-01-27 20:39 GMT+01:00 Juan Julián Merelo Guervós <
notifications@github.com>:
… I do have an issue with the external label. If it's something that has
been reported elsewhere, it might be better to reference it here. If it's
simply not related to this repo, we should simply close it. If we can fix
it by working around the documentation quirks and rewrite the POD6 code, we
should simply create a list of the pages affected and work through it,
removing the external label.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#167 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AMYVf0XbMcUvEom9BwKWP6XMS-cpP9d7ks5tO3tugaJpZM4GQo3S>
.
--
Sylvain "Skarsnik" Colinet
Victory was near but the power of the ring couldn't be undone
|
@Skarsnik apparently not, but that's external to this repo. except if you want to change content to avoid that. |
No. As for closing tickets for issues that are still relevant, let's please not. The problem is still there, let's have a ticket for every problem. Otherwise people may open new dup tickets because, well, who would search through the closed ones? |
@AlexDaniel this issue depends on another issue in the github/markdown (mentioned above) which does not seem like it's going to be closed or solved any time soon. In fact, googling perl6 pod rendering github returns that issue, not this one. There's slim chance someone else will open another ticket here on that particular matter. I think we can safely close this one. If some duplicate arrives, I'll close it immediately too. |
And I'm closing this one, since nobody objected to this comment two days ago. |
Uh… well, just because nobody objected doesn't really mean that it's right :) There's also my comment above, where I think I've said it all already. We have |
Has anyone experimented with rendering pod6 from perl 5? Pragmatically it may be easier to get github to install a Perl 5 module than Perl 6 itself. This would at least be progress (although clearly using Perl 6 would be better). |
I do have an issue with external tags in general. They tend to stay there
like for ever. Even after the original issue has been solved.
With respect to this one, I don't think it's going to ever be solved. Pod6
parsing is included in perl6, and the other way round, it needs to include
a perl6 parser. There's slim chance it's going to be written in any other
language, and due to security concerns, that it's going to be included in
GitHub. Just check the comments above and the comments in the GitHub
markdown issue, also mentioned above.
It serves no one, helps no one to have an issue that's not going to be
solved any time soon, or maybe never, sitting pretty for 2+ years. I would
like to have this kind of issues not only reopened, but also self-assigned
and with someone committed to solving it or at least tracking what's going
on in the other side. That, however, applies to other issues, NOT this one,
which will never be solved. And that's why I have closed it.
|
2018-03-12 9:11 GMT+01:00 Aleks-Daniel Jakimenko-Aleksejev <
notifications@github.com>:
Uh… well, just because nobody objected doesn't really mean that it's right
:)
There's also my comment above, where I think I've said it all already. We
have external tag, and marking an issue as such is enough. Closing it
does no good, having it open does not hurt and helps a lot when you need to
find something.
Having it open hurts when you try to set aside some time to solving issues
and need to establish priorities. Going into it, seeing it marked as
"external", meaning "it's somebody else's problem, so we can do zilch about
it, but let's leave it here anyway", wastes my time. Keeping it in the
tally of unsolved issues is not good image either. So it does hurt. It
hurts me, if no one else.
And when you need to find something, if google is used, it's going to
search into solved _and_ unsolved issues. It's not going to be more helpful
if it's got a green badge saying "Hey! I'm not helpful and don't solve
anything, but at least I'm green, meaning I care".
My point is that issues in a particular repository should result in
actionable changes in that particular repository. Should be close-able with
a commit. That's not peculiar to this repo. It's the same for any repo.
"external", by definition, can't be so. There's no commit in this repo (or
in another, for that matter) referring to this issue. There is a closed
issue, and a merged PR, time ago, and that's it. This means that the rest
of the repo improvements can safely proceed without this particular one
open.
EDIT: formatting
|
Okay.
Just like other issues that are hard to fix.
This is not specific to
Your pessimism is irrelevant to the fact that we have this problem that needs to be resolved.
We are volunteers. Feel free to self-assign.
I'm very sorry. FWIW you can include
Closing issues that are not resolved creates an image that is even worse.
Are you saying that I should search for tickets using google? That's not how I've been doing things, but that's an interesting idea…
That sounds pretty good to me? EDIT: to clarify, caring about something is the first step to getting something helpful, and eventually solving the issue
In an ideal world, yes. But sometimes we depend on other things and need to track progress and stuff. FWIW here's an example of an issue that needs no commits in the repo but heavily depends on other stuff. According to your logic that issue should not exist, even though it was (and still is!) super helpful.
The rest of the repo improvements can safely proceed with this particular one open. Sorry I don't think I want to discuss this matter more. EDIT: FWIW all references in irc log to this issue: poink |
Pinging again to see if there's anything missing from the PR. It looks like I've done all changes requested, but... |
Just as an update, there was an attempt to implement that in GitHub, but it failed since, for the time being, there is no way to parse the documents and avoid running code. There was a test repo here, but it has been since abandoned. So I'm closing this, since it seems impossible with the current (and foreseeable) state of things. |
… Yeah, but the issue is still not resolved, right? |
It's never going to be solved. I see no point in keeping it open. But, whatever. |
I haven't followed this issue, but can we stop this bullshit, please? Surely you can write a dumb parser that doesn't execute any code. How many of the actual |
In fact, you have to execute code in order to do correct syntax highlighting. Guess what, it doesn't stop all of the editors to highlight perl 6 code. |
Well, maybe you should have. Since the Pod6 parser is part of the perl6 parser, it's basically impossible to guarantee that pod parsing is not running any code. Check this answer in StackOverflow, for instance. What I'm telling above is the actual sequence of events, which I have been following (and that's the reason why I'm closing this). Please check this comment by Kivikakk, who was in charge of implementing this for GitHub, and their subsequent quitting. |
OK, sounds great, let's keep this open then.
Makes sense. We can also create a new ticket for our next iteration. |
It would be much better if we closed this issue and summarized the problems for whatever comes next in a new one. Lots of people (including me) have wasted a lot of energy to solve it, for no good. It can be said that it has contributed to the greater good of Pod::To::HTML rendering, which it has, but still there's some specific work done to solve the issue which has gone to waste, and for precisely the reasons that were advised in several comments in this issue some time ago. |
In fact, I am going to call a vote on closing this issue, for these reasons.
So please vote by thumbs up (closing the issue), thumbs down (keep it open). And, of course, comment to explain your vote if you so like. |
Ticket moved: Raku/user-experience#35 Please don't close still-unresolved issues without creating a new ticket first. |
I'm pretty new to the community, so I don't know if this was brought up in the past....
GitHub doesn't parse P6 pod and shows errors when viewing docs in this repo. I contacted GitHub and this was their response:
So if we could come up with some way to disambiguate between P5 and P6 pods, we could fix the issues in the repo (as well as any potential P6 modules that use P6 pod).
The text was updated successfully, but these errors were encountered: