Varnish undefined constant errors on all pressflow pages

[Benjamin Lowenstein]

Using stock 64-bit 1.1 beta-2 AMI on Amazon, every pressflow page shows three PHP errors:

    * Notice: Use of undefined constant VARNISH_NO_CLEAR - assumed 'VARNISH_NO_CLEAR' in include_once() (line 4 of /var/www/sites/all/modules/varnish/varnish.module).
    * Notice: Use of undefined constant VARNISH_DEFAULT_CLEAR - assumed 'VARNISH_DEFAULT_CLEAR' in include_once() (line 5 of /var/www/sites/all/modules/varnish/varnish.module).
    * Notice: Use of undefined constant VARNISH_SELECTIVE_CLEAR - assumed 'VARNISH_SELECTIVE_CLEAR' in include_once() (line 6 of /var/www/sites/all/modules/varnish/varnish.module).

I looked in varnish.module and indeed, it has:

define(VARNISH_NO_CLEAR, 0);
define(VARNISH_DEFAULT_CLEAR, 1);
define(VARNISH_SELECTIVE_CLEAR, 2); // Requires Expire.module to be enabled.

so I edited the file to be:

define("VARNISH_NO_CLEAR", 0);
define("VARNISH_DEFAULT_CLEAR", 1);
define("VARNISH_SELECTIVE_CLEAR", 2); // Requires Expire.module to be enabled.

And the errors disappeared.

[Josh Koenig (joshkoenig)]

I will get these changes pushed upstream to the module. It's odd that this didn't show up before in testing. Have you altered your PHP version or error logging in any way?

[Aaron Levy (aaronlevy)]

This was already fixed upstream:
http://drupalcode.org/viewvc/drupal/contributions/modules/varnish/varnish.module?r1=1.17&r2=1.18

[Benjamin Lowenstein (ben-colingo)]

Nope, I didn't do anything AFAIK - I just spun up a large instance from the AWS console

[Benjamin Lowenstein (ben-colingo)]

also occurs with the 32-bit 1.1 AMI