Skip to content

Include woff2 static files #5694

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

Closed
wants to merge 1 commit into from
Closed

Include woff2 static files #5694

wants to merge 1 commit into from

Conversation

ryanhiebert
Copy link
Contributor

The .woff2 fonts aren't being included in the release, and that's causing failures for my build using whitenoise.storage.CompressedManifestStaticFilesStorage, which looks for all referenced files from CSS files and compresses them with a static hash on the filename.

@ryanhiebert
Copy link
Contributor Author

3.7.3 For some reason did include woff2 in the PyPI releases, but 3.7.4 does not. I'm not sure what changed between those releases as far as deploy process, and it's possible that this won't fix it, depending on the cause. It still seems like it should be included in the manifest, though.

@ryanhiebert
Copy link
Contributor Author

I think that #5624 is the likely culprit, because before that the package data was given by a function in setup.py, rather than by looking at the MANIFEST.in. The woff2 file was the one that I was hitting, but I'm not yet sure that it's the only one that may have been lost by the change there.

@xordoquy
Copy link
Collaborator

Indeed that was working by unexpected side effect :(
I will try to ensure there are no other files forgotten elsewhere with drf.

@xordoquy xordoquy added the Bug label Dec 21, 2017
@xordoquy
Copy link
Collaborator

closes #5692

@axnsan12
Copy link
Contributor

*.ico seems to be missing too (static/rest_framework/docs/img/favicon.ico)

@tomchristie tomchristie added this to the Release 3.7.6 milestone Dec 21, 2017
@tomchristie
Copy link
Member

Thanks @ryanhiebert, @axnsan12.

Versions 3.7.5 & 3.7.6 now released. 🙃

@tomchristie
Copy link
Member

I think we're clear now, right?
Shout out if anything's still borked.

@ryanhiebert ryanhiebert deleted the manifest branch December 21, 2017 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants