Difference between revisions of "Talk:VCS package guidelines"

From ArchWiki
Jump to: navigation, search
Line 1: Line 1:
== Comments ==
+
== Updating a CVS repo ==
 
I don't use cvs. How can you describe the pkgver for cvs (for pacman 4.1)? --[[User:Dracorp|Dracorp]] ([[User talk:Dracorp|talk]]) 09:31, 6 April 2013 (UTC)
 
I don't use cvs. How can you describe the pkgver for cvs (for pacman 4.1)? --[[User:Dracorp|Dracorp]] ([[User talk:Dracorp|talk]]) 09:31, 6 April 2013 (UTC)
  
== Suggestions ==
+
== https:// vs git:// ==
  
 
Could we consider a guideline to use firewall-friendly protocols when possible (e.g. https://github.com/matplotlib/matplotlib.git instead of git://github.com/matplotlib/matplotlib.git)?
 
Could we consider a guideline to use firewall-friendly protocols when possible (e.g. https://github.com/matplotlib/matplotlib.git instead of git://github.com/matplotlib/matplotlib.git)?
 
--[[User:Mitch feaster|Mitch feaster]] 14:34, 15 November 2011 (EST)
 
--[[User:Mitch feaster|Mitch feaster]] 14:34, 15 November 2011 (EST)
  
----
+
== Some leftover changes ==
 +
 
 
Since [[User:Thestinger|Thestinger]] decided to [https://wiki.archlinux.org/index.php?title=VCS_PKGBUILD_Guidelines&diff=253547&oldid=253546 protect] the entire page (for whatever reason):
 
Since [[User:Thestinger|Thestinger]] decided to [https://wiki.archlinux.org/index.php?title=VCS_PKGBUILD_Guidelines&diff=253547&oldid=253546 protect] the entire page (for whatever reason):
  
<s>1) Thestinger's [https://wiki.archlinux.org/index.php?title=VCS_PKGBUILD_Guidelines&diff=253555&oldid=253552 change] (7:03, 10 April 2013 (UTC)):</s>
+
=== <s>$SRCDEST <u>is</u> required for SVN</s> ===
 +
 
 +
Thestinger's [https://wiki.archlinux.org/index.php?title=VCS_PKGBUILD_Guidelines&diff=253555&oldid=253552 change] (7:03, 10 April 2013 (UTC)):
 
  ''→‎Subversion:  $SRCDEST not necessary''
 
  ''→‎Subversion:  $SRCDEST not necessary''
  
Line 20: Line 23:
 
::Thanks, reverted. -- [[User:Thestinger|thestinger]] ([[User talk:Thestinger|talk]]) 16:45, 10 April 2013 (UTC)
 
::Thanks, reverted. -- [[User:Thestinger|thestinger]] ([[User talk:Thestinger|talk]]) 16:45, 10 April 2013 (UTC)
  
2) [[Danielwallace]]'s [https://wiki.archlinux.org/index.php?title=VCS_PKGBUILD_Guidelines&diff=253546&oldid=253520 change] (04:32, 10 April 2013 (UTC)):
+
=== The Git functions ===
 +
 
 +
[[Danielwallace]]'s [https://wiki.archlinux.org/index.php?title=VCS_PKGBUILD_Guidelines&diff=253546&oldid=253520 change] (04:32, 10 April 2013 (UTC)):
 
{{hc|pkgver() {
 
{{hc|pkgver() {
 
   cd local_repo
 
   cd local_repo
Line 28: Line 33:
 
- [...]
 
- [...]
 
|1142}}
 
|1142}}
That's good but you left the title and the sample output intact. Also, the first example still uses {{ic|git describe --always}}.
+
That's good but you left the title and the <s>sample output</s> intact. Also, the first example still uses {{ic|git describe --always}}.
  
 
--[[User:Det|Det]] ([[User talk:Det|talk]]) 16:41, 10 April 2013 (UTC)
 
--[[User:Det|Det]] ([[User talk: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
 
The following function will list not only the total number of commits, but also the tag of the last commit
 
{{hc|pkgver() {
 
{{hc|pkgver() {
Line 44: Line 49:
 
{{ic|Using the total count of commits and the tag of the last commit. }}
 
{{ic|Using the total count of commits and the tag of the last commit. }}
  
And the example output should be changed to something like
+
<s>And the example output should be changed to something like</s>
  
  1142.a17a017
+
  <s>1142.a17a017</s>
  
 
--[[User:X33a|X33a]] ([[User talk:X33a|talk]]) 17:50, 10 April 2013 (UTC)
 
--[[User:X33a|X33a]] ([[User talk: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. --[[User:Det|Det]] ([[User talk:Det|talk]]) 18: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. --[[User:Det|Det]] ([[User talk:Det|talk]]) 18:50, 10 April 2013 (UTC)
 +
 +
::Thanks, Thestinger but the title still only mentions the first {{ic|git}} command, while the other example should use {{ic|git rev-parse}} as well. --[[User:Det|Det]] ([[User talk:Det|talk]]) 13:15, 11 April 2013 (UTC)

Revision as of 13:15, 11 April 2013

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. https://github.com/matplotlib/matplotlib.git instead of git://github.com/matplotlib/matplotlib.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)
}
- [...]
1142

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)
}

1142

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

1142.a17a017

--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)