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

Fix #339: Add ChangeLog placeholder to fix symlink #362

Closed
wants to merge 2 commits into from

Conversation

houglum
Copy link

@houglum houglum commented May 24, 2016

Running python setup.py build_sphinx will result in ChangeLog
being overwritten with actual changes.

Running `python setup.py build_sphinx` will result in ChangeLog
being overwritten with actual changes.
@berkerpeksag
Copy link
Member

LGTM, thanks!

@houglum
Copy link
Author

houglum commented May 24, 2016

@rbtcollins - any verification needed besides ensuring ChangeLog is overwritten after build?

In order to use mock as a submodule, we can't rely on it being installed via pip, which pbr expects.  This allows us to use mock in projects (gsutil, specifically) that may be installed via methods other than pip (git repo, tarball, etc.).
@rbtcollins
Copy link
Member

I'm not happy with the manual work that this will reintroduce around version management; like I said in #385 - Requirements:

semver compatible versions for all git revs
clean changelog like currently produced and integrated into docs
clean support for conditional requirements (like we're using today)

tl;dr definitely not going to use manual versioning, life is too short. If you reset this back to the first commit only, which was fine, we can merge it.

@houglum
Copy link
Author

houglum commented Dec 11, 2016

Yep, I didn't intend to include the second change at the time I issued the pull request. We added that later after finding that some installation methods for gsutil (downloading via tarball, rather than installing with pip) would not include all necessary dependencies. Rather than include even more dependencies in the tarball, I just hard-coded the version in my fork we're using.

@houglum houglum closed this Dec 11, 2016
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

Successfully merging this pull request may close these issues.

3 participants