Bug#881432: debian-policy: Please clarify postinst invocation upon first installation

2017-11-11 Thread Sean Whitton
control: tag -1 -patch

Hello again,

On Sat, Nov 11 2017, Christoph Biedl wrote:

> 6.5. Summary of ways maintainer scripts are called
>
> in the paragraph after
>
> "postinst configure most-recently-configured-version"
>
> Suggested wording:
>
> | If this package was prevously uninstalled, the
> | "most-recently-configured-version" string is empty.

Sorry, I spoke too soon.  I withdraw my second.  Currently we have:

> If there is no most recently configured version ``dpkg`` will pass a
> null argument.

This seems to cover your case, but maybe you don't think it's
sufficient?

Further, in dpkg terminology packages aren't "uninstalled" but they are
"removed".  So even if we make a change your wording would not be
advisable.

-- 
Sean Whitton


signature.asc
Description: PGP signature


Bug#881432: debian-policy: Please clarify postinst invocation upon first installation

2017-11-11 Thread Mattia Rizzolo
On Sat, Nov 11, 2017 at 07:08:41PM +0100, Christoph Biedl wrote:
> 6.5. Summary of ways maintainer scripts are called
> 
> in the paragraph after
> 
> "postinst configure most-recently-configured-version"
> 
> Suggested wording:
> 
> | If this package was prevously uninstalled, the
> | "most-recently-configured-version" string is empty.

Seconded.

> Reading src/configure.c in dpkg, this should be technically correct.

Isn't there some dpkg manpage or document also nicely documenting this
somewhere?  I know guillem went to great effort of documenting how dpkg
behaves.

-- 
regards,
Mattia Rizzolo

GPG Key: 66AE 2B4A FCCF 3F52 DA18  4D18 4B04 3FCD B944 4540  .''`.
more about me:  https://mapreri.org : :'  :
Launchpad user: https://launchpad.net/~mapreri  `. `'`
Debian QA page: https://qa.debian.org/developer.php?login=mattia  `-


signature.asc
Description: PGP signature


Bug#881432: debian-policy: Please clarify postinst invocation upon first installation

2017-11-11 Thread Sean Whitton
control: -1 +patch

Hello Christoph,

On Sat, Nov 11 2017, Christoph Biedl wrote:

> 6.5. Summary of ways maintainer scripts are called
>
> in the paragraph after
>
> "postinst configure most-recently-configured-version"
>
> Suggested wording:
>
> | If this package was prevously uninstalled, the
> | "most-recently-configured-version" string is empty.

Seconded.

-- 
Sean Whitton


signature.asc
Description: PGP signature


Bug#881432: debian-policy: Please clarify postinst invocation upon first installation

2017-11-11 Thread Christoph Biedl
Package: debian-policy
Version: 4.1.1.1
Severity: wishlist

Hello,

the question of how to tell in postinst whether this is an upgrade or a
first installation comes up every now and then. Again yesterday in
#debian-mentors. Therefore I'm asking for a clarifcation in

6.5. Summary of ways maintainer scripts are called

in the paragraph after

"postinst configure most-recently-configured-version"

Suggested wording:

| If this package was prevously uninstalled, the
| "most-recently-configured-version" string is empty.

Reading src/configure.c in dpkg, this should be technically correct.

Also testing for an empty $2 in postinst is common according to
codesearch[1].

Regards,
Christoph

[1] 
https://codesearch.debian.net/search?q=-z+%22%5C%242%22+path%3Adebian%2F*.postinst


signature.asc
Description: Digital signature