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

Include the pdfjschildbootstrap.js file in the output for gulp mozcentral builds (PR 8023 follow-up) #8067

Merged
merged 1 commit into from
Feb 13, 2017
Merged

Include the pdfjschildbootstrap.js file in the output for gulp mozcentral builds (PR 8023 follow-up) #8067

merged 1 commit into from
Feb 13, 2017

Conversation

Snuffleupagus
Copy link
Collaborator

Yet another thing that I unfortunately missed during review of PR #8023.

Note that previously, in make.js this file was being preprocessed, however as far as I can tell that wasn't actually necessary. Hence this patch just copies the file to the proper output directory.

Fixes https://bugzilla.mozilla.org/show_bug.cgi?id=1338395#c8.

/cc @rvandermeulen

…central` builds (PR 8023 follow-up)

*Yet another thing that I unfortunately missed during review of PR 8023.*

Note that previously, in `make.js` this file was being preprocessed, however as far as I can tell that wasn't actually necessary. Hence this patch just copies the file to the proper output directory.

Fixes https://bugzilla.mozilla.org/show_bug.cgi?id=1338395#c8.
Copy link
Contributor

@yurydelendik yurydelendik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the patch.

@yurydelendik yurydelendik merged commit b509a3f into mozilla:master Feb 13, 2017
@Snuffleupagus Snuffleupagus deleted the gulp-mozcentral-pdfjschildbootstrap.js branch February 13, 2017 22:50
movsb pushed a commit to movsb/pdf.js that referenced this pull request Jul 14, 2018
…fjschildbootstrap.js

Include the `pdfjschildbootstrap.js` file in the output for `gulp mozcentral` builds (PR 8023 follow-up)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants