Re: svn:keywords is not set on newer packages

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



On Tue, Oct 27, 2009 at 4:13 AM, Eric Bélanger <snowmaniscool@xxxxxxxxx>wrote:

> On Tue, Oct 27, 2009 at 1:38 AM, Evangelos Foutras <foutrelis@xxxxxxxxx>
> wrote:
> > Eric Bélanger wrote:
> >>
> >> On Mon, Oct 26, 2009 at 10:39 PM, Evangelos Foutras <
> foutrelis@xxxxxxxxx>
> >> wrote:
> >>
> >>>
> >>> Hello,
> >>>
> >>> The $Id$ keyword came up while talking with Angel and I realized that
> the
> >>> svn:keywords property doesn't get set on PKGBUILD for new packages, and
> >>> thus
> >>> $Id$ doesn't get updated on commit. I believe we should fix this, as
> this
> >>> information is useful when determining when a PKGBUILD was last
> modified
> >>> and
> >>> by whom.
> >>>
> >>> May I suggest we do the following:
> >>>
> >>> 1) Edit the "Adding a package" section [1] of the packaging
> instructions
> >>> in
> >>> the DeveloperWiki to include this step. (Adding `svn propset
> svn:keywords
> >>> "Id" new-package/trunk/PKGBUILD' below the $EDITOR line perhaps?)
> >>> 2) Add this property to all packages (this has been done once before by
> >>> Aaron, if I remember correctly).
> >>>
> >>> What's your view on this?
> >>>
> >>> ----
> >>> [1]
> >>>
> >>>
> http://wiki.archlinux.org/index.php/DeveloperWiki:HOWTO_Be_A_Packager#Adding_a_package
> >>>
> >>>
> >>
> >> We should  do #1.  The Id tag should be set when a package is added to
> >> the repo.  As for option #2, it has already been done.
> >>
> >
> > Unless I'm missing something, it has only been done once, on 2008-04-18.
> > Packages added after that point do not have the svn:keywords property set
> > (e.g.: aufs2).
>
> Yes, that's what I meant.  It was done once on the whole repos when it
> was moved to svn.  Devs and TUs who adds packages need to run the svn
> propset command to set the Id tag.
>
> >
> > (Thanks for amending the developer wiki, by the way.)
> >
>

What about SVN autoprops, can't you just attach a "pre-made"
'~/.subversion/config' in the wiki and people can grab it from there.


-- 
Flávio Coutinho da Costa


[Index of Archives]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux