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

Unpin html5lib 0.95 requirement #360

Merged
merged 1 commit into from
Feb 27, 2014
Merged

Conversation

dbs
Copy link
Contributor

@dbs dbs commented Feb 26, 2014

Due to an html5lib regression (described in the thread at
https://groups.google.com/d/msg/rdflib-dev/ZcAgKzhS3vI/3mxIJz4rwWUJ) , we
opened html5lib/html5lib-python#67 on June 17th, 2013
and pinned html5lib to 0.95 in setup.py. The bug was fixed and a new release of
html5lib (1.0b3) was cut on July 24, 2013. The current version of html5lib is
0.999; let's unpin that html5lib requirement.

Signed-off-by: Dan Scott dan@coffeecode.net

Due to an html5lib regression (described in the thread at
https://groups.google.com/d/msg/rdflib-dev/ZcAgKzhS3vI/3mxIJz4rwWUJ) , we
opened html5lib/html5lib-python#67 on June 17th, 2013
and pinned html5lib to 0.95 in setup.py. The bug was fixed and a new release of
html5lib (1.0b3) was cut on July 24, 2013. The current version of html5lib is
0.999; let's unpin that html5lib requirement.

Signed-off-by: Dan Scott <dan@coffeecode.net>
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) when pulling 425148e on dbs:html5lib_is_fixed into 1282d41 on RDFLib:master.

@joernhees
Copy link
Member

seems legit to me, in future when we pin/fix a lib due to a bug and trigger an upstream fix we should maybe leave a long standing issue reminding us of the fact why it was pinned...

joernhees added a commit that referenced this pull request Feb 27, 2014
Unpin html5lib 0.95 requirement
@joernhees joernhees merged commit fdbb9e9 into RDFLib:master Feb 27, 2014
dbs added a commit to dbs/python-rdflib that referenced this pull request Mar 3, 2014
Per RDFLib/rdflib#360 - remove the pinning of html5lib
to 0.95, as versions after 1.0b2 resolve the regression. This patch matches
the upstream version.

Signed-off-by: Dan Scott <dan@coffeecode.net>
@dbs dbs deleted the html5lib_is_fixed branch April 3, 2014 16:10
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