Skip to content

(PDOC-26) Rename Puppetx to PuppetX #25

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

HAIL9000
Copy link
Contributor

@HAIL9000 HAIL9000 commented Feb 6, 2015

Due to the changes in PUP-3900 which renamed Puppetx to PuppetX
and puppetx to puppet_x, strings was failing since the namespace
had not been updated in strings. In order to be compatible with this
change, update strings by renaming the namespace to match.

In order to ensure strings is still compatible with earlier versions
of puppet that don't include the namespace change, declare our own
PuppetX module. This way, if it is an older version and the namespace
is called Puppetx, the necessary PuppetX namespace will be created.

@HAIL9000
Copy link
Contributor Author

HAIL9000 commented Feb 6, 2015

(This is the new pull request which is the updated version of #24)

Due to the changes in PUP-3900 which renamed Puppetx to PuppetX
and puppetx to puppet_x, strings was failing since the namespace
had not been updated in strings. In order to be compatible with this
change, update strings by renaming the namespace to match.

In order to ensure strings is still compatible with earlier versions
of puppet that don't include the namespace change, declare our own
PuppetX module. This way, if it is an older version and the namespace
is called Puppetx, the necessary PuppetX namespace will be created.
@HAIL9000 HAIL9000 force-pushed the issue/master/PDOC-26_update_puppetx_module_name branch from f579a17 to e5691d2 Compare February 6, 2015 22:09
hlindberg added a commit that referenced this pull request Feb 7, 2015
…tx_module_name

(PDOC-26) Rename Puppetx to PuppetX
@hlindberg hlindberg merged commit d9d5ae6 into puppetlabs:master Feb 7, 2015
@chelnak chelnak added the bugfix label Oct 13, 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.

3 participants