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

(maint) fix gettext rake tasks #180

Merged
merged 1 commit into from
Feb 23, 2017
Merged

Conversation

eputnam
Copy link
Contributor

@eputnam eputnam commented Feb 23, 2017

The gettext rake tasks are currently looking for a locales directory in the spec_helper when they should be looking in the module root directory. A way around this is to use Dir.pwd, because rake always moves to the directory where the Rakefile is to load it, making Dir.pwd always the module root (https://github.com/ruby/rake/blob/master/lib/rake/application.rb#L685-L687)

The gettext rake tasks are currently looking for a locales directory in the spec_helper when they should be looking in the module root directory. A way around this is to use Dir.pwd, because rake always moves to the directory where the Rakefile is to load it, making Dir.pwd always the module root.
@puppetcla
Copy link

Waiting for CLA signature by @eputnam

@eputnam - We require a Contributor License Agreement (CLA) for people who contribute to Puppet, but we have an easy click-through license with instructions, which is available at https://cla.puppet.com/

Note: if your contribution is trivial and you think it may be exempt from the CLA, please post a short reply to this comment with details. http://docs.puppet.com/community/trivial_patch_exemption.html

@hunner hunner merged commit 5a332e7 into puppetlabs:master Feb 23, 2017
eputnam added a commit to eputnam/puppetlabs_spec_helper that referenced this pull request Feb 23, 2017
Z release for gettext task fix puppetlabs#180
@eputnam eputnam mentioned this pull request Feb 23, 2017
eputnam added a commit to eputnam/puppetlabs_spec_helper that referenced this pull request Feb 23, 2017
Z release for gettext task fix puppetlabs#180
@chelnak chelnak added the bug label Oct 24, 2022
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