[Pkg-fedora-ds-maintainers] Bug#679870: Bug#679870: [RFR] templates://389-ds-base/{389-ds-base.templates}

[Timo Aaltonen]

On 03.07.2012 01:14, Christian PERRIER wrote:
> Please find, for review, the debconf templates for the 389-ds-base
> source package.
> 
> Given the current freeze, this review will  speed up so that we can
> soon move to the translation step.
> 
> This review will last from Monday, July 02, 2012 to July 05, 2012.
> 
> Please send reviews as unified diffs (diff -u) against the original
> files. Comments about your proposed changes will be appreciated.
> 
> Your review should be sent as an answer to this mail.
> 
> When appropriate, I will send intermediate requests for review, with
> "[RFRn]" (n>=2) as a subject tag.
> 
> When we will reach a consensus, I send a "Last Chance For
> Comments" mail with "[LCFC]" as a subject tag.
> 
> Finally, a summary will be sent to the review bug report,
> and a mail will be sent to this list with "[BTS]" as a subject tag.
> 
> Rationale:
> --- 389-ds-base.old/debian/389-ds-base.templates	2012-07-02 03:53:09.083979633 -0400
> +++ 389-ds-base/debian/389-ds-base.templates	2012-07-02 18:06:21.372227262 -0400
> @@ -1,5 +1,5 @@
>  Template: 389-ds-base/setup
>  Type: note
> -_description: Setup
> - Please use the /usr/sbin/setup-ds program to setup your 389 Directory
> +_Description: Setup needed for 389 directory server
> + Please use the /usr/sbin/setup-ds program to setup the 389 Directory
>   Server instance.
> 
> Indeed, this template is a good exampleof what we usually call
> "debconf abuse". Such notes should never be used for that kind of
> information. Rather, it pertains to README.Debian.
> 
> Still, I reworded it a little bit as I'm unsure that removing the
> template would still qualify the package for a freeze exception.
> 
> I changed the note title to something more meaningful. I also propose
> dropping "your": we commonly discourage the use of "personnalization" in
> such cases.

I've dropped it nevertheless, since the same info was mentioned in
README.Debian (just with too long lines, so rewrapped it).

Fixed in git, waiting for a sponsor.


-- 
t