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

[needs-rebase] Net_SMTP, Auth_SASL, Net_Socket - Load via composer #7361

Closed
wants to merge 1 commit into from

Conversation

totten
Copy link
Member

@totten totten commented Dec 2, 2015

Note: pear.php.net is in the middle of a multiday outage, so downloading
PEAR packages is a bit flaky. For Net_Socket and Auth_SASL, we
work-around the outage by referencing specific git commits in
composer.json. Fortunately, these packages are small and don't change
much, so these should be pretty close to the latest stable.

CRM-20561


Note: `pear.php.net` is in the middle of a multiday outage, so downloading
PEAR packages is a bit flaky.  For `Net_Socket` and `Auth_SASL`, we
work-around the outage by referencing specific git commits in
`composer.json`.  Fortunately, these packages are small and don't change
much, so these should be pretty close to the latest stable.
@totten
Copy link
Member Author

totten commented Dec 2, 2015

Looks like Net_SMTP v1.7.1 isn't installable on PHP 5.3 (although it might work... git log shows they updated just to match a PEAR guideline). Switching back to v1.6.x -- we were using 1.6.x before, and it seems to work with PHP 5.3.

@eileenmcnaughton eileenmcnaughton changed the title Net_SMTP, Auth_SASL, Net_Socket - Load via composer [needs-rebase] Net_SMTP, Auth_SASL, Net_Socket - Load via composer Jan 8, 2016
@eileenmcnaughton
Copy link
Contributor

I have updated the title of this PR to reflect the fact this PR is stale & needs rebasing. Please update the title once you have done this :-)

@eileenmcnaughton
Copy link
Contributor

@totten I'm going to close this - by the time it needs a rebase it's probably better just to start again. If this is worth keeping track of you should open a ticket but if it is not utterly below your caring threshhold it can drift into nothingness

@xurizaemon
Copy link
Member

Since there was existing work here, I raised it as CRM-20561

But we should use Net_SMTP 1.7.3 or better, since otherwise we will break sites using TLS on PHP >= 5.6.7 (CRM-18408).

@seamuslee001
Copy link
Contributor

@xurizaemon the problem with 1.7.3 is it requires php 5.4 which is beyond our minimum

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.

5 participants