[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: please review my template



On Wed, Nov 29, 2006 at 11:05:20PM +0100, Nicolas Boullis wrote:
> Hi,
> 
> On Wed, Nov 29, 2006 at 07:16:52PM +0000, MJ Ray wrote:
> > Dominic Hargreaves <dom@earth.li> wrote:
> > > I would use
> > > "deleted locally"
> > > "deleted remotely"
> > 
> > I would use those.  The hyphenated ones taste jargony.
> 
> OK. Should I also do this for the short descripton, as the attached 
> modified tamplate?

Yes. Well spotted :)

> > +1 to the s/used to //
> 
> OK. With the same reasonning, I also rephrased the next sentence to the 
> present tense.
> 
> 
> > Otherwise seems fine.
> 
> There is also this NEWS.Debian vs debconf note question...
> My reasonning for a debconf note was that:
>   1) there is a danger of data loss; a NEWS.Debian file is not vivible 
>      enough;
>   2) debconf offers an infrastructure for translations that NEWS.Debian 
>      lacks.
> 
> But I will happily listen to arguments for NEWS.Debian...

I think this is probably something to be discussed at a high level - ie
I always thought that NEWS items should be displayed to users by default
whenever possible. Personally I use apt-listchanges, but I guess most
people don't...

> Template: isync/upgrade_1.0
> Type: note
> _Description: isync won't refetch messages deleted locally
>  Versions of isync older than 1.0 refetch messages that were locally
>  deleted. Although it is a bug, some people may be using it as a
>  feature. Since 1.0, messages that are deleted locally will be deleted
>  remotely as well. People expecting the old buggy behavior will
>  experience data loss.

Looks good to me.

Cheers,

Dominic.

-- 
Dominic Hargreaves | http://www.larted.org.uk/~dom/
PGP key 5178E2A5 from the.earth.li (keyserver,web,email)



Reply to: