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

doc folder in tarball @ pypi absent of .inc files; networkx-1.9.1 #1263

Merged
merged 1 commit into from
Oct 11, 2014

Conversation

chebee7i
Copy link
Member

@chebee7i chebee7i commented Oct 7, 2014

The doc build (in release 1.9.1 is huge, however there are many warnings reporting absence if files of suffix .inc in folder doc/source/developer/gitwash.

InputError: [Errno 2] No such file or directory: 'source/developer/gitwash/links.inc'.

Sure enough in MANIFEST.in

recursive-include doc *.py *.rst Makefile *.html *.png *.txt *.css

Please add *.inc to MANIFEST.in

@chebee7i
Copy link
Member

chebee7i commented Oct 7, 2014

Thanks will do!

@chebee7i
Copy link
Member

chebee7i commented Oct 7, 2014

This might require a new release to 1.9.2.

@ysitu ysitu added this to the networkx-1.9.2 milestone Oct 8, 2014
@ysitu
Copy link
Contributor

ysitu commented Oct 8, 2014

I have tentatively created a networkx-1.9.2 milestone for this.

@chebee7i
Copy link
Member

chebee7i commented Oct 8, 2014

@idella is the build process broken because these are missing? Or just annoying?

@idella
Copy link
Author

idella commented Oct 9, 2014

It's not broken but it spews out copious lines of warnings which are most noticeable, so annoying really

@chebee7i
Copy link
Member

chebee7i commented Oct 9, 2014

Ok, then maybe it is not necessary to do a bug release just because of this. We will get it included though.

chebee7i added a commit that referenced this pull request Oct 11, 2014
doc folder in tarball @ pypi absent of .inc files; networkx-1.9.1
@chebee7i chebee7i merged commit 6ce68ca into networkx:master Oct 11, 2014
@chebee7i chebee7i removed this from the networkx-1.9.2 milestone Oct 11, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

3 participants