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

Bug#391023: new version of the patch (documenting XS-Vcs-Browser as well)



On Mon, Feb 19, 2007 at 10:44:07AM +0100, Stefano Zacchiroli wrote:
> On Wed, Oct 04, 2006 at 12:44:44PM +0200, Stefano Zacchiroli wrote:
> > The attached patch for the developers reference document the support for
> > the Vcs-* family of fields.
> 
> I attach a new version of the patch for the developer's reference.
> 
> The main change from the previous patch is that now two fields are
> supported: XS-Vcs-Browser (for a browsable version of the repository)
> and XS-Vcs-* (same semantics as before).  I also implemented some of the
> other comments received in this bug report and in past threads on
> -devel.
> 
> Re-reading from a native English speaker would be really appreciated.
It looks good to me with the changes below.

Cheers
Justin

> +      <p>
> +      The information is meant to be useful for the final user, willing to
> +      browse the latest work done on the package (e.g. when looking for the
> +      patch fixinv a bug tagged as <tt>pending</tt> in the bug tracking
               ^^^^^^
               fixing

> +      system).

> +      a package from the VCS sources. Other uses of this information might
> +      include automatic building of the latest VCS version of the given
> +      package, to this end the location pointed to by the field should better
               ^^^
Consider using a semicolon or period.

> +      be version agnostic and point to the main branch (for VCSs supporting
> +      such a concept). Also, the location pointed to should be accessible to
> +      the final user, fulfilling this requirement might imply pointing to an
                      ^^^
Consider using a semicolon or period.



Reply to: