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 php pear install #1

Open
wants to merge 1 commit into
base: dizzy
Choose a base branch
from
Open

Conversation

everlook
Copy link
Contributor

@everlook everlook commented Mar 2, 2017

The latest version of install-pear-nozlib.phar breaks in dizzy. Use a
cached copy in S3 as a quick fix.

Signed-off-by: Jeff Horn [email protected]

The latest version of install-pear-nozlib.phar breaks in dizzy. Use a
cached copy in S3 as a quick fix.

Signed-off-by: Jeff Horn <[email protected]>
@everlook everlook self-assigned this Mar 2, 2017
@everlook everlook requested review from cetola and fbertux March 2, 2017 23:43
Copy link
Contributor

@cetola cetola left a comment

Choose a reason for hiding this comment

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

Do you have a working SSTATE cache? That would make for a more robust solution. When supporting older releases, you should use SSTATE for packages that have not changed. That being said, this solution will work fine for now.

@everlook
Copy link
Contributor Author

everlook commented Mar 6, 2017

I rechecked today and PHP repackaged the pear installer to include the missing file. So, no need to merge this. I'll leave it here for future reference.

everlook added a commit that referenced this pull request Mar 13, 2017
* 7ea2486 Merge pull request #1 from added-a-mapping-parameter
* 417746b version number bump.  Renamed functions in translate_parser.c
* 8356168 Added a -m <max mapping size> parameter.

Signed-off-by: Jeff Horn <[email protected]>
fbertux added a commit that referenced this pull request Apr 12, 2017
This includes the following changes:

  - 7ea2486 Merge pull request #1 from jmore-reachtech/added-a-mapping-parameter
  - 417746b version number bump.  Renamed functions in translate_parser.c file.
  - 8356168 Added a -m <max mapping size> parameter.  Allows more than 400 translations to be defined in a translate.txt file.
  - 62434ff Added version info to the .pro file

Signed-off-by: Fabio Berton <[email protected]>
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