Bug#892142: debian-policy: update example to use default-mta instead of exim

2018-03-30 Thread Holger Levsen 
On Mon, Mar 05, 2018 at 05:12:16PM -0800, Jonathan Nieder wrote:
> Thanks!  This is non-normative, so a policy editor can just make the
> change.

I'll second it nonetheless :)

> How about this patch?  It should be possible to apply by downloading
> this message as an mbox and using "git am --scissors mbox-file".
> 
> -- >8 --
> Subject: Use default-mta in MTA dependency example
> 
> Reported-by: Paul Wise 
> Closes: #892142
> ---
>  debian/changelog| 5 +
>  policy/ch-relationships.rst | 2 +-
>  2 files changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/debian/changelog b/debian/changelog
> index 0ec082f..99f6068 100644
> --- a/debian/changelog
> +++ b/debian/changelog
> @@ -1,5 +1,6 @@
>  debian-policy (4.1.4.0) UNRELEASED; urgency=medium
>  
> +  [ Sean Whitton ]
>* Policy: Drop get-orig-source rules target
>  Wording: Helmut Grohne 
>  Seconded: Holger Levsen 
> @@ -29,6 +30,10 @@ debian-policy (4.1.4.0) UNRELEASED; urgency=medium
>* Fix indentation of description of the clean target (Closes: #889960).
>  Thanks Ferenc Wágner for the report.
>  
> +  [ Jonathan Nieder ]
> +  * Use default-mta instead of exim in dependency example (Closes: #892142).
> +Thanks to Paul Wise for the report.
> +
>   -- Sean Whitton   Fri, 29 Dec 2017 18:46:59 +
>  
>  debian-policy (4.1.3.0) unstable; urgency=medium
> diff --git a/policy/ch-relationships.rst b/policy/ch-relationships.rst
> index 77cf0e1..1eaa422 100644
> --- a/policy/ch-relationships.rst
> +++ b/policy/ch-relationships.rst
> @@ -45,7 +45,7 @@ For example, a list of dependencies might appear as:
>  
>  Package: mutt
>  Version: 1.3.17-1
> -Depends: libc6 (>= 2.2.1), exim | mail-transport-agent
> +Depends: libc6 (>= 2.2.1), default-mta | mail-transport-agent
>  
>  Relationships may be restricted to a certain set of architectures. This
>  is indicated in brackets after each individual package name and the
> -- 
 
seconded.


-- 
cheers,
Holger


signature.asc
Description: PGP signature

Bug#892142: debian-policy: update example to use default-mta instead of exim

2018-03-06 Thread Sean Whitton 
control: tag -1 +pending

Hello,

On Mon, Mar 05 2018, Jonathan Nieder wrote:

> This is non-normative, so a policy editor can just make the change.

Indeed.

> How about this patch?  It should be possible to apply by downloading
> this message as an mbox and using "git am --scissors mbox-file".

Or `M-x notmuch-pipe ( cd ~/src/debian-policy && git am --scissors )` :)

... except that it doesn't actually apply to the changelog because there
is fuzz, and `git am` is apparently intolerant of any fuzz.

`patch -p1 <.git/rebase-apply/patch && git add . && git am --continue`
did the trick.

Thank you for the patch.

-- 
Sean Whitton


signature.asc
Description: PGP signature

Bug#892142: debian-policy: update example to use default-mta instead of exim

2018-03-05 Thread Paul Wise 
On Mon, 2018-03-05 at 17:12 -0800, Jonathan Nieder wrote:

> How about this patch?

Looks good to me.

-- 
bye,
pabs

https://wiki.debian.org/PaulWise


signature.asc
Description: This is a digitally signed message part

Bug#892142: debian-policy: update example to use default-mta instead of exim

2018-03-05 Thread Jonathan Nieder 
tags 892142 + patch
quit

Hi,

Paul Wise wrote:

> In Policy 7.1. Syntax of relationship fields, there is an example:
>
>For example, a list of dependencies might appear as:
>
>Package: mutt
>Version: 1.3.17-1
>Depends: libc6 (>= 2.2.1), exim | mail-transport-agent
>
> I would like to suggest that this be updated to replace exim with
> default-mta, since exim is no longer in Debian as it has been replaced
> with exim4 and many years ago we switched to using the default-mta
> virtual package to make it easier to change the default MTA on Debian.
>
> https://www.debian.org/doc/debian-policy/#syntax-of-relationship-fields

Thanks!  This is non-normative, so a policy editor can just make the
change.

How about this patch?  It should be possible to apply by downloading
this message as an mbox and using "git am --scissors mbox-file".

-- >8 --
Subject: Use default-mta in MTA dependency example

Reported-by: Paul Wise 
Closes: #892142
---
 debian/changelog| 5 +
 policy/ch-relationships.rst | 2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/debian/changelog b/debian/changelog
index 0ec082f..99f6068 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,5 +1,6 @@
 debian-policy (4.1.4.0) UNRELEASED; urgency=medium
 
+  [ Sean Whitton ]
   * Policy: Drop get-orig-source rules target
 Wording: Helmut Grohne 
 Seconded: Holger Levsen 
@@ -29,6 +30,10 @@ debian-policy (4.1.4.0) UNRELEASED; urgency=medium
   * Fix indentation of description of the clean target (Closes: #889960).
 Thanks Ferenc Wágner for the report.
 
+  [ Jonathan Nieder ]
+  * Use default-mta instead of exim in dependency example (Closes: #892142).
+Thanks to Paul Wise for the report.
+
  -- Sean Whitton   Fri, 29 Dec 2017 18:46:59 +
 
 debian-policy (4.1.3.0) unstable; urgency=medium
diff --git a/policy/ch-relationships.rst b/policy/ch-relationships.rst
index 77cf0e1..1eaa422 100644
--- a/policy/ch-relationships.rst
+++ b/policy/ch-relationships.rst
@@ -45,7 +45,7 @@ For example, a list of dependencies might appear as:
 
 Package: mutt
 Version: 1.3.17-1
-Depends: libc6 (>= 2.2.1), exim | mail-transport-agent
+Depends: libc6 (>= 2.2.1), default-mta | mail-transport-agent
 
 Relationships may be restricted to a certain set of architectures. This
 is indicated in brackets after each individual package name and the
-- 
2.16.2.660.g709887971b

Bug#892142: debian-policy: update example to use default-mta instead of exim

2018-03-05 Thread Paul Wise 
Package: debian-policy
Version: 4.1.3.0
Severity: minor

In Policy 7.1. Syntax of relationship fields, there is an example:

   For example, a list of dependencies might appear as:

   Package: mutt
   Version: 1.3.17-1
   Depends: libc6 (>= 2.2.1), exim | mail-transport-agent

I would like to suggest that this be updated to replace exim with
default-mta, since exim is no longer in Debian as it has been replaced
with exim4 and many years ago we switched to using the default-mta
virtual package to make it easier to change the default MTA on Debian.

https://www.debian.org/doc/debian-policy/#syntax-of-relationship-fields

-- System Information:
Debian Release: buster/sid
  APT prefers testing-debug
  APT policy: (900, 'testing-debug'), (900, 'testing'), (800, 
'unstable-debug'), (800, 'unstable'), (790, 'buildd-unstable'), (700, 
'experimental-debug'), (700, 'experimental'), (690, 'buildd-experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 4.14.0-3-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_AU.utf8, LC_CTYPE=en_AU.utf8 (charmap=UTF-8), 
LANGUAGE=en_AU.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages debian-policy depends on:
ii  libjs-sphinxdoc  1.6.7-1

debian-policy recommends no packages.

Versions of packages debian-policy suggests:
pn  doc-base  

-- no debconf information

-- 
bye,
pabs

https://wiki.debian.org/PaulWise


signature.asc
Description: This is a digitally signed message part