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

removing pyparsing version requirement #696

Merged
merged 1 commit into from
Jan 24, 2017

Conversation

nateprewitt
Copy link
Contributor

The versioning requirement was put in place 4 years ago to handle older
versions of Python. setuptools no longer supports pyparsing < 2.1, so
we'll need to adhere to that to continue using pip or put a hard version
on pip. This seems like the more reasonable approach.

The versioning requirement was put in place 4 years ago to handle older
versions of Python. setuptools no longer supports pyparsing < 2.1, so
we'll need to adhere to that to continue using pip.
@coveralls
Copy link

coveralls commented Jan 23, 2017

Coverage Status

Coverage decreased (-0.003%) to 62.712% when pulling f6fd1e9 on nateprewitt:pyparsing_fix into 14243b6 on RDFLib:master.

@nateprewitt
Copy link
Contributor Author

nateprewitt commented Jan 23, 2017

While we're here, there's still code for this 2.5 requirement in setup.py. It can probably be left there for any users trying to install via setup.py, but 2.5 appears to not have been supported for almost 2 years now.

Edit: This change also means that requirements could be reduced to a single requirements.txt file if that was desired.

@gromgull gromgull merged commit 71dc600 into RDFLib:master Jan 24, 2017
@gromgull
Copy link
Member

thanks!

@gromgull
Copy link
Member

I would say we should clean up setup.py as well - it's not super urgent though.

@nateprewitt nateprewitt deleted the pyparsing_fix branch January 24, 2017 11:49
@joernhees joernhees added enhancement New feature or request meta Relates primarily to the project and not users of the project. labels Jan 25, 2017
@joernhees joernhees added this to the rdflib 4.2.2 milestone Jan 25, 2017
joernhees added a commit that referenced this pull request Jan 25, 2017
* master: (44 commits)
  quote cleanup OCD
  serializer/parser alias for 'ntriples'
  serializer/parser alias for 'ttl'
  cleanup
  remove outdated always skipped test
  a bit of changelog
  add a NTSerializer sub-class for nt11 (#700)
  Restrict normalization to unicode-compatible values (#674)
  fixes for turtle/trig namespace handling
  skip serialising empty default graph
  skip round-trip test, unfixable until 5.0
  prefix test for #428
  Added additional trig unit tests to highlight some currently occurring issues.
  remove ancient and broken 2.3 support code. (#681)
  updating deprecated testing syntax (#697)
  docs: clarify the use of an identifier when persisting a triplestore (#654)
  removing pyparsing version requirement (#696)
  made min/max aggregate functions support all literals (#694)
  actually fix projection from sub-queries
  added dawg tests for #607
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request meta Relates primarily to the project and not users of the project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants