Comment 8 for bug 668895

Revision history for this message
jazzynico (jazzynico) wrote :

The last patch was needed because the absolute path test was inverted [1]. It just removed the ! in the first test so that absolute paths pass the test as expected, and not relative ones. It was tested successfully on Windows XP and Ubuntu 10.10.
IMHO, removing the <check> in the INX files is a valid workaround. But I wonder what this <check> element is meant for. The extensions already have a <dependency> element that do almost exactly the same (but doesn't fail).

[1] Original script.cpp file, line 238:
    //Don't search when it is an absolute path. */
    if (!Glib::path_is_absolute(command)) {
        if (Glib::file_test(command, Glib::FILE_TEST_EXISTS)) {
            return true;
        } else {
            return false;
        }
    }