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

5.1 releases require ext-ftp #176

Closed
bytestream opened this Issue Mar 7, 2019 · 3 comments

Comments

Projects
None yet
2 participants
@bytestream
Copy link

bytestream commented Mar 7, 2019

This is perhaps a docs issue but if you install via composer then it requires ext-ftp, should probably be a suggest if you want to use the FTP component?

Problem 1
    - Installation request for sebastianfeldmann/ftp 0.9.2 -> satisfiable by sebastianfeldmann/ftp[0.9.2].
    - sebastianfeldmann/ftp 0.9.2 requires ext-ftp * -> the requested PHP extension ftp is missing from your system.
  Problem 2
    - sebastianfeldmann/ftp 0.9.2 requires ext-ftp * -> the requested PHP extension ftp is missing from your system.
    - phpbu/phpbu 5.1.9 requires sebastianfeldmann/ftp ~0.9 -> satisfiable by sebastianfeldmann/ftp[0.9.2].
    - Installation request for phpbu/phpbu 5.1.9 -> satisfiable by phpbu/phpbu[5.1.9].

@sebastianfeldmann sebastianfeldmann self-assigned this Mar 7, 2019

@bytestream

This comment has been minimized.

Copy link
Author

bytestream commented Mar 7, 2019

Issue was I installed on a machine with ext-ftp, pushed to git and another machine didn't have ext-ftp.

So if you install on a machine with ext-ftp then it locks sebastianfeldmann/ftp to 0.9.2, and if you install on a machine without then it locks to 0.9.1...

@sebastianfeldmann

This comment has been minimized.

Copy link
Owner

sebastianfeldmann commented Mar 7, 2019

I moved the sebastianfeldmann/ftp package to the suggest list
Now it is not installed by default anymore.
If you want to use the FTP sync, you have to require the lib manually from now on.

Version 5.1.10 is released with the described changes

@sebastianfeldmann

This comment has been minimized.

Copy link
Owner

sebastianfeldmann commented Mar 7, 2019

Thanks for reporting the issue :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.