Talk:VCS package guidelines

From ArchWiki
Revision as of 13:15, 11 April 2013 by Det (talk | contribs)
Jump to: navigation, search

Updating a CVS repo

I don't use cvs. How can you describe the pkgver for cvs (for pacman 4.1)? --Dracorp (talk) 09:31, 6 April 2013 (UTC)

https:// vs git://

Could we consider a guideline to use firewall-friendly protocols when possible (e.g. instead of git:// --Mitch feaster 14:34, 15 November 2011 (EST)

Some leftover changes

Since Thestinger decided to protect the entire page (for whatever reason):

$SRCDEST is required for SVN

Thestinger's change (7:03, 10 April 2013 (UTC)):

→‎Subversion:  $SRCDEST not necessary

Wrong, see the note you left behind. svn export doesn't copy .svn, which means getting the version won't work anymore:

[$srcdir]$ svnversion
Unversioned directory
Thanks, reverted. -- thestinger (talk) 16:45, 10 April 2013 (UTC)

The Git functions

Danielwallace's change (04:32, 10 April 2013 (UTC)):

pkgver() {
   cd local_repo
−  git rev-list --count HEAD
+  echo $(git rev-list --count master).$(git rev-parse --short master)
- [...]

That's good but you left the title and the sample output intact. Also, the first example still uses git describe --always.

--Det (talk) 16:41, 10 April 2013 (UTC)

Incomplete Git function

The following function will list not only the total number of commits, but also the tag of the last commit

pkgver() {
  cd local_repo
  echo $(git rev-list --count master).$(git rev-parse --short master)


So the following should be used as the example title:

Using the total count of commits and the tag of the last commit.

And the example output should be changed to something like


--X33a (talk) 17:50, 10 April 2013 (UTC)

Or "tag of the last one" to prevent unnecessary repetition, but I fear even the most obvious changes like this will take a very long time now that we can't do them anymore. --Det (talk) 18:50, 10 April 2013 (UTC)
Thanks, Thestinger but the title still only mentions the first git command, while the other example should use git rev-parse as well. --Det (talk) 13:15, 11 April 2013 (UTC)