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

Added docs to package fix issue #64 #66

Closed

Conversation

christofdamian
Copy link

I were not able to fix #64 without moving everything around. So here is an pull request doing just that.
I quite understand if you don't like to do it.

@mlively
Copy link
Collaborator

mlively commented Jul 12, 2012

This will definitely work for pear and composer, unfortunately it would break anybody that was using the older symfony2 vendor model since the src directory is changing. I am going to play around with this and see if I can figure out a way to get the best of both worlds.

@mlively
Copy link
Collaborator

mlively commented Jul 12, 2012

Does the fedora rpm stuff utilize pear?

@christofdamian
Copy link
Author

Yes, usually PHP packages are repacked from the PEAR packages where available.
When the tests are included they will be run when the package is build for addition safety. A lot of packages don't have any tests though.

@bfeaver
Copy link
Collaborator

bfeaver commented Aug 3, 2013

I'm closing this pull request and the accompanying issue #64. With composer coming into existence and us not really maintaining the PEAR package, there doesn't seem much issue just letting this lie around.

I'll leave it up to @mlively if he wants to reopen.

@christofdamian
Copy link
Author

Obviously I think this is a mistake.
PEAR is not only a useful distribution channel, that is used by many people including Linux distributions packagers.
It is also a way of improving the quality of software packages by not only targeting composer.

I'll will have to see how and if I manage for Fedora, as I personally don't use Phake any more at work.

@mlively
Copy link
Collaborator

mlively commented Aug 6, 2013

I still think I can manage this when time permits. At least until there is as clear a path from composer to package distributions as there is for pear. At some point I will have to check and figure out why the pull request breaks. Seeings as I can't use this pull request I'll keep the request closed but I'll reopen the underlying issue.

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.

Include LICENSE , README and CHANGELOG in package.xml
3 participants