Pacman

Historical bug tracker for the Pacman package manager.

The pacman bug tracker has moved to gitlab:
https://gitlab.archlinux.org/pacman/pacman/-/issues

This tracker remains open for interaction with historical bugs during the transition period. Any new bugs reports will be closed without further action.
Tasklist

FS#10905 - [patch] makepkg check for latest svn revision is slow on large repositories

Attached to Project: Pacman
Opened by . (Thralas) - Sunday, 13 July 2008, 23:40 GMT
Last edited by Dan McGee (toofishes) - Sunday, 20 July 2008, 15:11 GMT
Task Type Bug Report
Category makepkg
Status Closed
Assigned To Dan McGee (toofishes)
Allan McRae (Allan)
Architecture All
Severity Low
Priority Normal
Reported Version git
Due in Version 3.2.0
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Prior to installing or building a package which utilizes SVN sources makepkg will retrieve the latest revision number. Currently 'svn log' is used to determine the latest revision, which seems fine for smaller SVN repositories (e.g. Google's protobuf project), however on large ones (such as the Mono repository) the command seems to take a huge amount of time.

'svn info' provides the latest revision as well, being slightly slower on small repositories, but the time required seems to be consistent, as opposed to 'svn log'.

Example:
`time svn log http://protobuf.googlecode.com/svn/trunk/ --limit 1` -> ~2.0s
`time svn info http://protobuf.googlecode.com/svn/trunk/` -> ~2.5s

`time svn log svn://anonsvn.mono-project.com/source/trunk/mono --limit 1` -> ~35.0s
`time svn info svn://anonsvn.mono-project.com/source/trunk/mono` -> 3.5s

I've attached a git-diff which replaces 'svn log' by 'svn info'.
   makepkg.svn-info.patch (0.6 KiB)
This task depends upon

Closed by  Dan McGee (toofishes)
Sunday, 20 July 2008, 15:11 GMT
Reason for closing:  Implemented
Additional comments about closing:  Implemented in git by commit 0bf66b097fc8ccb394cb910764d171476e15d124
Comment by Dan McGee (toofishes) - Sunday, 13 July 2008, 23:48 GMT
Gotta account for this, but of course all they have is that stupid --xml option, no text-based machine parsible one:

$ LANG=de_DE.utf8 svn info http://protobuf.googlecode.com/svn/trunk/
Pfad: trunk
URL: http://protobuf.googlecode.com/svn/trunk
Basis des Projektarchivs: http://protobuf.googlecode.com/svn
UUID des Projektarchivs: 630680e5-0e50-0410-840e-4b1c322b438d
Revision: 2
Knotentyp: Verzeichnis
Letzter Autor: temporal
Letzte geänderte Rev: 2
Letztes Änderungsdatum: 2008-07-09 21:12:20 -0500 (Mi, 09 Jul 2008)
Comment by Allan McRae (Allan) - Monday, 14 July 2008, 04:29 GMT
Can we not just stick a LANG=C in the front of it? I have no idea how portable that is...
Comment by Xavier (shining) - Tuesday, 15 July 2008, 18:54 GMT
Yeah, just stick LC_ALL=C in front of it and it should work.
+1 for this way if it gives a 10x speedup with some repo.

Loading...