Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Change auto_discover default to 1. #5

Merged
merged 1 commit into from Apr 17, 2012

Conversation

Projects
None yet
3 participants
@ghost

ghost commented Apr 2, 2012

Channel auto discovery is an essential part of PEAR installations and it's very inconvenient to have to manually add all the required channels. This setting should be on by default.

Owner

CloCkWeRX commented Apr 6, 2012

@helgi @saltybeagle - I'm for this behaviour; what do you think

fabpot commented Apr 6, 2012

This change would be wonderful. 👍

Owner

saltybeagle commented Apr 6, 2012

No objection from me. I think security was the original concern, but the explosion of channels has resulted in this being more of an inconvenience.
@cweiske and @cellog Do you have any major concerns?

@ghost

ghost commented Apr 17, 2012

Any update? (pun unintended :)

@CloCkWeRX CloCkWeRX added a commit that referenced this pull request Apr 17, 2012

@CloCkWeRX CloCkWeRX Merge pull request #5 from drak/patch-1
Change auto_discover default to 1.
c1dacc3

@CloCkWeRX CloCkWeRX merged commit c1dacc3 into pear:master Apr 17, 2012

Owner

CloCkWeRX commented Apr 17, 2012

I've merged it, if late-comers want it removed a revert prior to release should be easy :)

@cweiske cweiske pushed a commit that referenced this pull request Feb 24, 2015

Christian Weiske another try #5 82c5295
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment