Ensure all commands fail early if patch file is not specified in options

Issue #8 new
Dave Reid repo owner created an issue

If drush_get_option('patch-file') is empty we should fail hard at the beginning of every command. Currently I don't think that is happening, or things are silently failing.

Comments (3)

  1. Miloš Kroulík

    Yes, thats exactly what I'm experiencing. The output is like this:

    [module] patched with [patchfile.patch].                                                                                                                                                                                        [success]
    file_put_contents(): Filename cannot be empty patchfile.drush.inc:512                                                                                                                                                                          [warning]
    Unable to append to file .                                                                                                                                                                                                                     [error]
    
  2. Stanislav Popov

    Also would be good to check that patchfile exists. It allows to define patch-file in global config and affects only sites that has patchfile.

  3. Log in to comment