Skip to content
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

Please pull move of URI parsing and escape logic to Perl6 URI project. #27

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ronaldxs
Copy link

@ronaldxs ronaldxs commented Sep 6, 2011

Hi,

I am the current maintainer of the Perl 6 URI project at https://github.com/ihrd/uri. I created a branch of November that uses the ihrd/uri module moving about 200 lines of code out of November in the process. The URI module from the Perl 6 URI project does parsing based on the full URI BNF from RFC 3986 and its Escape module does both UTF-8 URI encoding and decoding. All tests in make test for this fork run without error and I have manually tested the resulting Wiki under Apache.

Thank you,
Ron

Ron Schmidt added 3 commits July 31, 2011 20:33
…ut in early stages of testing with more work to do.
…ule. Passes make test but still needs testing through browser and apache.
@softmoth
Copy link
Collaborator

softmoth commented Sep 6, 2011

Ronald,

On Tue, Sep 6, 2011 at 11:35 AM, ronaldxs
reply@reply.github.com
wrote:

I am the current maintainer of the Perl 6 URI project at https://github.com/ihrd/uri.  I created a branch of November that uses the ihrd/uri module moving about 200 lines of code out of November in the process.  The URI module from the Perl 6 URI project does parsing based on the full URI BNF from RFC 3986 and its Escape module does both UTF-8 URI encoding and decoding.  All tests in make test for this fork run without error and I have manually tested the resulting Wiki under Apache.

Awesome, thanks! I'll take a look at this and merge it.

Tim

@ilyabelikin
Copy link
Contributor

Cool! I glad to see activity here :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants