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

wording of "files in version control do not match the sdist" message #34

Closed
cjerdonek opened this issue Oct 1, 2014 · 8 comments
Closed

Comments

@cjerdonek
Copy link

Hi, I think the following error message can be improved, "files in version control do not match the sdist!"

This error makes it look like check-manifest will be listing what files in version control are different from their versions in the sdist (e.g. if the files have uncommitted changes when running check-manifest). But what it is really saying is that the list of files in version control does not match the list in sdist (i.e. the file names don't match, and not necessarily the contents).

@mgedmin
Copy link
Owner

mgedmin commented Oct 1, 2014

Can you suggest a better phrasing?

Would "list of files in version control does not match the sdist" work?

@cjerdonek
Copy link
Author

How about "names of files in version control and sdist differ" or "paths of files in version control and sdist differ"?

@cjerdonek
Copy link
Author

Note: I updated my previous comment.

@mgedmin
Copy link
Owner

mgedmin commented Oct 1, 2014

Him, your first suggestion makes me think these are the same files (I.e. same content) that have differing names. And your second suggestion looks like the same files got moved into a different subdirectory.

@cjerdonek
Copy link
Author

Yes, I see what you're saying. How about, "list of files in version control and sdist differ"?

I think that's a little clearer than saying the list doesn't match the sdist (because it is really the sdist's list).

@cjerdonek
Copy link
Author

Or "list of files in version control and sdist do not match."

@mgedmin
Copy link
Owner

mgedmin commented Oct 2, 2014

I like that.

My English fails me: is it supposed to be "list of files in X and Y do not match" or "lists of files in X and Y do not match"?

@cjerdonek
Copy link
Author

The latter is the correct one (because it reduces to "lists do not match"), even though it doesn't sound as good. :)

@mgedmin mgedmin closed this as completed in 2e67aac Oct 2, 2014
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

No branches or pull requests

2 participants