grammar adjustment in dpkg

Thank you for reporting this bug. It's almost ready for confirming. However, we'd like to know the version of Ubuntu and dpkg you're using.

This can be obtained by the following commands:
apt-cache policy dpkg
and for the Ubuntu version:
lsb_release -a
or graphically
System->About Ubuntu

Thanks in advance.

I don't think this is an actual bug. "as you request" is perfectly good English, meaning that it could cause problems, but dpkg is going ahead with removing it because you told it to.

Perhaps the reporter was thinking that "as you requested" was more appropriate.

Yes. That's what I was thinking. Anyway, I'm on a typical Intrepid install.

According to packages.ubuntu.com the current version of dpkg in Intrepid is 1.14.20ubuntu6
If you're using a different version, please leave a reply.

The report now contains enough information to be confirmed.

That's what I'm using :)

Here's a patch for it.

Thanks for your work on this.

Do you think you could forward the change to Debian? https://wiki.ubuntu.com/Bugs/Debian

(and please link the upstream bug here)

Should I link the upstream bug using "Also affects project" or "Also affects distribution"?

(The report is here: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=533171)

Use "Also affects distribution".

Thanks for your patch. I'm going to take this off the sponsorship list, though; for small changes like this it usually isn't worth an upload on its own. Furthermore, this is a translated string in a package for which we don't use language packs, and so simply changing the string in an Ubuntu upload would result in significant translation regressions; it's much better for the change to be made in Debian.

FWIW, you missed several instances of the same text:

src/errors.c:99: fprintf(stderr, _("Package %s was on hold, processing it anyway as you request\n"),
src/remove.c:130: _("dpkg: %s: dependency problems, but removing anyway as you request:\n%s"),
src/processarc.c:158: fprintf(stderr, _("Verification on package %s failed,\nbut installing anyway as you request.\n"), filename);
src/configure.c:135: _("dpkg: %s: dependency problems, but configuring anyway as you request:\n%s"),
src/configure.c:219: "Installing new config file as you request.\n"),
src/archives.c:1047: " remove anyway as you request.\n"), fixbyrm->name);

Thanks for the reply Colin, I've added a bug watch. Sorry I missed all those other instances, I'll fix them later on today, or tomorrow morning.

Here's the new patch. I'll send it off to the upstream report too.

Here's the new patch. I'll forward it to debian too.

Just to keep you all informed I'm still working on this in debian.

That's good :)

On Wed, 2009-06-17 at 16:59 +0000, David Stansby wrote:
> Just to keep you all informed I'm still working on this in debian.
>

My patch has been accepted upstream now, and has been merged into the git repository. Should be making an appearance downstream soon.

Just a thought: Since I have provided a fix, and forwarded it upstream etc. should I remove the bitesize tag so other people don't think this bug isn't being worked on?

I've just uploaded dpkg 1.15.3ubuntu1 to Karmic, including David's change which was applied in Debian. Thanks!