Bug 344943

Summary:	app-portage/gentoolkit-dev: echangelog should check for CVS/svn first
Product:	Portage Development	Reporter:	Michał Górny <mgorny>
Component:	Tools	Assignee:	Portage Tools Team <tools-portage>
Status:	RESOLVED OBSOLETE
Severity:	enhancement
Priority:	High
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---

Description Michał Górny archtester

2010-11-10 15:47:44 UTC

Currently, the VCS detection code seems to check for VCSes in a lexical order, checking the full directory tree backwards for each of them. I think it would be better to either:

a) switch the loops -- i.e. check all VCSes for each dir, not all dirs for a VCS,
b) at least move CVS/svn checks elsewhere as they duplicate their directories in each repo dir.

This is a case e.g. whenever a subversion repositories is checked out within a larger git repo [1].

[1] https://github.com/mgorny/sunrise-commit/issues#issue/4

Comment 1 Christian Ruppert (idl0r) gentoo-dev

2012-04-09 01:47:37 UTC

subversion no longer uses the per directory .svn, it leaves only CVS and I'm not sure if it's worth to change.