Skip to content

On gem install, do not try to copy manpage files in the "extensions" step #356

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

Conversation

olleolleolle
Copy link
Collaborator

To dodge all issues with "you need git, you need Bundler, you need ..." when performing the "extensions" step in the install process, this change wants to drop that step.

Problems to solve:

What we miss out on with this change: man pages will no longer be copied to a location guessed by the Rakefile. (These man pages are not automatically updated.)

@olleolleolle olleolleolle changed the title On install, do not try to copy manpage files On gem install, do not try to copy manpage files in the "extensions" step Mar 19, 2016
@olleolleolle
Copy link
Collaborator Author

I was thinking of you, @jkeiser, when I made this change.

Several small things could be changed back to "development dependency" status if this were merged, right?

@olleolleolle
Copy link
Collaborator Author

The dependencies I refer to can be found in the commits @jkeiser added to help us out. If we opt not to run the Rakefile as part of the install, it's all simpler.

Here, they can be found in the history of the .gemspec: https://github.com/skywinder/github-changelog-generator/commits/master/github_changelog_generator.gemspec

skywinder added a commit that referenced this pull request Mar 22, 2016
…ding-on-gem-install

On gem install, do not try to copy manpage files in the "extensions" step
@skywinder skywinder merged commit c798266 into github-changelog-generator:master Mar 22, 2016
@olleolleolle olleolleolle deleted the feature/drop-extension-building-on-gem-install branch March 22, 2016 12:28
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.

2 participants