printing Din A4 format not working

Thank you very much for your solution here, I will take a look at this today or tomorrow

Till : Can we have your opinion on this?

philipp beitz : Did you try to print to see if it really fixed your issue? I see that changing Letter to A4 in cupsys works and seems to persist, however it does not change anything in /usr/Brother/inf files. I don't have a printer to test this so your feedback would be appreciated. Thanks!

it worked under gutsy so i guess it will work wit hardy i will test it if you want it...

Saïvann, can you patch all the PPD files appropriately (do it by some perl or sed lines in the debian/rules file) and build new packages, so that your drivers will work out of the box. Do this as quick as possible and I will try to get the fix into Hardy.

Till Kamppeter : Working on it today, not all packages need to be fixed, I'll do as quick and well as possible.

Till Kamppeter : brother-cups-wrapper-laser1 was the only one that needed to be fixed. I fixed the problem and my new package build successfully in my PPA. I attach a debdiff and the whole source package so you take the one that is easier for you.

Of course I forgot to fix the LP bug in my changelog, like I always do :) Here's the good debdiff.

Thanks for uploading. I will try this. What do you mean with Lp Bug?

philipp beitz : Thanks for your clear bug report, it's really appreciated. LP = Launchpad. When adding (LP: #<bugnumber>) in the changelog when packaging, it automatically close the bug when the update gets into repositories. Like you can see, it's still not uploaded/fixed, I'm not a MOTU so I can't do this but Till is and he will try to upload this before Hardy hit Final release.

Saïvann, thank you for doing this patch in time. I have improved it somewhat, making the debdiff even smaller and fixing not only PageSize but all options. The new debdiff is attached and the new packages are here:

http://www.linux-foundation.org/~till/tmp/ubuntu/hardy/brother-cups-wrapper-laser1/

MOTU release team, please take this fix into Hardy, as this makes the Brother printers finally working out-of-the-box. With this fix all the issues of the Brother drivers are fixed.

Saïvann, thank you again for your great work on packaging the Brother printer drivers.

No need to thank me :) i should thank you for fixing this problem so quick and just in time before releasing hardy.

actually, http://launchpadlibrarian.net/13775765/debdiff looks saner to me. Has this been tested yet?

I have tested both debdiffs and they really fix the PPDs in term of the PageSize option. And both do not introduce any new errors or warnings to the output of cupstestppd.

My version I have made for the following two reasons:

1. Saïvanns version applies to explicitly listed page sizes. If Brother supports more page sizes in the future, this version will not cover them => maintenance nightmare

2. My version covers also the other options, like MediaType or Resolution. So it is a more complete fix and it makes the Brother printers completely working with standard methods.

Attached is a slightly improved debdiff, in which the appending of the "" after each trailing colon is done only for lines which begin with an asterisk. This way only lines which really go into the PPD are modified. This is afer for future changes in the PPD generator scripts.

Ok for me then. Please upload.

(side note, it looks a little bit strange, that you keep Saivann in debian/changelog for the changelog entry, but I won't nitpick on this).

oh, note to archive-admins: please check with buildd admins before waving it through unapproved due to bug #220548, thanks.

I left Saïvann in as he did the fix and I only improved it. I will change it to my name and say thanks to Saïvann in the comment.

I will upload it with this new debdiff applied, fixing also bug #220548 to avoid any problems for the buildd.

Saïvann, can you correct this also in your other packages? Thanks.

Uploaded version with also the copyright file fixed.

Saïvann, please fix this also in your other packages.

Here's the debdiff for all cups wrapper packages that fix wrong copyright file, except the -extra one which is under the Brother license agreement.

This bug was fixed in the package brother-cups-wrapper-laser1 - 1.0.2-1-0ubuntu4

---------------
brother-cups-wrapper-laser1 (1.0.2-1-0ubuntu4) hardy; urgency=low

  * debian/rules: Add missing "" in option choice fields of the PPDs
    (LP: #219509). Thanks to Saïvann Carignan for finding this.
  * debian/rules: Removed more system manipulations from the generator
    scripts for the PPDs and filters (LP: #220548).
  * debian/copyright: Use verbatim license header of one of the generator
    scripts for PPDs and filters.

 -- Till Kamppeter <email address hidden> Tue, 22 Apr 2008 16:51:30 +0200