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: