hpcups.drv PPDs need *RequiresPageRegion attribute

[Tim Waugh]

To ensure the PCL page size command is sent we need to set RequiresPageRegion in generated PPDs. For explanation see:
  https://bugzilla.redhat.com/show_bug.cgi?id=518756#c8

[David Suffield (david-suffield)]

The attribute "*RequiresPageRegion All: True" should be generated by CUPSDDK (ie: ppdc).

I'm using cups-1.4-0.b2.18.fc11.x86_64 to create static hpcups PPDs and the attribute "*RequiresPageRegion All: True" is generated correctly.

This attribute is also generated dynamically with cupsddk 1.2.3.

[Tim Waugh (twaugh)]

CUPS 1.4.1 does not give this output. The code that emitted it has been explicitly removed in subversion revision 8603 while fixing STR #3152:

@@ -931,9 +932,6 @@ ppdcDriver::write_ppd_file(
       cupsFilePrintf(fp, "*ParamCustomPageSize Orientation: 5 int 0 0%s", lf);
   }

- if (type != PPDC_DRIVER_PS && !find_attr("RequiresPageRegion", NULL))
- cupsFilePrintf(fp, "*RequiresPageRegion All: True%s", lf);
-
   // All other options...
   for (g = (ppdcGroup *)groups->first(); g; g = (ppdcGroup *)groups->next())
   {

[David Suffield (david-suffield)]

Interesting :) I can blame STR #3152 on Smith Kennedy (my counterpart)...

Ok, I will add this attribute to my drv.

[Till Kamppeter (till-kamppeter)]

I have checked the current version (3.11.1) and the .drv file contains the 'Attribute "RequiresPageRegion" "All" "True"' and the generated PPD files contain '*RequiresPageRegion All: True' so the bug seems to be fixed upstream (and in Natty).