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.

Sign up for GitHub

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

Jump to bottom

MANIFEST.in causes install failure #29

Closed
boyxuper opened this issue Jul 31, 2013 · 2 comments
Closed

MANIFEST.in causes install failure #29

boyxuper opened this issue Jul 31, 2013 · 2 comments

Comments

@boyxuper
Copy link

include LICENSE
include README.rst
recursive-include provider/templates *.html
recursive-include provider/templates *.txt
recursive-include provider/ *json

File "x:\Python27\lib\distutils\util.py", line 201, in convert_path
raise ValueError, "path '%s' cannot end with '/'" % pathname

issue occurs when os.sep != '/'

solution is remove tailing /(provider/ => provider) at last line of MANIFEST.in
@kliyes
Copy link

kliyes commented Aug 5, 2013

Thanks!It's worked!

@eculver eculver mentioned this issue Oct 25, 2013
Closed
eculver added a commit that referenced this issue Oct 25, 2013
@eculver
#29 - Remove trailing slash that was causing installs to fail on Wind…
a6bbb29 
…ows.
@eculver
Copy link
Contributor

eculver commented Oct 25, 2013

Thanks for looking into this @boyxuper. I've tested the change and it doesn't seem to break the current functionality, so I pushed the fix to master.

@eculver eculver closed this as completed Oct 25, 2013
ysavary pushed a commit to epyx-src/django-oauth2-provider that referenced this issue Nov 20, 2014
Merge branch 'master' into fixes/exodoc
14c846b 
* master: (29 commits)
  caffeinehit#56 - Uncomment an assertion in scope tests. It was commented out whilst adding support for Django 1.6.
  Fix get_access_token failing on single tracker mode after token refresh (caffeinehit#70).
  Removing django 1.3 from tox and travis-ci matrices since it's no longer officially supported.
  Fixing silly bug in test wrapper's django version detection.
  caffeinehit#55 - Update travis-ci config to include Django 1.6.
  Django 1.6 detection for test wrapper script.
  caffeinehit#55 - Removing another unnecessary import.
  caffeinehit#55 - Removing unnecessary imports.
  caffeinehit#55 Adding Django 1.6 to tox coverage matrix.
  caffeinehit#55 - Fallback to simplejson in older versions of Django.
  Fixes caffeinehit#55 - Handle non-list/non-tuple form field values for scope introduced in Django 1.6
  caffeinehit#51 caffeinehit#53 - Support for model serialization/deserialization. This addresses the issues in Django 1.6 involving new session storage backend behavior.
  Fix caffeinehit#26 by checking for invalid data before attempting access.
  Use `constants` instead of going directly through settings when invalidating tokens and grants. This also replaces 'clean' with 'delete' for the new setting to be more clear as to what's being done under the hood.
  implement OAUTH_CLEAN_EXPIRED, clean as you go
  Adding a License section to README.
  Fixes caffeinehit#32 - Add `token_type` to access token response to conform to section 4.2.2 of the OAuth 2.0 specification.
  caffeinehit#29 - Remove trailing slash that was causing installs to fail on Windows.
  Addressing caffeinehit#28 by replacing the BSD license reference with MIT to match what's in the repo.
  Bumping version to 0.2.7-dev to avoid ambiguity in an established package.
  ...

Conflicts:
	provider/__init__.py
	provider/oauth2/views.py
joestump pushed a commit to sprintly/django-oauth2-provider that referenced this issue Jan 18, 2017
Merge pull request caffeinehit#29 from edx/cdyer/1.0.1
4f88783 
version bump
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@eculver @boyxuper @kliyes