I get the following error when I try to set the patch file storage path to sites/all/patches

    * warning: mkdir() [function.mkdir]: No such file or directory in /var/www/mysite/sites/all/modules/patch_manager/patch_manager.module on line 139.
    * Unable to create patch path

I do get a success message:

The configuration options have been saved.

... and the path setting seems to be retained when I revisit the configuration (admin/build/patch). Still the warning is there and might confuse users.

Comments

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented

ping?

aidanlis’s picture

aidanlis commented
Status: Active » Fixed

Fixed in the new release.

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented

In previous dev versions I see a 'Patch file storage path:' field available in the module's settings page (../admin/build/patch/configure). Now, all I see is a 'Patch binary path:' that defaults to /usr/bin/patch.

Where is the option to select path of stored patches? Is it hardcoded? If yes does it default to sites/all/patches or what? Am I missing something?

aidanlis’s picture

aidanlis commented

You can set this now by editing the CCK field, which makes it much more flexible and will work with other modules like filefield_*.

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented

...I just realized what you mean. One has to edit the 'Patch' content type's field_patch CCK field in ../admin/content/types/patch/fields/field_patch in the 'Path settings' fieldset (defaults to sites/default/files/patches and has tokens support!).

We definitely need documentation.

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented

...still, it wouldn't hurt to have a setting in the module's configuration page. It's the first place people look for setting options like this one.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.