Skip to content
This repository has been archived by the owner on Feb 1, 2024. It is now read-only.

Check existence of patch(1) #35

Open
arnested opened this issue Nov 17, 2014 · 2 comments
Open

Check existence of patch(1) #35

arnested opened this issue Nov 17, 2014 · 2 comments

Comments

@arnested
Copy link
Contributor

In bandaid.drush.inc we should check whether the patch(1) command actually exists.

I just discovered a CentOS server that came without the patch command installed. The error message from Bandaid was just "Could not apply patch" (which is of course not false but I guess we could be more helpful).

@xendk
Copy link
Owner

xendk commented Jan 28, 2015

Suggestions on how to do that?

@arnested
Copy link
Contributor Author

Hmmm... Getting the cause of a failed patch doesn't look easy.

We could change the last part of _bandaid_attempt_patch() and print $output to the screen if patching failed. That could provide some kind of hint no matter what caused an error.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants